diff options
author | Nathan Fontenot <nfont@linux.vnet.ibm.com> | 2013-04-29 03:45:36 +0000 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2013-04-30 15:59:35 +1000 |
commit | 601abdc3b4773afb9a80afc9c4c024724b7c5f4e (patch) | |
tree | 61994f55775088a18048c66f001432237e1632bf | |
parent | 177c19237b886a2ab71011c9e6f40190e5c24b83 (diff) | |
download | lwn-601abdc3b4773afb9a80afc9c4c024724b7c5f4e.tar.gz lwn-601abdc3b4773afb9a80afc9c4c024724b7c5f4e.zip |
powerpc/pseries: Correct builds break when CONFIG_SMP not defined
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>
-rw-r--r-- | arch/powerpc/mm/numa.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c index 490e39c8d8f6..c3bd046f801a 100644 --- a/arch/powerpc/mm/numa.c +++ b/arch/powerpc/mm/numa.c @@ -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); |