From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: [PATCH] Don't count packets we don't actually receive. Date: Tue, 14 Dec 2010 20:35:19 +0000 Message-ID: <1292358922-5569-2-git-send-email-paul.durrant@citrix.com> References: <1292358922-5569-1-git-send-email-paul.durrant@citrix.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <1292358922-5569-1-git-send-email-paul.durrant@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel@lists.xensource.com Cc: Paul Durrant List-Id: xen-devel@lists.xenproject.org Make sure we only bump rx_packets when we're definitely going to call netif_rx_ni(). Signed-off-by: Paul Durrant --- drivers/xen/netback/netback.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/xen/netback/netback.c b/drivers/xen/netback/netback.c index 1a4a20e..066d140 100644 --- a/drivers/xen/netback/netback.c +++ b/drivers/xen/netback/netback.c @@ -1519,9 +1519,6 @@ static void net_tx_submit(struct xen_netbk *netbk) skb->dev = netif->dev; skb->protocol = eth_type_trans(skb, skb->dev); - netif->stats.rx_bytes += skb->len; - netif->stats.rx_packets++; - if (skb->ip_summed == CHECKSUM_PARTIAL) { if (skb_checksum_setup(skb)) { DPRINTK("Can't setup checksum in net_tx_action\n"); @@ -1537,6 +1534,9 @@ static void net_tx_submit(struct xen_netbk *netbk) continue; } + netif->stats.rx_bytes += skb->len; + netif->stats.rx_packets++; + netif_rx_ni(skb); netif->dev->last_rx = jiffies; } -- 1.5.6.5