Date: Sun, 27 Feb 2000 16:41:28 +0000 (GMT) From: Doug Rabson <dfr@nlsystems.com> To: Soren Schmidt <sos@freebsd.dk> Cc: Sren Schmidt <sos@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ata ata-all.c Message-ID: <Pine.BSF.4.21.0002271639440.8714-100000@salmon.nlsystems.com> In-Reply-To: <200002271613.RAA43235@freebsd.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 27 Feb 2000, Soren Schmidt wrote: > It seems Doug Rabson wrote: > > On Sun, 27 Feb 2000, Sren Schmidt wrote: > > > > > sos 2000/02/27 04:41:08 PST > > > > > > Modified files: > > > sys/dev/ata ata-all.c > > > Log: > > > Fix the problem that caused the boot to fail when modules were loaded. > > > > > > Real braino, confuses two different softc types, I wonder how this > > > could ever work :( > > > > Actually, it was confusing two devclasses and ended up with ata_devclass > > pointing at "atapci". This meant that lookups using ata_devclass returned > > devices like atapci0 instead of ata0 and things went downhill fast... > > That was what I meant but unclearly formulated. I still wonder how > this could work :| I think that we were unlucky in that when no modules were loaded, the various registrations happened a different order which left ata_devclass pointing at the right value. Loading modules can re-order the registration slightly as new SYSINITs are sorted into the list. -- Doug Rabson Mail: dfr@nlsystems.com Nonlinear Systems Ltd. Phone: +44 181 442 9037 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0002271639440.8714-100000>