diff options
author | Stephen Boyd <sboyd@codeaurora.org> | 2013-05-20 20:57:16 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-21 11:56:21 -0400 |
commit | aa033810461ee56abbef6cef10aabd6b97f5caee (patch) | |
tree | 9c21df247fe421edf215515d3f2ab9c99ef46964 | |
parent | 0ddaa974258a4cde9a06c079bfd7863644e10d31 (diff) |
ARM: smp: Drop RCU_NONIDLE usage in cpu_die()
Before f7b861b7a6d9 ("arm: Use generic idle loop") ARM would kill the
CPU within the rcu idle section. Now that the rcu_idle_enter()/exit()
pair have been pushed lower down in the idle loop this is no longer true
and so using RCU_NONIDLE here is no longer necessary and also harmful
because RCU is not actually idle at this point.
Cc: Russell King <linux@arm.linux.org.uk>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | arch/arm/kernel/smp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c index 47ab90563bf4..550d63cef68e 100644 --- a/arch/arm/kernel/smp.c +++ b/arch/arm/kernel/smp.c | |||
@@ -251,7 +251,7 @@ void __ref cpu_die(void) | |||
251 | * this returns, power and/or clocks can be removed at any point | 251 | * this returns, power and/or clocks can be removed at any point |
252 | * from this CPU and its cache by platform_cpu_kill(). | 252 | * from this CPU and its cache by platform_cpu_kill(). |
253 | */ | 253 | */ |
254 | RCU_NONIDLE(complete(&cpu_died)); | 254 | complete(&cpu_died); |
255 | 255 | ||
256 | /* | 256 | /* |
257 | * Ensure that the cache lines associated with that completion are | 257 | * Ensure that the cache lines associated with that completion are |