Date: Thu, 6 Dec 2007 04:21:17 -0500 From: John Baldwin <jhb@freebsd.org> To: Marcel Moolenaar <marcel@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: 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 Message-ID: <200712060421.18367.jhb@freebsd.org> In-Reply-To: <200712060232.lB62WhFL011174@repoman.freebsd.org> References: <200712060232.lB62WhFL011174@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200712060421.18367.jhb>