microblaze: avoid interrupt race conditions
authorStephan Linz <linz@li-pro.net>
Wed, 22 Feb 2012 18:12:43 +0000 (19:12 +0100)
committerMichal Simek <monstr@monstr.eu>
Thu, 23 Feb 2012 11:16:04 +0000 (12:16 +0100)
The interrupt acknowledge action have to run after the
registered interrupt handler. So we have a chance to
bear out the corresponding interrupt request in the
corresponding controller hardware.

With this reordering, we optain a proper interrupt
handling for level triggered interrupt sources -- for
example the new axi_timer v1.02.a introduced in ISE 13.2.

Signed-off-by: Stephan Linz <linz@li-pro.net>
Acked-by: Michal Simek <monstr@monstr.eu>
arch/microblaze/cpu/interrupts.c

index 5a1321128a40e99d0f51306f153f71b295439863..e7ca859bfe446be14489239250740f0c40fe2133 100644 (file)
@@ -155,8 +155,6 @@ void interrupt_handler (void)
 #endif
        struct irq_action *act = vecs + irqs;
 
-       intc->iar = mask << irqs;
-
 #ifdef DEBUG_INT
        printf
            ("Jumping to interrupt handler rutine addr %x,count %x,arg %x\n",
@@ -165,6 +163,8 @@ void interrupt_handler (void)
        act->handler (act->arg);
        act->count++;
 
+       intc->iar = mask << irqs;
+
 #ifdef DEBUG_INT
        printf ("Dump INTC reg, isr %x, ier %x, iar %x, mer %x\n", intc->isr,
                intc->ier, intc->iar, intc->mer);