x86: remove vector_lock around cpu_online_map
authorGlauber Costa <gcosta@redhat.com>
Mon, 3 Mar 2008 17:13:04 +0000 (14:13 -0300)
committerIngo Molnar <mingo@elte.hu>
Thu, 17 Apr 2008 15:40:56 +0000 (17:40 +0200)
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>
arch/x86/kernel/smpboot_64.c

index 1a592400d94f6633f806e617c262a0b3b090d79f..ca3a3c5b64fea453ff8c7bb21925a9a4426922d6 100644 (file)
@@ -261,9 +261,9 @@ void __cpuinit start_secondary(void)
        /*
         * Allow the master to continue.
         */
+       spin_unlock(&vector_lock);
        cpu_set(smp_processor_id(), cpu_online_map);
        per_cpu(cpu_state, smp_processor_id()) = CPU_ONLINE;
-       spin_unlock(&vector_lock);
 
        unlock_ipi_call_lock();
 
@@ -879,10 +879,8 @@ int __cpu_disable(void)
        local_irq_disable();
        remove_siblinginfo(cpu);
 
-       spin_lock(&vector_lock);
        /* It's now safe to remove this processor from the online map */
        cpu_clear(cpu, cpu_online_map);
-       spin_unlock(&vector_lock);
        remove_cpu_from_maps();
        fixup_irqs(cpu_online_map);
        return 0;