Messages in this thread | | | Date | Thu, 27 Mar 2008 13:22:55 +0100 | From | Jarek Poplawski <> | Subject | Re: [Bugme-new] [Bug 10326] New: inconsistent lock state in net_rx_action |
| |
On Thu, Mar 27, 2008 at 11:56:19AM +0100, Peter Zijlstra wrote: > On Thu, 2008-03-27 at 02:18 -0700, Andrew Morton wrote: ... > > I bet the net code is wrong and we missed it ;)
It looks like you are natural born winner! Congratulations!
> How about this: > > <irqs disabled> > > netpoll_poll() > poll_napi() > spin_trylock(&napi->poll_lock) > poll_one_napi() > napi->poll() := sky2_poll() > napi_complete() > local_irq_disable() > local_irq_enable() <--- *BUG*
Yes! I missed it's unconditional here... Great catch!
On the other hand, still a question why lockdep doesn't see this every day?
Jarek P.
| |