diff options
author | Wang Chen <wangchen@cn.fujitsu.com> | 2008-05-20 05:13:52 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-05-22 14:02:46 -0400 |
commit | f7f312a0c7e7a1947cf193e0e94a257ad7742cb2 (patch) | |
tree | c142f82b0177595e82aab7a802e2495afed4fcdf | |
parent | 56cfe5d028687468f76e8b613c63ca41f209982d (diff) |
3C509: rx_bytes should not be increased when alloc_skb failed
If alloc_skb failed, the recieved packet will be dropped. Do not increase
rx_bytes for dropped packet.
Signed-off-by: Wang Chen <wangchen@cn.fujitsu.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
-rw-r--r-- | drivers/net/3c509.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/3c509.c b/drivers/net/3c509.c index e6c545fe5f58..fe6d84105e55 100644 --- a/drivers/net/3c509.c +++ b/drivers/net/3c509.c | |||
@@ -1063,7 +1063,6 @@ el3_rx(struct net_device *dev) | |||
1063 | struct sk_buff *skb; | 1063 | struct sk_buff *skb; |
1064 | 1064 | ||
1065 | skb = dev_alloc_skb(pkt_len+5); | 1065 | skb = dev_alloc_skb(pkt_len+5); |
1066 | dev->stats.rx_bytes += pkt_len; | ||
1067 | if (el3_debug > 4) | 1066 | if (el3_debug > 4) |
1068 | printk("Receiving packet size %d status %4.4x.\n", | 1067 | printk("Receiving packet size %d status %4.4x.\n", |
1069 | pkt_len, rx_status); | 1068 | pkt_len, rx_status); |
@@ -1078,6 +1077,7 @@ el3_rx(struct net_device *dev) | |||
1078 | skb->protocol = eth_type_trans(skb,dev); | 1077 | skb->protocol = eth_type_trans(skb,dev); |
1079 | netif_rx(skb); | 1078 | netif_rx(skb); |
1080 | dev->last_rx = jiffies; | 1079 | dev->last_rx = jiffies; |
1080 | dev->stats.rx_bytes += pkt_len; | ||
1081 | dev->stats.rx_packets++; | 1081 | dev->stats.rx_packets++; |
1082 | continue; | 1082 | continue; |
1083 | } | 1083 | } |