On 1 December 2015 at 22:46, Jan Viktorin <viktorin at rehivetech.com> wrote: > On Tue, 1 Dec 2015 20:13:49 +0530 > Jerin Jacob <jerin.jacob at caviumnetworks.com> wrote: > >> > enum rte_acl_classify_alg alg = RTE_ACL_CLASSIFY_DEFAULT; >> > >> > -#ifdef RTE_ARCH_ARM64 >> > +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) >> > alg = RTE_ACL_CLASSIFY_NEON; >> >> I believe SIMD is optional in armv7. If true, select alg as >> RTE_ACL_CLASSIFY_NEON only when cpufeature NEON enabled. > > Yes. Or, probably, we can be happy with > > #if defined(__ARM_NEON_FP) > ... > #endif > > as it is currently done in rte_memcpy_32.h. > > Regards > Jan
Athough optional for armv7, I believe there is NEON in most of the popular armv7a chips. Anyway, I will add the checking... Thanks!
