From owner-cvs-all Tue Dec 17 19:42: 9 2002 Delivered-To: cvs-all@freebsd.org Received: from green.bikeshed.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 3DBCE37B401; Tue, 17 Dec 2002 19:42:07 -0800 (PST) Received: from green.bikeshed.org (lhit9i93e8jrybrn@green.bikeshed.org [10.0.0.1] (may be forged)) by green.bikeshed.org (8.12.6/8.12.6) with ESMTP id gBI3g6im015212; Tue, 17 Dec 2002 22:42:06 -0500 (EST) (envelope-from green@green.bikeshed.org) Received: from localhost (green@localhost) by green.bikeshed.org (8.12.6/8.12.6/Submit) with ESMTP id gBI3g5H8015208; Tue, 17 Dec 2002 22:42:06 -0500 (EST) Message-Id: <200212180342.gBI3g5H8015208@green.bikeshed.org> X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Bruce Evans Cc: "M. Warner Losh" , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/boot/i386/boot2 boot2.c In-Reply-To: Your message of "Wed, 18 Dec 2002 14:02:43 +1100." <20021218130326.G23022-100000@gamplex.bde.org> From: "Brian F. Feldman" Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Tue, 17 Dec 2002 22:42:05 -0500 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Bruce Evans wrote: > On Tue, 17 Dec 2002, Brian F. Feldman wrote: > > > "M. Warner Losh" wrote: > > > In message: <200212172213.gBHMD7wd013565@green.bikeshed.org> > > > "Brian F. Feldman" writes: > > > : Warner Losh wrote: > > > : > imp 2002/12/17 14:00:06 PST > > > : > > > > : > Modified files: > > > : > sys/boot/i386/boot2 boot2.c > > > : > Log: > > > : > Reduce diffs with Peter's expanded diffs: > > > : > 1) Put back the keyboard printing printf, at the cost of 58 bytes. > > > : > 2) Minor tweak to getstr at no apparent cost. > > > : > > > : Hmm, after all this work on boot2, I remain confused as to why noone else > > > : working on the boot blocks seems to have any trouble reinstalling them > > > : to test. Am I really the only one that had to fix disklabel.c so that it > > > : would install boot blocks instead of just dying on a post-GEOM system? > > > : It seems strange that it's only a few people here that would be having that > > > : issue :) > > > > > > I had to hack disklabel.c in unnatural ways to install to test. > > > > Is this better? > > This seems to be mainly to work around missing support for DIOCWLABEL in > GEOM. I don't use GEOM, so I have no problems with disk labels. To put this in context, this is because with GEOM the disklabels now explicitly contain (in the new format) the offset in terms of the slice, rather than starting at 0. The disklabel -r can't possibly work because disklabel(8) won't read the correct (old-style, 0 offset) label off the device, which won't match the adjusted format that disklabel without -r gets from GEOM. So, when disklabel(8) thinks that the labels are full of errors because it thinks offsets are always supposed to start at 0 in the disklabel, it generates lots of warnings. Writing over the label on the disk won't work with GEOM, and just calling DIOCWDINFO won't work when you disklabel -r (also disklabel -B's implicit -r) because the offsets would be incorrect. -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org <> bfeldman@tislabs.com \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message