diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2009-03-02 10:13:32 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-03-02 18:05:45 -0500 |
commit | 044d408409cc4e1bc75c886e27ca85c270db104c (patch) | |
tree | c2286707e9d4983a4828b0a4133181031de36964 /kernel/irq/handle.c | |
parent | c02368a9d059322f913a58111eade87a656fefd5 (diff) |
genirq: assert that irq handlers are indeed running in hardirq context
Make sure the genirq layer handlers are indeed running handlers
in hardirq context. That is the genirq expectation and doing
anything else is broken.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Andrew Morton <akpm@linux-foundation.org>
LKML-Reference: <1236006812.5330.632.camel@laptop>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/irq/handle.c')
-rw-r--r-- | kernel/irq/handle.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c index 3aba8d12f328..a2ee682bca2e 100644 --- a/kernel/irq/handle.c +++ b/kernel/irq/handle.c | |||
@@ -328,6 +328,8 @@ irqreturn_t handle_IRQ_event(unsigned int irq, struct irqaction *action) | |||
328 | irqreturn_t ret, retval = IRQ_NONE; | 328 | irqreturn_t ret, retval = IRQ_NONE; |
329 | unsigned int status = 0; | 329 | unsigned int status = 0; |
330 | 330 | ||
331 | WARN_ONCE(!in_irq(), "BUG: IRQ handler called from non-hardirq context!"); | ||
332 | |||
331 | if (!(action->flags & IRQF_DISABLED)) | 333 | if (!(action->flags & IRQF_DISABLED)) |
332 | local_irq_enable_in_hardirq(); | 334 | local_irq_enable_in_hardirq(); |
333 | 335 | ||