diff options
author | Zwane Mwaikambo <zwane@arm.linux.org.uk> | 2006-01-11 16:43:09 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-11 22:01:57 -0500 |
commit | 329d400f47ddfe8ff599823d739c5c5565da3207 (patch) | |
tree | 903b45e7ebcccbbd8172e207315c036ceb70f6e8 /include/asm-i386 | |
parent | 73ca5358aab55e2e29993a31f3776c54c05ae729 (diff) |
[PATCH] x86_64/i386: Remove preempt disable calls in lowlevel IPI
I noticed that some lowlevel send_IPI_mask helpers had a hotplug/preempt
race whereupon the cpu_online_map was read before disabling preemption;
...
cpumask_t mask = cpu_online_map;
int cpu = get_cpu();
cpu_clear(cpu, mask);
...
But then i realised that there is no need for these lowlevel functions to
be going through all this trouble when all the callers are already made
hotplug/preempt safe.
Signed-off-by: Zwane Mwaikambo <zwane@arm.linux.org.uk>
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/asm-i386')
-rw-r--r-- | include/asm-i386/mach-default/mach_ipi.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/asm-i386/mach-default/mach_ipi.h b/include/asm-i386/mach-default/mach_ipi.h index cc756a67cd63..a1d0072e36bc 100644 --- a/include/asm-i386/mach-default/mach_ipi.h +++ b/include/asm-i386/mach-default/mach_ipi.h | |||
@@ -15,11 +15,9 @@ static inline void __local_send_IPI_allbutself(int vector) | |||
15 | { | 15 | { |
16 | if (no_broadcast) { | 16 | if (no_broadcast) { |
17 | cpumask_t mask = cpu_online_map; | 17 | cpumask_t mask = cpu_online_map; |
18 | int this_cpu = get_cpu(); | ||
19 | 18 | ||
20 | cpu_clear(this_cpu, mask); | 19 | cpu_clear(smp_processor_id(), mask); |
21 | send_IPI_mask(mask, vector); | 20 | send_IPI_mask(mask, vector); |
22 | put_cpu(); | ||
23 | } else | 21 | } else |
24 | __send_IPI_shortcut(APIC_DEST_ALLBUT, vector); | 22 | __send_IPI_shortcut(APIC_DEST_ALLBUT, vector); |
25 | } | 23 | } |