diff options
author | Figo.zhang <zhangtianfei@leadcoretech.com> | 2010-06-07 21:13:22 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-06-10 23:14:08 -0700 |
commit | 349124a00754129a5f1e43efa84733e364bf3749 (patch) | |
tree | acff33465eb6b21aadca78657a7fa330d6987936 /drivers/net/8139cp.c | |
parent | 07a0f0f07a68014c92c752a5598102372bddf46e (diff) | |
download | blackbird-op-linux-349124a00754129a5f1e43efa84733e364bf3749.tar.gz blackbird-op-linux-349124a00754129a5f1e43efa84733e364bf3749.zip |
net8139: fix a race at the end of NAPI
fix a race at the end of NAPI complete processing, it had
better do __napi_complete() first before re-enable interrupt.
Signed-off-by:Figo.zhang <figo1802@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/8139cp.c')
-rw-r--r-- | drivers/net/8139cp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/8139cp.c b/drivers/net/8139cp.c index 9c149750e2bf..284a5f4a63ac 100644 --- a/drivers/net/8139cp.c +++ b/drivers/net/8139cp.c @@ -598,8 +598,8 @@ rx_next: goto rx_status_loop; spin_lock_irqsave(&cp->lock, flags); - cpw16_f(IntrMask, cp_intr_mask); __napi_complete(napi); + cpw16_f(IntrMask, cp_intr_mask); spin_unlock_irqrestore(&cp->lock, flags); } |