powerpc/pseries: Correct builds break when CONFIG_SMP not defined
authorNathan Fontenot <nfont@linux.vnet.ibm.com>
Mon, 29 Apr 2013 03:45:36 +0000 (03:45 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 30 Apr 2013 05:59:35 +0000 (15:59 +1000)
Correct build failure for powerpc/pseries builds with CONFIG_SMP not defined.

The function cpu_sibling_mask has no meaning (or definition) when CONFIG_SMP
is not defined. Additionally, the updating of NUMA affinity for a CPU in a UP
system doesn't really make sense.

This patch ifdef's out the code making the affinity updates for PRRN events to
fix the following build break.

arch/powerpc/mm/numa.c: In function ‘stage_topology_update’:
arch/powerpc/mm/numa.c:1535: error: implicit declaration of function ‘cpu_sibling_mask’
arch/powerpc/mm/numa.c:1535: warning: passing argument 3 of ‘cpumask_or’ makes pointer from integer without a cast
make[1]: *** [arch/powerpc/mm/numa.o] Error 1

Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/mm/numa.c

index 490e39c8d8f65de27534998420948d6827664baa..c3bd046f801a18b7cac8298375e040fdd91ecb37 100644 (file)
@@ -1530,6 +1530,8 @@ static void reset_topology_timer(void)
        mod_timer(&topology_timer, topology_timer.expires);
 }
 
+#ifdef CONFIG_SMP
+
 static void stage_topology_update(int core_id)
 {
        cpumask_or(&cpu_associativity_changes_mask,
@@ -1563,6 +1565,8 @@ static struct notifier_block dt_update_nb = {
        .notifier_call = dt_update_callback,
 };
 
+#endif
+
 /*
  * Start polling for associativity changes.
  */
@@ -1574,7 +1578,9 @@ int start_topology_update(void)
                if (!prrn_enabled) {
                        prrn_enabled = 1;
                        vphn_enabled = 0;
+#ifdef CONFIG_SMP
                        rc = of_reconfig_notifier_register(&dt_update_nb);
+#endif
                }
        } else if (firmware_has_feature(FW_FEATURE_VPHN) &&
                   get_lppaca()->shared_proc) {
@@ -1599,7 +1605,9 @@ int stop_topology_update(void)
 
        if (prrn_enabled) {
                prrn_enabled = 0;
+#ifdef CONFIG_SMP
                rc = of_reconfig_notifier_unregister(&dt_update_nb);
+#endif
        } else if (vphn_enabled) {
                vphn_enabled = 0;
                rc = del_timer_sync(&topology_timer);