diff options
author | David S. Miller <davem@davemloft.net> | 2008-03-27 17:42:50 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-03-27 17:42:50 -0700 |
commit | 50fd4407b8bfbde7c1a0bfe4f24de7df37164342 (patch) | |
tree | 57a9f05b130903fb2c6c12412352928bb24dc3a7 /mm | |
parent | 8eeee8b152ae6bbe181518efaf62ba8e9c613693 (diff) | |
download | op-kernel-dev-50fd4407b8bfbde7c1a0bfe4f24de7df37164342.zip op-kernel-dev-50fd4407b8bfbde7c1a0bfe4f24de7df37164342.tar.gz |
[NET]: Use local_irq_{save,restore}() in napi_complete().
Based upon a lockdep report.
Since ->poll() can be invoked from netpoll with interrupts
disabled, we must not unconditionally enable interrupts
in napi_complete().
Instead we must use local_irq_{save,restore}().
Noticed by Peter Zijlstra:
<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*
<irq>
irq_exit()
do_softirq()
net_rx_action()
spin_lock(&napi->poll_lock) <--- Deadlock!
Because we still hold the lock....
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions