On Thu, Nov 29, 2012 at 09:10:58PM +0100, Mike Belopuhov wrote: > On 29 November 2012 20:49, Claudio Jeker <cje...@diehard.n-r-g.com> wrote: > > On Thu, Nov 29, 2012 at 11:50:09AM -0500, Kenneth R Westerback wrote: > >> On Thu, Nov 29, 2012 at 04:41:09PM +0100, Mike Belopuhov wrote: > >> > hi, > >> > > >> > drivers ex age alc ale jme se vic vte xe upl and octeon/cmac > >> > make use of the if_iqdrops counter that is not shown by any of our > >> > tools (like netstat). looks like most of its usage comes from > >> > freebsd where they show it in the "netstat -di" output in a new > >> > column. do we want to do that or just convert them to if_ierrors > >> > since 90% of our drivers do only if_ierrors. there's also doesn't > >> > seem to be any rule when to use if_iqdrops (well, since in most > >> > drivers there's no input queueing -- check out upl(4) :) > >> > > >> > the diff below changes all the drivers in our tree to use > >> > if_ierrors instead of if_iqdrops. i've decided to leave > >> > octeon/cmac driver as is because if_iqdrops is used for > >> > debugging purposes there. > >> > > >> > ack? nack? meh? > >> > >> Seems like a good idea to me to not lose those errors. I have > >> no great desire for a new column in netstat, but no great > >> antagonism to one either. > >> > > > > I guess the idea is to show which interfaces are overloading the input > > queues. At least that is my interpretation of if_iqdrops (input queue > > drops). In a away it makes sense but I'm not to attached to it. I would > > prefer we would actually print out the output queue drops on the interface > > queue. > > > > do you mean IFQ_ENQUEUE(&ifp->if_snd) drops? > they're exported via sysctl_ifq and it seems > like netstat has the printing code commented > out. > Yes. In that case we should make them visible.
-- :wq Claudio