[src/trunk]: src/sys/dev/pci fix: ifp->if_obytes and ifp->if_omcasts are not ...

[knakahara <knakahara%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/3c640f852501
branches:  trunk
changeset: 819384:3c640f852501
user:      knakahara <knakahara%NetBSD.org@localhost>
date:      Fri Dec 02 01:48:44 2016 +0000

description:
fix: ifp->if_obytes and ifp->if_omcasts are not counted up for packets which is sent later.

pointed out by ozaki-r@n.o, thanks.

diffstat:

 sys/dev/pci/if_wm.c |  36 ++++++++++++++++++++++++------------
 1 files changed, 24 insertions(+), 12 deletions(-)

diffs (80 lines):

diff -r 1e7316c3247b -r 3c640f852501 sys/dev/pci/if_wm.c
--- a/sys/dev/pci/if_wm.c       Fri Dec 02 00:50:09 2016 +0000
+++ b/sys/dev/pci/if_wm.c       Fri Dec 02 01:48:44 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: if_wm.c,v 1.454 2016/12/01 02:36:50 knakahara Exp $    */
+/*     $NetBSD: if_wm.c,v 1.455 2016/12/02 01:48:44 knakahara Exp $    */
 
 /*
  * Copyright (c) 2001, 2002, 2003, 2004 Wasabi Systems, Inc.
@@ -84,7 +84,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_wm.c,v 1.454 2016/12/01 02:36:50 knakahara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_wm.c,v 1.455 2016/12/02 01:48:44 knakahara Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_net_mpsafe.h"
@@ -6334,6 +6334,10 @@
 
        KASSERT(ifp->if_extflags & IFEF_START_MPSAFE);
 
+       /*
+        * ifp->if_obytes and ifp->if_omcasts are added in if_transmit()@if.c.
+        */
+
        mutex_enter(txq->txq_lock);
        if (!txq->txq_stopping)
                wm_start_locked(ifp);
@@ -6365,12 +6369,14 @@
                return ENOBUFS;
        }
 
+       /*
+        * XXXX NOMPSAFE: ifp->if_data should be percpu.
+        */
+       ifp->if_obytes += m->m_pkthdr.len;
+       if (m->m_flags & M_MCAST)
+               ifp->if_omcasts++;
+
        if (mutex_tryenter(txq->txq_lock)) {
-               /* XXXX should be per TX queue */
-               ifp->if_obytes += m->m_pkthdr.len;
-               if (m->m_flags & M_MCAST)
-                       ifp->if_omcasts++;
-
                if (!txq->txq_stopping)
                        wm_transmit_locked(ifp, txq);
                mutex_exit(txq->txq_lock);
@@ -6902,6 +6908,10 @@
 
        KASSERT(ifp->if_extflags & IFEF_START_MPSAFE);
 
+       /*
+        * ifp->if_obytes and ifp->if_omcasts are added in if_transmit()@if.c.
+        */
+
        mutex_enter(txq->txq_lock);
        if (!txq->txq_stopping)
                wm_nq_start_locked(ifp);
@@ -6933,12 +6943,14 @@
                return ENOBUFS;
        }
 
+       /*
+        * XXXX NOMPSAFE: ifp->if_data should be percpu.
+        */
+       ifp->if_obytes += m->m_pkthdr.len;
+       if (m->m_flags & M_MCAST)
+               ifp->if_omcasts++;
+
        if (mutex_tryenter(txq->txq_lock)) {
-               /* XXXX should be per TX queue */
-               ifp->if_obytes += m->m_pkthdr.len;
-               if (m->m_flags & M_MCAST)
-                       ifp->if_omcasts++;
-
                if (!txq->txq_stopping)
                        wm_nq_transmit_locked(ifp, txq);
                mutex_exit(txq->txq_lock);