x86/topo: Replace x86_has_numa_in_package

.. with the brand spanking new topology_num_nodes_per_package().

Having the topology setup determine this value during MADT/SRAT parsing before
SMP bringup avoids having to detect this situation when building the SMP
topology masks.

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Ingo Molnar <mingo@kernel.org>
Tested-by: Tony Luck <tony.luck@intel.com>
Tested-by: K Prateek Nayak <kprateek.nayak@amd.com>
Tested-by: Zhang Rui <rui.zhang@intel.com>
Tested-by: Chen Yu <yu.c.chen@intel.com>
Tested-by: Kyle Meyer <kyle.meyer@hpe.com>
Link: https://patch.msgid.link/20260303110100.123701837@infradead.org

diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
index 5cd6950ab672a05e16d5d5b136c30ffcdad1f2d6..db3e481cdbb2e63171ce551d548aed35741b399f 100644 (file)
--- a/arch/x86/kernel/smpboot.c
+++ b/arch/x86/kernel/smpboot.c
@@ -468,13 +468,6 @@ static int x86_cluster_flags(void)
 }
 #endif
 
-/*
- * Set if a package/die has multiple NUMA nodes inside.
- * AMD Magny-Cours, Intel Cluster-on-Die, and Intel
- * Sub-NUMA Clustering have this.
- */
-static bool x86_has_numa_in_package;
-
 static struct sched_domain_topology_level x86_topology[] = {
        SDTL_INIT(tl_smt_mask, cpu_smt_flags, SMT),
 #ifdef CONFIG_SCHED_CLUSTER
@@ -496,7 +489,7 @@ static void __init build_sched_topology(void)
         * PKG domain since the NUMA domains will auto-magically create the
         * right spanning domains based on the SLIT.
         */
-       if (x86_has_numa_in_package) {
+       if (topology_num_nodes_per_package() > 1) {
                unsigned int pkgdom = ARRAY_SIZE(x86_topology) - 2;
 
                memset(&x86_topology[pkgdom], 0, sizeof(x86_topology[pkgdom]));
@@ -550,7 +543,7 @@ int arch_sched_node_distance(int from, int to)
        case INTEL_GRANITERAPIDS_X:
        case INTEL_ATOM_DARKMONT_X:
 
-               if (!x86_has_numa_in_package || topology_max_packages() == 1 ||
+               if (topology_max_packages() == 1 || topology_num_nodes_per_package() == 1 ||
                    d < REMOTE_DISTANCE)
                        return d;
 
@@ -606,7 +599,7 @@ void set_cpu_sibling_map(int cpu)
                o = &cpu_data(i);
 
                if (match_pkg(c, o) && !topology_same_node(c, o))
-                       x86_has_numa_in_package = true;
+                       WARN_ON_ONCE(topology_num_nodes_per_package() == 1);
 
                if ((i == cpu) || (has_smt && match_smt(c, o)))
                        link_mask(topology_sibling_cpumask, cpu, i);