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

[Prarit Bhargava]

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
> > spin_lock_irqsave().
> >
> >     
>
> 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 ...

P.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/