* alpha: Replace bogus in_interrupt()
@ 2020-10-14 15:47 Thomas Gleixner
0 siblings, 0 replies; only message in thread
From: Thomas Gleixner @ 2020-10-14 15:47 UTC (permalink / raw)
To: linux-alpha
Cc: Richard Henderson, Matt Turner, Ivan Kokshaysky,
Sebastian Andrzej Siewior
in_interrupt() is true for a variety of things including bottomhalf
disabled regions. Deducing hard interrupt context from it is dubious at
best.
Use in_irq() which is true if called in hard interrupt context. Otherwise
calling irq_exit() would do more harm than good.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---
arch/alpha/kernel/process.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/alpha/kernel/process.c b/arch/alpha/kernel/process.c
index 7462a7911002..1589467e7528 100644
--- a/arch/alpha/kernel/process.c
+++ b/arch/alpha/kernel/process.c
@@ -134,7 +134,7 @@ common_shutdown_1(void *generic_ptr)
#ifdef CONFIG_DUMMY_CONSOLE
/* If we've gotten here after SysRq-b, leave interrupt
context before taking over the console. */
- if (in_interrupt())
+ if (in_irq())
irq_exit();
/* This has the effect of resetting the VGA video origin. */
console_lock();
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2020-10-14 15:47 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-14 15:47 alpha: Replace bogus in_interrupt() Thomas Gleixner
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.