summaryrefslogtreecommitdiffstats
path: root/drivers/net/lasi_82596.c
diff options
context:
space:
mode:
authorRoel Kluin <roel.kluin@gmail.com>2009-03-04 09:05:56 +0100
committerDavid S. Miller <davem@davemloft.net>2009-03-04 09:11:52 +0100
commitb9bdcd9bd78d253dcc8e13c29f0acd67e080e7c1 (patch)
tree6ee1908315df4e2347c16eb63fee6bd2762a749b /drivers/net/lasi_82596.c
parentnet: more timeouts that reach -1 (diff)
downloadlinux-b9bdcd9bd78d253dcc8e13c29f0acd67e080e7c1.tar.xz
linux-b9bdcd9bd78d253dcc8e13c29f0acd67e080e7c1.zip
net pcmcia: worklimit reaches -1
with while (--worklimit >= 0); worklimit reaches -1 after the loop. In 3c589_cs.c this caused a warning not to be printed. In 3c574_cs.c contrastingly, el3_rx() treats worklimit differently: static int el3_rx(struct net_device *dev, int worklimit) { while (--worklimit >= 0) { ... } return worklimit; } el3_rx() is only called by function el3_interrupt(): twice: static irqreturn_t el3_interrupt(int irq, void *dev_id) { int work_budget = max_interrupt_work; while(...) { if (...) work_budget = el3_rx(dev, work_budget); if (...) work_budget = el3_rx(dev, work_budget); if (--work_budget < 0) { ... break; } } } The error path can occur 2 too early. Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/lasi_82596.c')
0 files changed, 0 insertions, 0 deletions