[lkml]   [2009]   [Nov]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH]: use spin_lock_irqsave in try_one_irq()

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 ...


 \ /
  Last update: 2009-11-03 16:09    [W:0.048 / U:6.844 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site