diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-03-04 00:05:56 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-03-04 00:11:52 -0800 |
commit | b9bdcd9bd78d253dcc8e13c29f0acd67e080e7c1 (patch) | |
tree | 6ee1908315df4e2347c16eb63fee6bd2762a749b /drivers/net/cxgb3/mc5.c | |
parent | 858b9ced6e73a0f087294c398a1ae70a7eeed94f (diff) |
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/cxgb3/mc5.c')
0 files changed, 0 insertions, 0 deletions