cvs commit: src/sys/conf NOTES files options src/sys/geom/part g_part.c g_part.h g_part_bsd.c src/sys/ia64/conf DEFAULTS

[John Baldwin]

On Wednesday 05 December 2007 09:32:42 pm Marcel Moolenaar wrote:
> marcel      2007-12-06 02:32:42 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/conf             NOTES files options 
>     sys/geom/part        g_part.c g_part.h 
>     sys/ia64/conf        DEFAULTS 
>   Added files:
>     sys/geom/part        g_part_bsd.c 
>   Log:
>   Add a BSD disklabel backend to g_part:
>   o  Disklabels can have between 8 and 20 partitions (inclusive).
>   o  No device special file is created for the raw partition.
>   o  Switch ia64 to use this backend.
>   o  No support for boot code yet.

So one issue with gpart is it doesn't handle attribute requests yet, so it 
doesn't have the feature where geom_bsd only attaches when a parent partition 
has the right type.  I ran into this for GPT testing where due to the way we 
align disks at work for RAID striping, the beignning of my GPT slice for / 
had a BSD label at the start, so I would end up with bogus /dev/ad0p1a 
devices and scary warnings in dmesg about a busted BSD label because offsets 
didn't match up.  I fixed this in 6.x by making GEOM_GPT export its UUID via 
an attr and having geom_bsd check that like it does now for MBR::type and 
PC98::type.  You will need to handle this in gpart as well.

Also, the inability to kldload GPT support is kind of lame now.  For my GPT
testing with QEMU on my laptop I had to grab GEOM_GPT out of the attic and
build its module.

-- 
John Baldwin