powerpc/numa: Print debug statements only when required

Currently, a debug message gets printed every time an attempt to
add(remove) a CPU. However this is redundant if the CPU is already added
(removed) from the node.

Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20210826100521.412639-4-srikar@linux.vnet.ibm.com

diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c
index f4e52581713e4a3e11043f6420816da07bca6b11..52bc63ee6a6a74988ce1637c7afd753c7e5eb756 100644 (file)
--- a/arch/powerpc/mm/numa.c
+++ b/arch/powerpc/mm/numa.c
@@ -146,10 +146,10 @@ static void map_cpu_to_node(int cpu, int node)
 {
        update_numa_cpu_lookup_table(cpu, node);
 
-       pr_debug("adding cpu %d to node %d\n", cpu, node);
-
-       if (!(cpumask_test_cpu(cpu, node_to_cpumask_map[node])))
+       if (!(cpumask_test_cpu(cpu, node_to_cpumask_map[node]))) {
+               pr_debug("adding cpu %d to node %d\n", cpu, node);
                cpumask_set_cpu(cpu, node_to_cpumask_map[node]);
+       }
 }
 
 #if defined(CONFIG_HOTPLUG_CPU) || defined(CONFIG_PPC_SPLPAR)
@@ -157,10 +157,9 @@ static void unmap_cpu_from_node(unsigned long cpu)
 {
        int node = numa_cpu_lookup_table[cpu];
 
-       pr_debug("removing cpu %lu from node %d\n", cpu, node);
-
        if (cpumask_test_cpu(cpu, node_to_cpumask_map[node])) {
                cpumask_clear_cpu(cpu, node_to_cpumask_map[node]);
+               pr_debug("removing cpu %lu from node %d\n", cpu, node);
        } else {
                pr_warn("Warning: cpu %lu not found in node %d\n", cpu, node);
        }