diff mbox

powerpc: set IRQF_NO_THREAD for xmon/cascade handlers

Message ID 874mgsoddp.fsf@linutronix.de (mailing list archive)
State Accepted
Headers show

Commit Message

John Ogness Nov. 11, 2015, 1:48 p.m. UTC 
The xmon and cascade irq handlers must not run as threads.
pmac_pic_lock is already a raw_spinlock, but the irq flag
IRQF_NO_THREAD needs to be set as well.

Signed-off-by: John Ogness <john.ogness@linutronix.de>
---
 arch/powerpc/platforms/powermac/pic.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Michael Ellerman Nov. 26, 2015, 12:15 p.m. UTC | #1 
On Wed, 2015-11-11 at 13:48:50 UTC, John Ogness wrote:
> The xmon and cascade irq handlers must not run as threads.
> pmac_pic_lock is already a raw_spinlock, but the irq flag
> IRQF_NO_THREAD needs to be set as well.
> 
> Signed-off-by: John Ogness <john.ogness@linutronix.de>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/57f889471c0fb55cbb0db98b

cheers
diff mbox

Patch

diff --git a/arch/powerpc/platforms/powermac/pic.c b/arch/powerpc/platforms/powermac/pic.c
index 6f4f8b0..9815463 100644
--- a/arch/powerpc/platforms/powermac/pic.c
+++ b/arch/powerpc/platforms/powermac/pic.c
@@ -258,13 +258,14 @@  static unsigned int pmac_pic_get_irq(void)
 #ifdef CONFIG_XMON
 static struct irqaction xmon_action = {
 	.handler	= xmon_irq,
-	.flags		= 0,
+	.flags		= IRQF_NO_THREAD,
 	.name		= "NMI - XMON"
 };
 #endif
 
 static struct irqaction gatwick_cascade_action = {
 	.handler	= gatwick_action,
+	.flags		= IRQF_NO_THREAD,
 	.name		= "cascade",
 };