Date: Mon, 28 Aug 1995 10:47:08 -0700 (PDT) From: Morgan Davis <root@io.cts.com> To: jkh@time.cdrom.com (Jordan K. Hubbard) Cc: bde@zeta.org.au, current@freebsd.org, vak@cronyx.ru Subject: Re: wd0 detect fails Message-ID: <199508281747.KAA01743@io.cts.com> In-Reply-To: <3969.809616183@time.cdrom.com> from "Jordan K. Hubbard" at Aug 28, 95 06:23:03 am
next in thread | previous in thread | raw e-mail | index | archive | help
Jordan K. Hubbard writes: > > > Cdrom support was added. The following change looks wrong (but works > > here): > > > > diff -c -2 -r1.81 wd.c > > *** 1.81 1995/05/16 07:52:04 > > --- wd.c 1995/08/19 19:40:54 > > [change elided] > > So, did this work? Should we bring it into -current? It was just > this type of potentially fatal WDC interaction that the code was > committed to test! No. Like you, I was a little confused by this, too. Bruce's diff was not a fix. It was a comparison between the older wd.c and the current one that has the changes in it for IDE CD-ROM support. I did a little debugging by putting some printfs in the code and rebooting my system to see what kind of status result my caching IDE controller returned, and it had an extra, unexpected bit turned on. The official fix will likely involve masking out that bit when checking for the desired status result. Bruce can tell you more. In any case, at least we know what the trouble is, and I am running everything -current except for my patch that gets around the unexpected status result. --Morgan
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199508281747.KAA01743>