DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] config/x86: increase lcores for AMD EPYC in 'native' scenario
@ 2025-02-28 20:35 Andre Muezerie
  0 siblings, 0 replies; only message in thread
From: Andre Muezerie @ 2025-02-28 20:35 UTC (permalink / raw)
  To: Bruce Richardson, Konstantin Ananyev, Sivaprasad Tummala,
	Morten Brørup
  Cc: dev, Andre Muezerie, stable

File config/meson.build has logic to move cpu_instruction_set from 'auto'
to 'native' or 'generic'.

When calling get_option('cpu_instruction_set') again from
config/x86/meson.build we get 'auto' again, instead of expected 'native'.

The fix is to not call get_option('cpu_instruction_set') again, just rely
on the work done by config/meson.build and trust that cpu_instruction_set
already has the correct value. Note that getting/processing
cpu_instruction_set is not x86-specific, so should be handled by
config/meson.build.

Fixes: fd8b058b322e ("config/x86: increase lcores for AMD EPYC cross build")
Cc: stable@dpdk.org

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 config/x86/meson.build | 1 -
 1 file changed, 1 deletion(-)

diff --git a/config/x86/meson.build b/config/x86/meson.build
index 47a5b0c04a..a6d917c78c 100644
--- a/config/x86/meson.build
+++ b/config/x86/meson.build
@@ -104,7 +104,6 @@ epyc_zen_cores = {
     '__znver1__':128
     }
 
-cpu_instruction_set = get_option('cpu_instruction_set')
 if cpu_instruction_set == 'native'
     foreach m:epyc_zen_cores.keys()
         if cc.get_define(m, args: machine_args) != ''
-- 
2.48.1.vfs.0.0


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2025-02-28 20:36 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-02-28 20:35 [PATCH] config/x86: increase lcores for AMD EPYC in 'native' scenario Andre Muezerie

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).