[POWERPC] Improve robustness of the UIC cascade handler
authorDavid Gibson <david@gibson.dropbear.id.au>
Tue, 14 Aug 2007 03:52:42 +0000 (13:52 +1000)
committerPaul Mackerras <paulus@samba.org>
Fri, 17 Aug 2007 01:02:06 +0000 (11:02 +1000)
At present the cascade interrupt handler for the UIC (interrupt
controller on 4xx embedded chips) will misbehave badly if it is called
spuriously - that is if the handler is invoked when no interrupts are
asserted in the child UIC.

Although spurious interrupts shouldn't happen, it's good to behave
robustly if they do.  This patch does so by checking for and ignoring
spurious interrupts.

Signed-off-by: Valentine Barshak <vbarshak@ru.mvista.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/sysdev/uic.c

index 22c219e448d4f95c46749278950ad5934b4dabf8..47180b3fca5675519591f5f897d03f38dbb1e1cf 100644 (file)
@@ -266,6 +266,9 @@ irqreturn_t uic_cascade(int virq, void *data)
        int subvirq;
 
        msr = mfdcr(uic->dcrbase + UIC_MSR);
+       if (!msr) /* spurious interrupt */
+               return IRQ_HANDLED;
+
        src = 32 - ffs(msr);
 
        subvirq = irq_linear_revmap(uic->irqhost, src);