In message: <200902011937.32679.hsela...@c2i.net> Hans Petter Selasky <hsela...@c2i.net> writes: : Hi Warner, : : On Sunday 01 February 2009, M. Warner Losh wrote: : > In message: <20090201175021.ga32...@citylink.fud.org.nz> : > : > Andrew Thompson <thom...@freebsd.org> writes: : : > The only way that a 'deferred attach' makes sense is : : > if the ifnet and other external resources are setup as part of : > that deferred attach. That way, you don't have the NULL pointer issue. : : That was what the initial code did. : : > : > However, doing that introduces races with devd, which are a pita to : > cope with... Even without deferring the setting up if ifnet, you have : > races with devd if you defer things in attach that can be hard to cope : > with in the code. : : No, not if the ifnet attach is deferred too. : : My conclusion is: Do not make match rules for "rumX/uralX/zydX", instead match : for the IFNET event in devd.conf. : : devctl_notify("IFNET", ifp->if_xname, "ATTACH", NULL);
Yes. We already do that. I was thinking of the geom/device race that we haven't closed rather than this race which we have. Warner _______________________________________________ freebsd-usb@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-usb To unsubscribe, send any mail to "freebsd-usb-unsubscr...@freebsd.org"