Re: [PATCH]: use spin_lock_irqsave in try_one_irq()

From: Prarit Bhargava
Date: Tue Nov 03 2009 - 10:06:12 EST

Yong Zhang wrote:
This happens because the &desc->lock is taken with spin_lock_irqsave and
just a spin_lock. In the try_one_irq(), this lock really should be a

Cc'ed Ingo and Thomas.

The reason is that try_one_irq() is called both from hardirq context and softirq
context. And by default the timer handler poll_all_shared_irqs() is
called with irq enabled.
Then the two usage will cause inconsistent.

So I think the following patch is also workable to you.

Ah, okay. I will retest and get back to you ...

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at