From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so2.wedos.net (wes1-so2.wedos.net [46.28.106.16]) by dpdk.org (Postfix) with ESMTP id 5286A2E8D for ; Sat, 19 Mar 2016 10:26:31 +0100 (CET) Received: from pcviktorin.fit.vutbr.cz (pcviktorin.fit.vutbr.cz [147.229.13.147]) by wes1-so2.wedos.net (Postfix) with ESMTPSA id 3qRxYL6wXcz1qw; Sat, 19 Mar 2016 10:26:30 +0100 (CET) From: Jan Viktorin To: thomas.monjalon@6wind.com Cc: jerin.jacob@caviumnetworks.com, tomaszx.kulasek@intel.com, jianbo.liu@linaro.org, dev@dpdk.org, Jan Viktorin Date: Sat, 19 Mar 2016 10:26:30 +0100 Message-Id: <1458379590-18618-1-git-send-email-viktorin@rehivetech.com> X-Mailer: git-send-email 2.7.0 Subject: [dpdk-dev] [PATCH v2] 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 09:26:31 -0000 The RTE_MACHINE_CPUFLAG_NEON was only a result of the gcc testing. However, the target CPU may not support NEON or the user can disable to use it (as it does not always improve the performance). The RTE_MACHINE_CPUFLAG_NEON detection is now based on both, the __ARM_NEON_FP feature from gcc and CONFIG_RTE_ARCH_ARM_NEON from the .config. The memcpy implemention is driven by RTE_MACHINE_CPUFLAG_NEON, so the reason to disable NEON is hidden for the actual code. Signed-off-by: Jan Viktorin --- v2: fix l3fwm_em.c to refer RTE_MACHINE_CPUFLAG_NEON instead of __ARM_NEON --- examples/l3fwd/l3fwd_em.c | 2 +- lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h | 4 ++-- mk/machine/armv7a/rte.vars.mk | 2 +- mk/rte.cpuflags.mk | 2 ++ 4 files changed, 6 insertions(+), 4 deletions(-) diff --git a/examples/l3fwd/l3fwd_em.c b/examples/l3fwd/l3fwd_em.c index 0adf8f4..4983eed 100644 --- a/examples/l3fwd/l3fwd_em.c +++ b/examples/l3fwd/l3fwd_em.c @@ -250,7 +250,7 @@ em_mask_key(void *key, xmm_t mask) return _mm_and_si128(data, mask); } -#elif defined(__ARM_NEON) +#elif defined(RTE_MACHINE_CPUFLAG_NEON) static inline xmm_t em_mask_key(void *key, xmm_t mask) { diff --git a/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h b/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h index df47c0d..ad8bc65 100644 --- a/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h +++ b/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h @@ -42,7 +42,7 @@ extern "C" { #include "generic/rte_memcpy.h" -#ifdef __ARM_NEON_FP +#ifdef RTE_MACHINE_CPUFLAG_NEON /* ARM NEON Intrinsics are used to copy data */ #include @@ -325,7 +325,7 @@ rte_memcpy_func(void *dst, const void *src, size_t n) return memcpy(dst, src, n); } -#endif /* __ARM_NEON_FP */ +#endif /* RTE_MACHINE_CPUFLAG_NEON */ #ifdef __cplusplus } diff --git a/mk/machine/armv7a/rte.vars.mk b/mk/machine/armv7a/rte.vars.mk index 48d3979..7a167c1 100644 --- a/mk/machine/armv7a/rte.vars.mk +++ b/mk/machine/armv7a/rte.vars.mk @@ -62,6 +62,6 @@ ifdef CONFIG_RTE_ARCH_ARM_TUNE MACHINE_CFLAGS += -mtune=$(CONFIG_RTE_ARCH_ARM_TUNE) endif -ifeq ($(CONFIG_RTE_ARCH_ARM_NEON),y) +ifdef $(RTE_MACHINE_CPUFLAG_NEON) MACHINE_CFLAGS += -mfpu=neon endif diff --git a/mk/rte.cpuflags.mk b/mk/rte.cpuflags.mk index 19a3e7e..1947511 100644 --- a/mk/rte.cpuflags.mk +++ b/mk/rte.cpuflags.mk @@ -111,9 +111,11 @@ CPUFLAGS += VSX endif # ARM flags +ifeq ($(CONFIG_RTE_ARCH_ARM_NEON),y) ifneq ($(filter $(AUTO_CPUFLAGS),__ARM_NEON_FP),) CPUFLAGS += NEON endif +endif ifneq ($(filter $(AUTO_CPUFLAGS),__ARM_FEATURE_CRC32),) CPUFLAGS += CRC32 -- 2.7.0