Return to BSD News archive
Newsgroups: comp.unix.bsd
Path: sserve!manuel!munnari.oz.au!mips!mips!newsun!gateway.novell.com!terry
From: terry@npd.Novell.COM (Terry Lambert)
Subject: Re: 386bsd-0.1: primary bootstrap (wdbootblk.c) problem & fix.
Message-ID: <1992Jul27.192502.15727@gateway.novell.com>
Keywords: 386bsd boot bootstrap wdbootblk.c
Sender: news@gateway.novell.com (NetNews)
Nntp-Posting-Host: thisbe.eng.sandy.novell.com
Organization: Novell NPD -- Sandy, UT
References: <greg.712111605@hibp1.ecse.rpi.edu> <1992Jul27.172708.3363@gateway.novell.com> <greg.712261295@hibp1.ecse.rpi.edu>
Date: Mon, 27 Jul 1992 19:25:02 GMT
In article <greg.712261295@hibp1.ecse.rpi.edu> greg@ecse.rpi.edu (Greg) writes:
>Terry, please reread my message. THIS IS NOT THE SAME BUG!!!!!!
>This problem is in wdbootblk.c. The code contained in wd.c has not
>even been loaded off the disk at the point when this hang occurs!
>To understand what is going on, study wdbootblk.c.
I think I goofed in recommending the wd.c fix for this problem;
however, the status register being polled after the delay is wd_status,
and I still believe that a canonical fix would be looking at wd_altsts
instead.
Greg's delay loop will have the same effect -- that is, making sure
the controller has finished it's reset prior to using it.
The confusion seems to have originated from the various paths in
the boot and non-boot sections of the kernel which lead to files of the
same name. I would suggest that any future patches refrain from the use
of relative paths to identify the file being patched. In particular, it
would be nice if they contained the full path from the root of the volume
("/") rather than something like "./xxx/yyy/zzz.c", which assumes they are
starting in the same directory that patch poster did.
...This, of course, wouldn't have saved me from foot-in-mouth in
this particular case 8-(.
Terry Lambert
terry_lambert@gateway.novell.com
terry@icarus.weber.edu
---
Disclaimer: Any opinions in this posting are my own and not those of
my present or previous employers.