On Mon, Dec 26, 2011 at 07:25:29PM -0800, YongHyeon PYUN wrote: Y> Index: sys/net/if_vlan.c Y> =================================================================== Y> --- sys/net/if_vlan.c (revision 228906) Y> +++ sys/net/if_vlan.c (working copy) Y> @@ -1012,10 +1012,12 @@ Y> { Y> struct ifvlan *ifv; Y> struct ifnet *p; Y> - int error; Y> + int error, len, mcast; Y> Y> ifv = ifp->if_softc; Y> p = PARENT(ifv); Y> + len = m->m_pkthdr.len; Y> + mcast = (m->m_flags & (M_MCAST | M_BCAST)) ? 1 : 0; Y> Y> BPF_MTAP(ifp, m); Y> Y> @@ -1025,7 +1027,7 @@ Y> */ Y> if (!UP_AND_RUNNING(p)) { Y> m_freem(m); Y> - ifp->if_collisions++; Y> + ifp->if_oerrors++; Y> return (0); Y> } Y> Y> @@ -1081,9 +1083,11 @@ Y> * Send it, precisely as ether_output() would have. Y> */ Y> error = (p->if_transmit)(p, m); Y> - if (!error) Y> + if (!error) { Y> ifp->if_opackets++; Y> - else Y> + ifp->if_omcasts += mcast; Y> + ifp->if_obytes += len; Y> + } else Y> ifp->if_oerrors++; Y> return (error); Y> }
Thanks, Pyun! You can count me in as reviewer. I have also added jhb@ to Cc, the author of changed code, so that he can review, too. -- Totus tuus, Glebius. _______________________________________________ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"