diff options
author | Stephen Boyd <sboyd@codeaurora.org> | 2013-07-30 22:09:46 (GMT) |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-08-01 13:41:39 (GMT) |
commit | 44424c34049f41123a3a8b4853822f47f4ff03a2 (patch) | |
tree | 98e3b4564c5146e0692826f7940be33d2b645c87 /arch/arm/kernel/bios32.c | |
parent | 067e710b9a982a92cc8294d7fa0f1e924c65bba1 (diff) | |
download | linux-fsl-qoriq-44424c34049f41123a3a8b4853822f47f4ff03a2.tar.xz |
ARM: 7803/1: Fix deadlock scenario with smp_send_stop()
If one process calls sys_reboot and that process then stops other
CPUs while those CPUs are within a spin_lock() region we can
potentially encounter a deadlock scenario like below.
CPU 0 CPU 1
----- -----
spin_lock(my_lock)
smp_send_stop()
<send IPI> handle_IPI()
disable_preemption/irqs
while(1);
<PREEMPT>
spin_lock(my_lock) <--- Waits forever
We shouldn't attempt to run any other tasks after we send a stop
IPI to a CPU so disable preemption so that this task runs to
completion. We use local_irq_disable() here for cross-arch
consistency with x86.
Reported-by: Sundarajan Srinivasan <sundaraj@codeaurora.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/kernel/bios32.c')
0 files changed, 0 insertions, 0 deletions