From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so1.wedos.net (wes1-so1.wedos.net [46.28.106.15]) by dpdk.org (Postfix) with ESMTP id 8E3AC5685 for ; Sat, 19 Mar 2016 20:58:10 +0100 (CET) Received: from pcviktorin.fit.vutbr.cz (pcviktorin.fit.vutbr.cz [147.229.13.147]) by wes1-so1.wedos.net (Postfix) with ESMTPSA id 3qSCZB111zz4f3; Sat, 19 Mar 2016 20:58:10 +0100 (CET) From: Jan Viktorin To: dev@dpdk.org Cc: Jan Viktorin , thomas.monjalon@6wind.com, jerin.jacob@caviumnetworks.com, tomaszx.kulasek@intel.com, jianbo.liu@linaro.org, david.marchand@6wind.com Date: Sat, 19 Mar 2016 20:58:01 +0100 Message-Id: <1458417485-29436-1-git-send-email-viktorin@rehivetech.com> X-Mailer: git-send-email 2.7.0 In-Reply-To: <1458379590-18618-1-git-send-email-viktorin@rehivetech.com> References: <1458379590-18618-1-git-send-email-viktorin@rehivetech.com> Subject: [dpdk-dev] [PATCH v3 0/4] arm: detect NEON by RTE_MACHINE_CPUFLAG_NEON flag only X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 19 Mar 2016 19:58:10 -0000 Hello, finally, I've broken the original patch into 4 pieces as it solves more issues and not just a single one. * As Thomas have already mentioned, the CONFIG_RTE_ARCH_ARM_NEON is confusing. So, I've decided to remove it entirely and provide another option for a more specific purpose: CONFIG_RTE_ARCH_ARM_NEON_MEMCPY. * The RTE_MACHINE_CPUFLAG_NEON detection is now based on __ARM_NEON as only this compiler definition gives us the arm_neon.h and is compatible with arm64. In DPDK, the RTE_MACHINE_CPUFLAG_NEON should be prefered over the __ARM_NEON. I'd recommend the same for x86 code (__SSE2__)... History: v2 * fix l3fwm_em.c to refer RTE_MACHINE_CPUFLAG_NEON instead of __ARM_NEON v3 * divided into 4 patches as there are more independent problems * compiles well for armv7 * (probably) fixes RTE_MACHINE_CPUFLAG_NEON detection on arm64 Jan Viktorin (4): arm: remove CONFIG_RTE_ARCH_ARM_NEON arm: detect NEON cpu feature by checking __ARM_NEON arm: detect NEON by checking RTE_MACHINE_CPUFLAG_NEON eal/arm: introduce CONFIG_RTE_ARCH_ARM_NEON_MEMCPY config/defconfig_arm-armv7a-linuxapp-gcc | 2 +- config/defconfig_arm64-armv8a-linuxapp-gcc | 1 - examples/l3fwd/l3fwd_em.c | 2 +- lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h | 8 ++++++-- mk/machine/armv7a/rte.vars.mk | 2 -- mk/rte.cpuflags.mk | 2 +- 6 files changed, 9 insertions(+), 8 deletions(-) -- 2.7.0