aboutsummaryrefslogtreecommitdiffstats
path: root/arch/parisc/kernel/smp.c
diff options
context:
space:
mode:
authorJames Bottomley <jejb@parisc-linux.org>2005-11-17 16:24:52 -0500
committerKyle McMartin <kyle@parisc-linux.org>2005-11-17 16:24:52 -0500
commit9a8b4584065dd241d6c2bf818e349986bd900b8e (patch)
treee801bca17874bdb64aa686f32d20dce76c07473e /arch/parisc/kernel/smp.c
parent6b1de9161e973bac8c4675db608fe4f38d2689bd (diff)
[PARISC] Make sure timer and IPI execute with interrupts disabled
Fix a longstanding smp bug The problem is that both the timer and ipi interrupts are being called with interrupts enabled, which isn't what anyone is expecting. The IPI issue has just started to show up by causing a BUG_ON in the slab debugging code. The timer issue never shows up because there's an eiem work around in our irq.c The fix is to label both these as SA_INTERRUPT which causes the generic irq code not to enable interrupts. I also suspect the smp_call_function timeouts we're seeing might be connected with the fact that we disable IPIs when handling any other type of interrupt. I've put a WARN_ON in the code for executing smp_call_function() with IPIs disabled. Signed-off-by: James Bottomley <jejb@parisc-linux.org> Signed-off-by: Kyle McMartin <kyle@parisc-linux.org>
Diffstat (limited to 'arch/parisc/kernel/smp.c')
-rw-r--r--arch/parisc/kernel/smp.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/parisc/kernel/smp.c b/arch/parisc/kernel/smp.c
index a9ecf6465784..268b0f2a328e 100644
--- a/arch/parisc/kernel/smp.c
+++ b/arch/parisc/kernel/smp.c
@@ -338,6 +338,10 @@ smp_call_function (void (*func) (void *info), void *info, int retry, int wait)
338 338
339 /* Can deadlock when called with interrupts disabled */ 339 /* Can deadlock when called with interrupts disabled */
340 WARN_ON(irqs_disabled()); 340 WARN_ON(irqs_disabled());
341
342 /* can also deadlock if IPIs are disabled */
343 WARN_ON((get_eiem() & (1UL<<(CPU_IRQ_MAX - IPI_IRQ))) == 0);
344
341 345
342 data.func = func; 346 data.func = func;
343 data.info = info; 347 data.info = info;