diff options
author | Keshavamurthy, Anil S <anil.s.keshavamurthy@intel.com> | 2007-05-02 13:27:18 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2007-05-02 13:27:18 -0400 |
commit | e8a72ffa3aa618fb25b5727c0e0ae939d30d66c0 (patch) | |
tree | f4e3cf88e1c11ad2ea4c4f3ca1f2db651c811d57 /arch | |
parent | 141a892f57972b01891df7036f567a70459c19ac (diff) |
[PATCH] i386: avoid checking for cpu gone when CONFIG_HOTPLUG_CPU not defined
Avoid checking for cpu gone in mm hot path when CONFIG_HOTPLUG_CPU is not
defined.
Signed-off-by: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
Signed-off-by: Andi Kleen <ak@suse.de>
Cc: Andi Kleen <ak@suse.de>
Cc: Gautham R Shenoy <ego@in.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/i386/kernel/smp.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/arch/i386/kernel/smp.c b/arch/i386/kernel/smp.c index f8667109db1c..f98c3ffd6fc3 100644 --- a/arch/i386/kernel/smp.c +++ b/arch/i386/kernel/smp.c | |||
@@ -359,10 +359,12 @@ void native_flush_tlb_others(const cpumask_t *cpumaskp, struct mm_struct *mm, | |||
359 | BUG_ON(cpu_isset(smp_processor_id(), cpumask)); | 359 | BUG_ON(cpu_isset(smp_processor_id(), cpumask)); |
360 | BUG_ON(!mm); | 360 | BUG_ON(!mm); |
361 | 361 | ||
362 | #ifdef CONFIG_HOTPLUG_CPU | ||
362 | /* If a CPU which we ran on has gone down, OK. */ | 363 | /* If a CPU which we ran on has gone down, OK. */ |
363 | cpus_and(cpumask, cpumask, cpu_online_map); | 364 | cpus_and(cpumask, cpumask, cpu_online_map); |
364 | if (cpus_empty(cpumask)) | 365 | if (unlikely(cpus_empty(cpumask))) |
365 | return; | 366 | return; |
367 | #endif | ||
366 | 368 | ||
367 | /* | 369 | /* |
368 | * i'm not happy about this global shared spinlock in the | 370 | * i'm not happy about this global shared spinlock in the |