Re: [dpdk-dev] [PATCH] Add build option to enable/disable AVX2 support

[Bruce Richardson <bruce.richardson@intel.com>]

On Mon, Oct 26, 2020 at 03:30:27PM +0100, Felix Moessbauer wrote:
> This patch introduces a meson option to disable the AVX2 support.
> If the build should be for a target without AVX2 support, the
> know can be turned to false, even if the compiler supports AVX2.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>  meson.build       | 3 ++-
>  meson_options.txt | 1 +
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/meson.build b/meson.build
> index 0ee98be..2e2380c 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -18,7 +18,8 @@ pktgen_conf = configuration_data()
>  cc = meson.get_compiler('c')
>  
>  add_project_arguments('-march=native', language: 'c')
> -if cc.has_argument('-mavx2')
> +
> +if get_option('enable-avx2') and cc.has_argument('-mavx2')
>  	add_project_arguments('-mavx2', language: 'c')
>  endif
>  add_project_arguments('-DALLOW_EXPERIMENTAL_API', language: 'c')

Hi,

This code does not appear in the upstream DPDK project [1]. This looks like
a pktgen patch instead, is this the case? If so, please put pktgen
somewhere in the patch title to make it easier to disposition.

Thanks,
/Bruce

[1] http://git.dpdk.org/dpdk/tree/meson.build