diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2010-02-22 18:12:30 +0100 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2010-02-22 18:12:30 +0100 |
| commit | ee2bdb55130653718b1b4fdb40d58ddb55bbe78c (patch) | |
| tree | bb8af9fcdb244a828add484dfb76078f71a0db97 | |
| parent | ddb34cf8cb1cc24764b72b608cd6d148d116a9f3 (diff) | |
| download | lwn-ee2bdb55130653718b1b4fdb40d58ddb55bbe78c.tar.gz lwn-ee2bdb55130653718b1b4fdb40d58ddb55bbe78c.zip | |
genirq: Fix stale oneshot flag.
The oneshot flag might be set when the thread starts and cleared later
in handle_edge_irq() - I know it's a hack :)
Check the flags in the irq thread code instead of using a possibly
stale value.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
| -rw-r--r-- | kernel/irq/manage.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c index 1efad981d63d..7d108d072dc9 100644 --- a/kernel/irq/manage.c +++ b/kernel/irq/manage.c @@ -621,7 +621,7 @@ static int irq_thread(void *data) struct sched_param param = { .sched_priority = MAX_USER_RT_PRIO/2, }; struct irqaction *action = data; struct irq_desc *desc = irq_to_desc(action->irq); - int wake, oneshot = desc->status & IRQ_ONESHOT; + int wake; sched_setscheduler(current, SCHED_FIFO, ¶m); current->extra_flags |= PFE_HARDIRQ; @@ -649,7 +649,7 @@ static int irq_thread(void *data) action->thread_fn(action->irq, action->dev_id); - if (oneshot) + if (desc->status & IRQ_ONESHOT) irq_finalize_oneshot(action->irq, desc, action); } |