Re: [dpdk-dev] [PATCH v6 1/5] efd: new Elastic Flow Distributor library

[Thomas Monjalon]

2017-01-16 19:21, Pablo de Lara:
> +#if defined(RTE_ARCH_X86)
> +       if (RTE_EFD_VALUE_NUM_BITS > 3 && 
> rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX2))

magic number spotted!
A comment would be probably helpful in this case.

> +               table->lookup_fn = EFD_LOOKUP_AVX2;
> +       else
> +#endif
> +               table->lookup_fn = EFD_LOOKUP_SCALAR;
>