diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2011-02-25 11:17:18 -0500 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2011-02-25 11:17:18 -0500 |
commit | 1204e95689f9fbd245a4ce5c1b0cd0a9b77f8d25 (patch) | |
tree | 7dafda97ffb9156853904873c1397ac2d2b5b9d7 | |
parent | fd4afaf33313d94f548cb09129ecba3dbab62931 (diff) |
genirq: Make warning in handle_percpu_event useful
The WARN_ON_ONCE in handle_percpu_event() which emits a warning when
an action handler returns with interrupts enabled is not really
useful. It does not reveal the interrupt number and handler function
which caused it. Make it WARN_ONCE() and add the information.
Reported-by: Tony Luck <tony.luck@intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r-- | kernel/irq/handle.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c index e099e9e9de0b..b110c835e070 100644 --- a/kernel/irq/handle.c +++ b/kernel/irq/handle.c | |||
@@ -64,7 +64,8 @@ handle_irq_event_percpu(struct irq_desc *desc, struct irqaction *action) | |||
64 | res = action->handler(irq, action->dev_id); | 64 | res = action->handler(irq, action->dev_id); |
65 | trace_irq_handler_exit(irq, action, res); | 65 | trace_irq_handler_exit(irq, action, res); |
66 | 66 | ||
67 | if (WARN_ON_ONCE(!irqs_disabled())) | 67 | if (WARN_ONCE(!irqs_disabled(),"irq %u handler %pF enabled interrupts\n", |
68 | irq, action->handler)) | ||
68 | local_irq_disable(); | 69 | local_irq_disable(); |
69 | 70 | ||
70 | switch (res) { | 71 | switch (res) { |