DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH v1] config/arm: add armv7 native config
@ 2021-11-18 10:45 Juraj Linkeš
  2021-12-31  7:11 ` Ruifeng Wang
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Juraj Linkeš @ 2021-11-18 10:45 UTC (permalink / raw)
  To: thomas, david.marchand, bruce.richardson, Honnappa.Nagarahalli,
	Ruifeng.Wang, ferruh.yigit, christian.ehrhardt
  Cc: dev, Juraj Linkeš

Arvm7 native build fails with this error:
../config/meson.build:364:1: ERROR: Problem encountered:
Number of CPU cores not specified.

This is because RTE_MAX_LCORE is not set. We also need to set
RTE_MAX_NUMA_NODES in armv7 native builds.

Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU counts detection")

Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
 config/arm/meson.build | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/config/arm/meson.build b/config/arm/meson.build
index 213324d262..57980661b2 100644
--- a/config/arm/meson.build
+++ b/config/arm/meson.build
@@ -432,6 +432,8 @@ if dpdk_conf.get('RTE_ARCH_32')
     else
         # armv7 build
         dpdk_conf.set('RTE_ARCH_ARMv7', true)
+        dpdk_conf.set('RTE_MAX_LCORE', 128)
+        dpdk_conf.set('RTE_MAX_NUMA_NODES', 8)
         # the minimum architecture supported, armv7-a, needs the following,
         machine_args += '-mfpu=neon'
     endif
-- 
2.20.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: [PATCH v1] config/arm: add armv7 native config
  2021-11-18 10:45 [PATCH v1] config/arm: add armv7 native config Juraj Linkeš
@ 2021-12-31  7:11 ` Ruifeng Wang
  2021-12-31 17:23 ` Stephen Hemminger
  2022-01-25 10:08 ` [PATCH v2] " Juraj Linkeš
  2 siblings, 0 replies; 5+ messages in thread
From: Ruifeng Wang @ 2021-12-31  7:11 UTC (permalink / raw)
  To: Juraj Linkeš,
	thomas, david.marchand, bruce.richardson, Honnappa Nagarahalli,
	ferruh.yigit, christian.ehrhardt
  Cc: dev, nd

> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Sent: Thursday, November 18, 2021 6:46 PM
> To: thomas@monjalon.net; david.marchand@redhat.com;
> bruce.richardson@intel.com; Honnappa Nagarahalli
> <Honnappa.Nagarahalli@arm.com>; Ruifeng Wang
> <Ruifeng.Wang@arm.com>; ferruh.yigit@intel.com;
> christian.ehrhardt@canonical.com
> Cc: dev@dpdk.org; Juraj Linkeš <juraj.linkes@pantheon.tech>
> Subject: [PATCH v1] config/arm: add armv7 native config
> 
> Arvm7 native build fails with this error:

Typo, 'Armv7'

> ../config/meson.build:364:1: ERROR: Problem encountered:
> Number of CPU cores not specified.
> 
> This is because RTE_MAX_LCORE is not set. We also need to set
> RTE_MAX_NUMA_NODES in armv7 native builds.
> 
> Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU counts detection")
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
>  config/arm/meson.build | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/config/arm/meson.build b/config/arm/meson.build index
> 213324d262..57980661b2 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -432,6 +432,8 @@ if dpdk_conf.get('RTE_ARCH_32')
>      else
>          # armv7 build
>          dpdk_conf.set('RTE_ARCH_ARMv7', true)
> +        dpdk_conf.set('RTE_MAX_LCORE', 128)
> +        dpdk_conf.set('RTE_MAX_NUMA_NODES', 8)
>          # the minimum architecture supported, armv7-a, needs the following,
>          machine_args += '-mfpu=neon'
>      endif
> --
> 2.20.1
Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v1] config/arm: add armv7 native config
  2021-11-18 10:45 [PATCH v1] config/arm: add armv7 native config Juraj Linkeš
  2021-12-31  7:11 ` Ruifeng Wang
@ 2021-12-31 17:23 ` Stephen Hemminger
  2022-01-25 10:08 ` [PATCH v2] " Juraj Linkeš
  2 siblings, 0 replies; 5+ messages in thread
From: Stephen Hemminger @ 2021-12-31 17:23 UTC (permalink / raw)
  To: Juraj Linkeš
  Cc: thomas, david.marchand, bruce.richardson, Honnappa.Nagarahalli,
	Ruifeng.Wang, ferruh.yigit, christian.ehrhardt, dev

On Thu, 18 Nov 2021 11:45:51 +0100
Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:

> Arvm7 native build fails with this error:
> ../config/meson.build:364:1: ERROR: Problem encountered:
> Number of CPU cores not specified.
> 
> This is because RTE_MAX_LCORE is not set. We also need to set
> RTE_MAX_NUMA_NODES in armv7 native builds.
> 
> Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU counts detection")
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
>  config/arm/meson.build | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/config/arm/meson.build b/config/arm/meson.build
> index 213324d262..57980661b2 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -432,6 +432,8 @@ if dpdk_conf.get('RTE_ARCH_32')
>      else
>          # armv7 build
>          dpdk_conf.set('RTE_ARCH_ARMv7', true)
> +        dpdk_conf.set('RTE_MAX_LCORE', 128)
> +        dpdk_conf.set('RTE_MAX_NUMA_NODES', 8)

In DPDK 19.11 the ARMv7 config says armv7 does not support NUMA.
Pleas set NUMA_NODES to 1

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2] config/arm: add armv7 native config
  2021-11-18 10:45 [PATCH v1] config/arm: add armv7 native config Juraj Linkeš
  2021-12-31  7:11 ` Ruifeng Wang
  2021-12-31 17:23 ` Stephen Hemminger
@ 2022-01-25 10:08 ` Juraj Linkeš
  2022-02-10 16:34   ` Thomas Monjalon
  2 siblings, 1 reply; 5+ messages in thread
From: Juraj Linkeš @ 2022-01-25 10:08 UTC (permalink / raw)
  To: thomas, david.marchand, bruce.richardson, Honnappa.Nagarahalli,
	Ruifeng.Wang, ferruh.yigit, christian.ehrhardt, stephen
  Cc: dev, Juraj Linkeš

Armv7 native build fails with this error:
../config/meson.build:364:1: ERROR: Problem encountered:
Number of CPU cores not specified.

This is because RTE_MAX_LCORE is not set. We also need to set
RTE_MAX_NUMA_NODES in armv7 native builds.

Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU counts detection")

Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>
---
 config/arm/meson.build | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/config/arm/meson.build b/config/arm/meson.build
index c3a3f2faaf..a0e91c0f6b 100644
--- a/config/arm/meson.build
+++ b/config/arm/meson.build
@@ -434,6 +434,8 @@ if dpdk_conf.get('RTE_ARCH_32')
         # armv7 build
         dpdk_conf.set('RTE_ARCH_ARMv7', true)
         dpdk_conf.set('RTE_ARCH', 'armv7')
+        dpdk_conf.set('RTE_MAX_LCORE', 128)
+        dpdk_conf.set('RTE_MAX_NUMA_NODES', 1)
         # the minimum architecture supported, armv7-a, needs the following,
         machine_args += '-mfpu=neon'
     endif
-- 
2.20.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] config/arm: add armv7 native config
  2022-01-25 10:08 ` [PATCH v2] " Juraj Linkeš
@ 2022-02-10 16:34   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2022-02-10 16:34 UTC (permalink / raw)
  To: Honnappa.Nagarahalli, Juraj Linkeš
  Cc: david.marchand, bruce.richardson, Ruifeng.Wang, ferruh.yigit,
	christian.ehrhardt, stephen, dev

25/01/2022 11:08, Juraj Linkeš:
> Armv7 native build fails with this error:
> ../config/meson.build:364:1: ERROR: Problem encountered:
> Number of CPU cores not specified.
> 
> This is because RTE_MAX_LCORE is not set. We also need to set
> RTE_MAX_NUMA_NODES in armv7 native builds.
> 
> Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU counts detection")
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>

Applied with backport tag, thanks.




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-02-10 16:35 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-18 10:45 [PATCH v1] config/arm: add armv7 native config Juraj Linkeš
2021-12-31  7:11 ` Ruifeng Wang
2021-12-31 17:23 ` Stephen Hemminger
2022-01-25 10:08 ` [PATCH v2] " Juraj Linkeš
2022-02-10 16:34   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).