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
