diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-06-19 12:43:33 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-06-19 23:28:02 (GMT) |
commit | 658924dc9ae2ca8e3c46f36306f5dbd501cf4688 (patch) | |
tree | a10de7dfe5bd02f963455559868d4574cb08724c | |
parent | 00ce2d5614a67521241d024c6e50afe3bd3df049 (diff) | |
download | linux-658924dc9ae2ca8e3c46f36306f5dbd501cf4688.tar.xz |
hp100: fix an skb->len race
As soon as skb is given to hardware and spinlock released, TX completion
can free skb under us. Therefore, we should update netdev stats before
spinlock release.
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/hp100.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/hp100.c b/drivers/net/hp100.c index 8e10d2f..c3ecb11 100644 --- a/drivers/net/hp100.c +++ b/drivers/net/hp100.c @@ -1580,12 +1580,12 @@ static netdev_tx_t hp100_start_xmit_bm(struct sk_buff *skb, hp100_outl(ringptr->pdl_paddr, TX_PDA_L); /* Low Prio. Queue */ lp->txrcommit++; - spin_unlock_irqrestore(&lp->lock, flags); - /* Update statistics */ dev->stats.tx_packets++; dev->stats.tx_bytes += skb->len; + spin_unlock_irqrestore(&lp->lock, flags); + return NETDEV_TX_OK; drop: |