aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/irq
diff options
context:
space:
mode:
authorThomas Gleixner <tglx@linutronix.de>2011-02-25 11:17:18 -0500
committerThomas Gleixner <tglx@linutronix.de>2011-02-25 11:17:18 -0500
commit1204e95689f9fbd245a4ce5c1b0cd0a9b77f8d25 (patch)
tree7dafda97ffb9156853904873c1397ac2d2b5b9d7 /kernel/irq
parentfd4afaf33313d94f548cb09129ecba3dbab62931 (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>
Diffstat (limited to 'kernel/irq')
-rw-r--r--kernel/irq/handle.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c
index e099e9e9de0..b110c835e07 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) {