diff options
author | Atsushi Nemoto <anemo@mba.ocn.ne.jp> | 2010-02-24 01:00:17 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-26 05:08:34 -0500 |
commit | 9c5f9c2861909753140ad6a41cdc77cdf20c1dc9 (patch) | |
tree | 0244279540075f960b4afb6927b433db7dee36f7 /drivers | |
parent | fbc450b137e59205b98a172326415cd981c9caa6 (diff) |
isa-skelton: Remove a wrong netif_wake_queue() call
The netif_wake_queue() is called correctly (i.e. only on !txfull
condition) from net_tx(). So Unconditional call to the
netif_wake_queue() here is wrong. This might cause calling of
start_xmit routine on txfull state and trigger tx-ring overflow.
This fix is ported from commit 662a96bd6f020782dfbdc0d0bd177c7dbb556687
("tc35815: Remove a wrong netif_wake_queue() call which triggers BUG_ON").
Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/isa-skeleton.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/isa-skeleton.c b/drivers/net/isa-skeleton.c index d09e4d3a15a6..9e55c3007743 100644 --- a/drivers/net/isa-skeleton.c +++ b/drivers/net/isa-skeleton.c | |||
@@ -536,7 +536,6 @@ static irqreturn_t net_interrupt(int irq, void *dev_id) | |||
536 | /* Transmit complete. */ | 536 | /* Transmit complete. */ |
537 | net_tx(dev); | 537 | net_tx(dev); |
538 | np->stats.tx_packets++; | 538 | np->stats.tx_packets++; |
539 | netif_wake_queue(dev); | ||
540 | } | 539 | } |
541 | #endif | 540 | #endif |
542 | if (status & COUNTERS_INTR) { | 541 | if (status & COUNTERS_INTR) { |