sparc64: Do not disable interrupts in nmi_cpu_busy()

Message ID
State Accepted
Delegated to: David Miller
Headers show

Commit Message

David Miller Aug. 12, 2014, 3:46 a.m.
nmi_cpu_busy() is a SMP function call that just makes sure that all of the
cpus are spinning using cpu cycles while the NMI test runs.

It does not need to disable IRQs because we just care about NMIs executing
which will even with 'normal' IRQs disabled.

It is not legal to enable hard IRQs in a SMP cross call, in fact this bug
triggers the BUG check in irq_work_run_list():


Because now irq_work_run() is invoked from the tail of

Signed-off-by: David S. Miller <>

This should fix bootup failures with Linus's current tree.

 arch/sparc/kernel/nmi.c | 1 -
 1 file changed, 1 deletion(-)


diff --git a/arch/sparc/kernel/nmi.c b/arch/sparc/kernel/nmi.c
index 3370945..5b1151d 100644
--- a/arch/sparc/kernel/nmi.c
+++ b/arch/sparc/kernel/nmi.c
@@ -130,7 +130,6 @@  static inline unsigned int get_nmi_count(int cpu)
 static __init void nmi_cpu_busy(void *data)
-	local_irq_enable_in_hardirq();
 	while (endflag == 0)