aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGlauber Costa <gcosta@redhat.com>2008-03-03 12:13:04 -0500
committerIngo Molnar <mingo@elte.hu>2008-04-17 11:40:56 -0400
commitfc25da9ec6c910976b76c70f7604a838679f75b2 (patch)
tree5bcf4c5fe59de6a27367c55948c7553938b43bc3
parent70708a18e834fd709a4f497bb419ec84d1eb3511 (diff)
x86: remove vector_lock around cpu_online_map
This lock does not protect cpu_online_map, so its length can be shortened, and in some cases, removed. Signed-off-by: Glauber Costa <gcosta@redhat.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r--arch/x86/kernel/smpboot_64.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/x86/kernel/smpboot_64.c b/arch/x86/kernel/smpboot_64.c
index 1a592400d94f..ca3a3c5b64fe 100644
--- a/arch/x86/kernel/smpboot_64.c
+++ b/arch/x86/kernel/smpboot_64.c
@@ -261,9 +261,9 @@ void __cpuinit start_secondary(void)
261 /* 261 /*
262 * Allow the master to continue. 262 * Allow the master to continue.
263 */ 263 */
264 spin_unlock(&vector_lock);
264 cpu_set(smp_processor_id(), cpu_online_map); 265 cpu_set(smp_processor_id(), cpu_online_map);
265 per_cpu(cpu_state, smp_processor_id()) = CPU_ONLINE; 266 per_cpu(cpu_state, smp_processor_id()) = CPU_ONLINE;
266 spin_unlock(&vector_lock);
267 267
268 unlock_ipi_call_lock(); 268 unlock_ipi_call_lock();
269 269
@@ -879,10 +879,8 @@ int __cpu_disable(void)
879 local_irq_disable(); 879 local_irq_disable();
880 remove_siblinginfo(cpu); 880 remove_siblinginfo(cpu);
881 881
882 spin_lock(&vector_lock);
883 /* It's now safe to remove this processor from the online map */ 882 /* It's now safe to remove this processor from the online map */
884 cpu_clear(cpu, cpu_online_map); 883 cpu_clear(cpu, cpu_online_map);
885 spin_unlock(&vector_lock);
886 remove_cpu_from_maps(); 884 remove_cpu_from_maps();
887 fixup_irqs(cpu_online_map); 885 fixup_irqs(cpu_online_map);
888 return 0; 886 return 0;