[PATCH] config/arm: make SoC-generic more generic

Jerin Jacob jerinjacobk at gmail.com
Tue Nov 15 08:08:16 CET 2022


On Tue, Nov 15, 2022 at 7:46 AM Chengwen Feng <fengchengwen at huawei.com> wrote:
>
> The non-generic soc has 1280 max-lcores and 32 max-numas, but the
> implementer-generic still has 256 max-lcores and 4 max-numas, which may
> result in restrictions on the use of binaries compiled by soc-generic
> in these systems.
>
> This patch changes the soc-generic max-lcores to 1280 and max-numas to
> 32 to make it more generic.

I think, it is not specific to ARM architecture issue. Even x86 config
set as below.
dpdk_conf.set('RTE_MAX_LCORE', 128)
dpdk_conf.set('RTE_MAX_NUMA_NODES', 32)

I think, it makes sense to keep generic as "most commonly used max"
per application, expectation cases can have separate config.

Also, this is for SINGLE application, even if you have 1280 cores a
single application most likely won't use that.
So in that way this patch is not needed.

>
> Also, because single numa is a special case of multiple numas, mark
> soc-generic's numa flag as true.
>
> Signed-off-by: Chengwen Feng <fengchengwen at huawei.com>
> ---
>  config/arm/meson.build | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/config/arm/meson.build b/config/arm/meson.build
> index 6442ec9596..97b91cf609 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -32,8 +32,8 @@ implementer_generic = {
>      'flags': [
>          ['RTE_MACHINE', '"armv8a"'],
>          ['RTE_USE_C11_MEM_MODEL', true],
> -        ['RTE_MAX_LCORE', 256],
> -        ['RTE_MAX_NUMA_NODES', 4]
> +        ['RTE_MAX_LCORE', 1280],
> +        ['RTE_MAX_NUMA_NODES', 32]
>      ],
>      'part_number_config': {
>          'generic': {
> @@ -271,7 +271,8 @@ implementers = {
>  soc_generic = {
>      'description': 'Generic un-optimized build for armv8 aarch64 exec mode',
>      'implementer': 'generic',
> -    'part_number': 'generic'
> +    'part_number': 'generic',
> +    'numa': true
>  }
>
>  soc_generic_aarch32 = {
> --
> 2.17.1
>


More information about the dev mailing list