Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 15 Dec 2013 12:46:03 -0800
From:      Adrian Chadd <adrian@freebsd.org>
To:        Ryan Stone <rysto32@gmail.com>
Cc:        freebsd-net <freebsd-net@freebsd.org>
Subject:   Re: buf_ring in HEAD is racy
Message-ID:  <CAJ-VmonCnML_Fs-Lp2gAHRNUarSjU3VCXgU13BV2noFybEVo2A@mail.gmail.com>
In-Reply-To: <CAFMmRNz9P9ZnCRFEEdAgPxzaCUXFts=5Rs64zi3x=BDJa_7nWQ@mail.gmail.com>
References:  <CAFMmRNyJpvZ0AewWr62w16=qKer%2BFNXUJJy0Qc=EBqMnUV3OyQ@mail.gmail.com> <CAJ-VmonJG-M_f_m36f-z3ArumBKNdt5%2B6muwFJzWABRxRQWJaw@mail.gmail.com> <CAFMmRNz9P9ZnCRFEEdAgPxzaCUXFts=5Rs64zi3x=BDJa_7nWQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 14 December 2013 22:40, Ryan Stone <rysto32@gmail.com> wrote:
> On Sat, Dec 14, 2013 at 7:06 PM, Adrian Chadd <adrian@freebsd.org> wrote:
>> oh cool, you just did the output-drops thing I was about to code up.
>> We're missing those counters at work and the ops guys poked me about
>> it.
>>
>> I'll also give that a whirl locally and see about working with jack to
>> get it into -HEAD / MFC'ed to 10.
>>
>> Thanks!
>
> The reason why I call it hacky is because these drops are properly
> categorized as discards, not errors.  But there's not currently a
> really good way to distinguish between the two in FreeBSD right now
> (there is the drops counter in the if_snd queue but netstat seemed to
> be ignoring that even with -d specified and blindly overwriting it
> would seem to be incompatible with altq anyway)

Hm. A few drivers (eg cxgbe) update the ifsnd drops counter.

oh, holy crap! How'd I never notice that the drops column is empty?

How about I just go ahead and add that field to the ifnet statistics
and then teach netstat how to display them? What do you think?

Then we can "fix" the other drivers such as cxgbe (for reasons
surrounding "we use this at work" :) so they can properly use this?



-adrian



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmonCnML_Fs-Lp2gAHRNUarSjU3VCXgU13BV2noFybEVo2A>