powerpc/smp: Lookup avail once per device tree node
authorMichael Ellerman <mpe@ellerman.id.au>
Wed, 14 Feb 2024 13:14:04 +0000 (00:14 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 14 Feb 2024 13:14:04 +0000 (00:14 +1100)
The of_device_is_available() check only needs to be done once per device
node, there's no need to repeat it for each thread. Move it out of the
loop.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20231229120107.2281153-3-mpe@ellerman.id.au
arch/powerpc/kernel/setup-common.c

index 9b142b9..375bade 100644 (file)
@@ -468,17 +468,16 @@ void __init smp_setup_cpu_maps(void)
 
                nthreads = len / sizeof(int);
 
+               bool avail = of_device_is_available(dn);
+               if (!avail)
+                       avail = !of_property_match_string(dn,
+                                       "enable-method", "spin-table");
+
                for (j = 0; j < nthreads && cpu < nr_cpu_ids; j++) {
-                       bool avail;
 
                        DBG("    thread %d -> cpu %d (hard id %d)\n",
                            j, cpu, be32_to_cpu(intserv[j]));
 
-                       avail = of_device_is_available(dn);
-                       if (!avail)
-                               avail = !of_property_match_string(dn,
-                                               "enable-method", "spin-table");
-
                        set_cpu_present(cpu, avail);
                        set_cpu_possible(cpu, true);
                        cpu_to_phys_id[cpu] = be32_to_cpu(intserv[j]);