diff options
author | Stephen Hemminger <shemminger@linux-foundation.org> | 2008-01-07 03:23:04 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-09 02:30:01 -0500 |
commit | c6a1b62de9d043f274ec3ae2e207908c6d5feff3 (patch) | |
tree | 4546e2a0b4fcea21c30bca1e60f6b81af62c7d7c /drivers/net/tulip | |
parent | edba2a1fefc6296bc527754dee1c72a625bb675a (diff) |
[TULIP]: NAPI full quantum bug.
This should fix the kernel warn/oops reported while routing.
The tulip driver has a fencepost bug with new NAPI in 2.6.24
It has an off by one bug if a full quantum is reached.
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tulip')
-rw-r--r-- | drivers/net/tulip/interrupt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/tulip/interrupt.c b/drivers/net/tulip/interrupt.c index 365331446387..0461956d3b50 100644 --- a/drivers/net/tulip/interrupt.c +++ b/drivers/net/tulip/interrupt.c | |||
@@ -151,7 +151,8 @@ int tulip_poll(struct napi_struct *napi, int budget) | |||
151 | if (tulip_debug > 5) | 151 | if (tulip_debug > 5) |
152 | printk(KERN_DEBUG "%s: In tulip_rx(), entry %d %8.8x.\n", | 152 | printk(KERN_DEBUG "%s: In tulip_rx(), entry %d %8.8x.\n", |
153 | dev->name, entry, status); | 153 | dev->name, entry, status); |
154 | if (work_done++ >= budget) | 154 | |
155 | if (++work_done >= budget) | ||
155 | goto not_done; | 156 | goto not_done; |
156 | 157 | ||
157 | if ((status & 0x38008300) != 0x0300) { | 158 | if ((status & 0x38008300) != 0x0300) { |