Date: Fri, 4 Jul 2008 01:30:18 +0100 (BST) From: Robert Watson <rwatson@FreeBSD.org> To: arch@FreeBSD.org, net@FreeBSD.org Subject: Re: Remaining non-MPSAFE netisr handlers Message-ID: <20080704012901.U90881@fledge.watson.org> In-Reply-To: <20080526102345.G26343@fledge.watson.org> References: <20080526102345.G26343@fledge.watson.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 26 May 2008, Robert Watson wrote: > In the continuing campaign to eliminate the Giant lock from the dregs of the > network stack, I thought I'd send out a list of non-MPSAFE netisr handlers: > > Location Handler Removed with IFF_NEEDSGIANT > dev/usb/usb_ethersubr.c:120 usbintr Yes > net/if_ppp.c:277 pppintr Yes > netinet6/ip6_input.c ip6_input No > > The plan for 8.0 is to remove the NETISR_MPSAFE flag -- all netisr handlers > will be executed without the Giant lock. This doesn't prohibit acquiring > Giant in the handler if required, although that's undesirable for the > obvious reasons (potentially stalling interrupt handling, etc). Obviously, > what would be most desirable is eliminating the remaining requirement for > Giant in the IPv6 input path, primarily consisting of mld6 and nd6. > > With this in mind, my current plan is to remove the flag and add explicit > Giant acquisition for any remaining handlers in June when IFF_NEEDSGIANT > device drivers are disabled. I've now removed the NETISR_MPSAFE flag -- all netisr handlers are now assumed to DTRT with respect to locking. At least until usb and ppp are sorted out, I've introduced NETISR_FORCEQUEUE as an interim measure, which allows protocols to request that they always operate the deferred dispatch, meaning they can acquire Giant if they need to, and modified those two to do so. That should go away by 8.0 also. 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?20080704012901.U90881>