[dpdk-dev] [PATCH v2 11/17] librte_acl: add AVX2 as new rte_acl_classify() method

Thomas Monjalon thomas.monjalon at 6wind.com
Mon Jan 19 18:22:40 CET 2015


2015-01-12 19:16, Konstantin Ananyev:
> +/*
> + * Select highest avaialbe classify method as default one.

Typo here.
Actually, I've seen few typos when browsing your big patchset
but I don't remember exactly where.
Maybe that a tool like codespell could help.

[...]

> +	else if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_SSE4_1))
> +#else
>  	if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_SSE4_1))
> +#endif

Minor nit, it could be
+	else
+#endif
 	if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_SSE4_1))

-- 
Thomas


More information about the dev mailing list