]> git.baikalelectronics.ru Git - kernel.git/commit
cpu/hotplug: Fix smpboot thread ordering
authorThomas Gleixner <tglx@linutronix.de>
Thu, 10 Mar 2016 19:42:08 +0000 (20:42 +0100)
committerThomas Gleixner <tglx@linutronix.de>
Thu, 10 Mar 2016 20:21:59 +0000 (21:21 +0100)
commit0e8aac5a378a50ab7a88e00d9e7761790a8c1317
tree5f22c738d4a79fb4ed816e3453445f55d0e6626e
parent970cfffef95e0ba24393727fdf1190fb96081e6b
cpu/hotplug: Fix smpboot thread ordering

Commit 33a189ee1c6d moved the smpboot thread park/unpark invocation to the
state machine. The move of the unpark invocation was premature as it depends
on work in progress patches.

As a result cpu down can fail, because rcu synchronization in takedown_cpu()
eventually requires a functional softirq thread. I never encountered the
problem in testing, but 0day testing managed to provide a reliable reproducer.

Remove the smpboot_threads_park() call from the state machine for now and put
it back into the original place after the rcu synchronization.

I'm embarrassed as I knew about the dependency and still managed to get it
wrong. Hotplug induced brain melt seems to be the only sensible explanation
for that.

Fixes: 33a189ee1c6d "cpu/hotplug: Unpark smpboot threads from the state machine"
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Peter Zijlstra <peterz@infradead.org>
kernel/cpu.c