powerpc/64: Replay hypervisor maintenance interrupt first
authorNicholas Piggin <npiggin@gmail.com>
Wed, 14 Sep 2016 03:01:21 +0000 (13:01 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 20 Sep 2016 04:35:34 +0000 (14:35 +1000)
The HMI (Hypervisor Maintenance Interrupt) is defined by the
architecture to be higher priority than other maskable interrupts, so
replay it first, as a best-effort to replay according to hardware
priorities.

Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/irq.c

index 9594ac8e7090e7862b66d9e85ca22a77a9159e03..471f65ad1b9b68a63ebfb0c3ee07ddfc923a96f4 100644 (file)
@@ -156,6 +156,15 @@ notrace unsigned int __check_irq_replay(void)
                lv1_get_version_info(&tmp, &tmp2);
        }
 
+       /*
+        * Check if an hypervisor Maintenance interrupt happened.
+        * This is a higher priority interrupt than the others, so
+        * replay it first.
+        */
+       local_paca->irq_happened &= ~PACA_IRQ_HMI;
+       if (happened & PACA_IRQ_HMI)
+               return 0xe60;
+
        /*
         * We may have missed a decrementer interrupt. We check the
         * decrementer itself rather than the paca irq_happened field
@@ -191,11 +200,6 @@ notrace unsigned int __check_irq_replay(void)
        }
 #endif /* CONFIG_PPC_BOOK3E */
 
-       /* Check if an hypervisor Maintenance interrupt happened */
-       local_paca->irq_happened &= ~PACA_IRQ_HMI;
-       if (happened & PACA_IRQ_HMI)
-               return 0xe60;
-
        /* There should be nothing left ! */
        BUG_ON(local_paca->irq_happened != 0);