Re: powerpc/64: replay hypervisor maintenance priority

2016-09-20 Thread Michael Ellerman
On Wed, 2016-14-09 at 03:01:21 UTC, Nicholas Piggin wrote:
> The hmi is defined 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 

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/e0e0d6b7390b61feb06350ef4a

cheers


[PATCH] powerpc/64: replay hypervisor maintenance priority

2016-09-13 Thread Nicholas Piggin
The hmi is defined 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 
---
 arch/powerpc/kernel/irq.c | 14 +-
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c
index 3cb46a3..ad1a930 100644
--- a/arch/powerpc/kernel/irq.c
+++ b/arch/powerpc/kernel/irq.c
@@ -155,6 +155,15 @@ notrace unsigned int __check_irq_replay(void)
}
 
/*
+* 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
 * in case we also had a rollover while hard disabled
@@ -189,11 +198,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);
 
-- 
2.9.3