netstat -i Ierrs column, Is it total, or per second?
John Baldwin
jhb at freebsd.org
Mon Aug 31 21:49:47 UTC 2009
On Monday 31 August 2009 3:15:53 pm Andrew Brampton wrote:
> 2009/8/31 John Baldwin <jhb at freebsd.org>:
> > It should be total and it sounds like a bug in the device driver. It
looks
> > like ixgbe_update_stats_counters() overwrites the accumulated value of
> > if_ierrors:
> >
> > /* Rx Errors */
> > ifp->if_ierrors = total_missed_rx + adapter->stats.crcerrs +
> > adapter->stats.rlec;
> >
> > It also increments if_ierrors in ixgbe_rxeof(). The driver should only do
one
> > or the other, but probably not both.
> >
> > --
> > John Baldwin
> >
>
> Thanks for your reply. I had wondered that, but looking at
> e1000/if_em.c it does a similar thing. However, a quick look at
> non-intel drivers and it seems others don't. So perhaps this is a
> problem across the intel drivers?
>
> So anyway I spent my afternoon reading the ixgbe spec sheet and
> creating the attached patch, which hopefully fixes this problem. I
> will forward this patch to freebsd <at> intel.com unless someone can
> point me toward the maintainers email address, or should I just create
> a PR?
I cc'd him on my earlier reply (jfv@) so he should have seen your patch
already.
--
John Baldwin
More information about the freebsd-hackers
mailing list