[dpdk-dev] [PATCH] config/x86: add support for AMD platform

[Song, Keesang]

[AMD Official Use Only]

It would be great to make max lcores dynamically allocated.
Thanks so much Thomas for your confirmation that this is to be merge in 21.11-rc2.

Best,
Keesang

-----Original Message-----
From: Thomas Monjalon <thomas at monjalon.net>
Sent: Tuesday, November 2, 2021 8:19 AM
To: Aman Kumar <aman.kumar at vvdntech.in>
Cc: dev at dpdk.org; Song, Keesang <Keesang.Song at amd.com>; david.marchand at redhat.com
Subject: Re: [PATCH] config/x86: add support for AMD platform

[CAUTION: External Email]

02/11/2021 15:52, Aman Kumar:
> -Dcpu_instruction_set=znverX meson option can be used to build dpdk
> for AMD platforms. Supported options are znver1, znver2 and znver3.
>
> Signed-off-by: Aman Kumar <aman.kumar at vvdntech.in>
> ---
> +# AMD platform support
> +if get_option('cpu_instruction_set') == 'znver1'
> +    dpdk_conf.set('RTE_MAX_LCORE', 256) elif
> +get_option('cpu_instruction_set') == 'znver2'
> +    dpdk_conf.set('RTE_MAX_LCORE', 512) elif
> +get_option('cpu_instruction_set') == 'znver3'
> +    dpdk_conf.set('RTE_MAX_LCORE', 512) endif

Ideally we should try getting rid of this config and make all structs depending on max lcores allocated dynamically.
I think it may be an interesting target for next year 22.11 release.

About the patch itself, I am OK to merge it in 21.11-rc2.