BUG: 9.0 stage 2 boot (/boot/boot)

[John Baldwin]

On Friday, March 02, 2012 2:58:16 pm rank1seeker at gmail.com wrote:
> ----- Original Message -----
> From: John Baldwin <jhb at freebsd.org>
> To: rank1seeker at gmail.com
> Cc: hackers at freebsd.org, "Roman Divacky" <rdivacky at freebsd.org>, "Andriy Gapon" <avg at freebsd.org>
> Date: Fri, 2 Mar 2012 14:46:59 -0500
> Subject: Re: BUG: 9.0 stage 2 boot (/boot/boot)
> 
> > On Friday, March 02, 2012 1:46:37 pm rank1seeker at gmail.com wrote:
> > > ----- Original Message -----
> > > From: John Baldwin <jhb at freebsd.org>
> > > To: rank1seeker at gmail.com
> > > Cc: hackers at freebsd.org, "Roman Divacky" <rdivacky at freebsd.org>, "Andriy Gapon" <avg at freebsd.org>
> > > Date: Fri, 2 Mar 2012 12:24:24 -0500
> > > Subject: Re: BUG: 9.0 stage 2 boot (/boot/boot)
> > > 
> > > > On Friday, March 02, 2012 10:17:03 am rank1seeker at gmail.com wrote:
> > > > > ----- Original Message -----
> > > > > From: John Baldwin <jhb at freebsd.org>
> > > > > To: rank1seeker at gmail.com
> > > > > Cc: hackers at freebsd.org, "Roman Divacky" <rdivacky at freebsd.org>, Andriy Gapon <avg at freebsd.org>
> > > > > Date: Thu, 1 Mar 2012 16:14:43 -0500
> > > > > Subject: Re: BUG: 9.0 stage 2 boot (/boot/boot)
> > > > > 
> > > > > 
> > > > > > > However, avg@ might have found the actual cause of the bug.  And Roman did
> > > > > > > indeed break this earlier.  Try the updated boot2_test.patch again.
> > > > > > 
> > > > > > You should still try this.
> > > > > > 
> > > > > > -- 
> > > > > > John Baldwin
> > > > > > 
> > > > > 
> > > > > 
> > > > > Using patch:  MD5 (boot2_test.patch) = 72412f9b47b114302268561f12c1b46b
> > > > > 
> > > > > http://www.starforce.biz/stage2boot_1.jpg
> > > > > 
> > > > > This is the worst patch as even RE-typing doesn't work.
> > > > > That is, it is completely unable to boot.
> > > > > 
> > > > > Throw me another patch!
> > > > 
> > > > Ah, was using sizeof() on the wrong thing, and I have a fix for the garbage
> > > > you saw for kname as well.  Patch is updated at the normal URL.
> > > > 
> > > > > > Please don't treat John this way.  He has put in a lot of time for
> > > > > > you, and many others, and is one of THE experts.
> > > > > > 
> > > > > > Chris
> > > > > 
> > > > > 
> > > > > He didn't got a bad treatment, nor any side is angry.
> > > > > Relax! ;)
> > > > 
> > > > Actually, you've been rather rude for the entire thread, I am just doing my
> > > > best to ignore it in the interest of getting the bug fixed.
> > > > 
> > > > -- 
> > > > John Baldwin
> > > > 
> > > 
> > > I'm sorry if you think so, especially part where I am rude through the entire thread. That I can hardly believe.
> > > I am also actively testing and am persistant in solving issue.
> > > Or is problem in my perception?
> > 
> > I think it may just be that your language comes off as being a bit demanding,
> > plus a fair bit of all caps, etc.  If English is not your native language
> > then that may also be a factor.  If you are not intending to be rude then that
> > is plenty good enough for me. :)
> > 
> > > Anyway, yours latest patch has a same md5 hash as a previous one and also built binary /boot/boot had same md5 hash.
> > > So I've simply skipped installing boot code.
> > 
> > Yes, my bad, I updated it locally but forgot to upload it to the URL.
> > It should really be updated now.
> > 
> > -- 
> > John Baldwin
> > 
> 
> 
> Using MD5 (boot2_test.patch) = 05c46840163f789d362b46bf51b6c79f
> 
> I can't build it.
> 
> # cd /sys/boot
> # make clean && make obj && make depend && make && make install && make clean
> 
> ------
> ===> i386/boot2 (all)
> objcopy -S -O binary boot1.out boot1
> dd if=/dev/zero of=boot2.ldr bs=512 count=1
> 1+0 records in
> 1+0 records out
> 512 bytes transferred in 0.000116 secs (4409617 bytes/sec)
> cc -Os  -fno-guess-branch-probability  -fomit-frame-pointer  -fno-unit-at-a-time  -mno-align-long-strings  -mrtd  -mregparm=3  -DUSE_XREAD  -
DUFS1_AND_UFS2  -DFLAGS=0x80  -DSIOPRT=0x3f8  -DSIOFMT=0x3  -DSIOSPD=9600  -I/usr/src/sys/boot/i386/boot2/../../common  -
I/usr/src/sys/boot/i386/boot2/../btx/lib -I.  -Wall -Waggregate-return -Wbad-function-cast -Wcast-align  -Wmissing-declarations -Wmissing-
prototypes -Wnested-externs  -Wpointer-arith -Wshadow -Wstrict-prototypes -Wwrite-strings  -Winline --param max-inline-insns-single=100 -
ffreestanding -mpreferred-stack-boundary=2  -mno-mmx -mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -msoft-float -std=gnu99   -S -o boot2.s.tmp 
/usr/src/sys/boot/i386/boot2/boot2.c
> /usr/src/sys/boot/i386/boot2/boot2.c: In function 'main':
> /usr/src/sys/boot/i386/boot2/boot2.c:282: error: expected ':' before ')' token
> /usr/src/sys/boot/i386/boot2/boot2.c: In function 'parse':
> /usr/src/sys/boot/i386/boot2/boot2.c:468: warning: passing argument 1 of 'memcpy' discards qualifiers from pointer target type

Oops, should be fixed now.

-- 
John Baldwin