Date: Tue, 04 Dec 2012 21:02:16 +0100 From: Andre Oppermann <oppermann@networx.ch> To: Adrian Chadd <adrian@freebsd.org> Cc: Barney Cordoba <barney_cordoba@yahoo.com>, John Baldwin <jhb@freebsd.org>, freebsd-net@freebsd.org Subject: Re: Latency issues with buf_ring Message-ID: <50BE56C8.1030804@networx.ch> In-Reply-To: <CAJ-Vmo=tFFkeK2uADMPuBrgX6wN_9TSjAgs0WKPCrEfyhkG6Pw@mail.gmail.com> References: <1353259441.19423.YahooMailClassic@web121605.mail.ne1.yahoo.com> <201212041108.17645.jhb@freebsd.org> <CAJ-Vmo=tFFkeK2uADMPuBrgX6wN_9TSjAgs0WKPCrEfyhkG6Pw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 04.12.2012 20:34, Adrian Chadd wrote: > .. and it's important to note that buf_ring itself doesn't have the > race condition; it's the general driver implementation that's racy. > > I have the same races in ath(4) with the watchdog programming. Exactly > the same issue. Our IF_* stack/driver boundary handoff isn't up to the task anymore. Also the interactions are either poorly defined or understood in many places. I've had a few chats with yongari@ and am experimenting with a modernized interface in my branch. The reason I stumbled across it was because I'm extending the hardware offload feature set and found out that the stack and the drivers (and the drivers among themself) are not really in sync with regards to behavior. For most if not all ethernet drivers from 100Mbit/s the TX DMA rings are so large that buffering at the IFQ level doesn't make sense anymore and only adds latency. So it could simply directly put everything into the TX DMA and not even try to soft-queue. If the TX DMA ring is full ENOBUFS is returned instead of filling yet another queue. However there are ALTQ interactions and other mechanisms which have to be considered too making it a bit more involved. I'm coming up with a draft and some benchmark results for an updated stack/driver boundary in the next weeks before xmas. -- Andre
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?50BE56C8.1030804>