Subject: Re: disklabel(8) and machdep on-disk structures issues
To: None <tech-kern@NetBSD.org>
From: Manuel Bouyer <email@example.com>
Date: 11/15/2003 18:54:32
On Thu, Nov 06, 2003 at 11:38:57PM +0100, Manuel Bouyer wrote:
> [ please reply to tech-kern, unless your anserw is really port-specific ]
> On Fri, Oct 31, 2003 at 05:07:04PM +0100, Manuel Bouyer wrote:
> > So I propose:
> > - fix next68k disksubr.c to always write a next-compatible disklabel
> > in addition to the NetBSD disklabel (need to find some space for
> > the NetBSD disklabel first, shouldn't be too hard).
> > - Fix the sun3 LABELOFFSET, and factor out disksubr.c between sun3, sparc and
> > sparc64 (with some compat code to look for a NetBSD disklabel at
> > LABELOFFSET=64 in the sun3 case)
> > - remove #ifdef __sparc__ hack from disklabel(8), and change it to issue
> > a DIOCWDINFO after writing the disklabel to the raw partition in the
> > -r/-I case (so that the kernel can convert the label if needed)
> Attached is a patch implementing this. Tested on sparc, sun3 and next68k.
> We now have a sys/dev/sun/disksubr.c, based on sparc/disksubr.c. The
> sparc/disksubr.c and sparc64/disksubr.c differed only by some includes
> (BTW, both #included too much things). sun3/disksubr.c has some more
> differences, especially the groveling code to find a NetBSD disklabel
> in the first sector. I integrated this in sys/dev/sun/disksubr.c, as fallback
> if no NetBSD or Sun label have been found at the expected place (this will
> ensure that disk writtens with disklabel -r using the wrong Sun3 labeloffset
> will still be readable).
> next68k still doesn't write a NetBSD disklanel; I didn't find a proper place
> for it on disk yet (LABELSECTOR=15 should put it at the end of the NeXT v3
> label, but the machine doens't boot with it ...).
> Comments ?
> I'd like to commit this in about a week, as it fixes 2 bugs:
> - disklabel -r sd0 reports "no disk label" on sun3 because of the wrong
> - next68k can't be installed on a SCSI disk, because in order to write
> a label to a virgin disk you have to use disklanel -I or -r, which will
> install a NetBSD disklabel and prevent the kernel from writing a NeXT
FYI, I just commited this patch.
Manuel Bouyer <firstname.lastname@example.org>
NetBSD: 24 ans d'experience feront toujours la difference