Date: Tue, 17 Feb 2009 10:26:05 +0000 (GMT) From: Robert Watson <rwatson@FreeBSD.org> To: Krassimir Slavchev <krassi@bulinfo.net> Cc: net@FreeBSD.org, Julian Elischer <julian@elischer.org>, current@FreeBSD.org Subject: Re: HEADS UP: IFF_NEEDSGIANT consumers to be disabled, removed Message-ID: <alpine.BSF.2.00.0902171023020.5806@fledge.watson.org> In-Reply-To: <499A884A.4040408@bulinfo.net> References: <20080526110543.J26343@fledge.watson.org> <alpine.BSF.2.00.0902161233440.5806@fledge.watson.org> <4999F7F9.4030204@elischer.org> <499A884A.4040408@bulinfo.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 17 Feb 2009, Krassimir Slavchev wrote: >> if_sr and if_ar are really simple and could probably be converted >> "trivially".. especially if their netgraph code is used. >> >> however I wonder if anyone still has that hardware (they are drivers for >> two sync serial cards). > > I still have such Digi/Arnet SYNC/570i PCI card and I used to use it for a > long time with 4.x and if_ar driver without any problems. > > Thanks to John Hay for well written driver! I would be quite happy to see the remaining drivers be converted over -- when using IFF_NEEDSGIANT, they do potentially see a significant performance loss as a result of having to defer execution to Giant-holding contexts. However, the sooner the better, as stripping the Giant compat stuff will allow us to clean up the compat shims, in turn removing the need for deferred execution to avoid calling those shims in unfortunate contexts for multicast, and simplifies the forthcoming address list locking work. Robert N M Watson Computer Laboratory University of Cambridge
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.0902171023020.5806>