On 10/13/2016 7:06 PM, Jerin Jacob wrote: > On Fri, Oct 14, 2016 at 12:17:05AM +0530, Hemant Agrawal wrote: >> if machine level CRC extension are available, offload the >> hash to machine provide functions e.g. armv8-a CRC extensions >> support it >> >> Signed-off-by: Hemant Agrawal <hemant.agrawal at nxp.com> >> Reviewed-by: Jerin Jacob <jerin.jacob at caviumnetworks.com> >> --- >> examples/l3fwd/l3fwd_em.c | 24 ++++++++++++++---------- >> 1 file changed, 14 insertions(+), 10 deletions(-) >> >> diff --git a/examples/l3fwd/l3fwd_em.c b/examples/l3fwd/l3fwd_em.c >> index 89a68e6..d92d0aa 100644 >> --- a/examples/l3fwd/l3fwd_em.c >> +++ b/examples/l3fwd/l3fwd_em.c >> @@ -57,13 +57,17 @@ >> >> #include "l3fwd.h" >> >> -#ifdef RTE_MACHINE_CPUFLAG_SSE4_2 >> +#if defined(RTE_MACHINE_CPUFLAG_SSE4_2) && >> defined(RTE_MACHINE_CPUFLAG_CRC32) > > The will evaluate as FALSE always. > > Please change to logical OR operation here. ie #if > defined(RTE_MACHINE_CPUFLAG_SSE4_2) || > defined(RTE_MACHINE_CPUFLAG_CRC32) > Oops! Will fix it.
>> +#define EM_HASH_CRC 1 >> +#endif >
