diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2010-07-20 14:34:50 +0200 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2010-07-20 19:15:07 +0200 |
| commit | 59fdcc01a4952a521dbae619254d1d4e3f0651e7 (patch) | |
| tree | 3ffebfb3294aab2c2cf5ffe01a3e18917319a30b | |
| parent | 5805f5ac2a7b6ca1fb32e406523a00ac1eb45ffb (diff) | |
| download | lwn-59fdcc01a4952a521dbae619254d1d4e3f0651e7.tar.gz lwn-59fdcc01a4952a521dbae619254d1d4e3f0651e7.zip | |
powerpc: cpu-hotplug: Prevent softirq wakeup on wrong CPU
When the plugged CPU sets the online flag it enables interrupts and
goes idle. When an interrupt happens and tried to wakeup a softirq
thread before the other cpu sets the active flag, then the softirq
thread is put on one of the active cpus. Prevent this by waiting for
the cpu_active bit.
Temporary workaround. Needs more thought.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
| -rw-r--r-- | arch/powerpc/kernel/smp.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c index c2ee14498077..c09bbc72640b 100644 --- a/arch/powerpc/kernel/smp.c +++ b/arch/powerpc/kernel/smp.c @@ -541,6 +541,9 @@ int __devinit start_secondary(void *unused) of_node_put(l2_cache); ipi_call_unlock(); + while (!cpumask_test_cpu(smp_processor_id(), cpu_active_mask)) + cpu_relax(); + local_irq_enable(); cpu_idle(); |