On 23.05.2011 21:38, Warner Losh wrote: > I'm all for carefully moving forward. There were parts of the patch > that seemed to be over-reaching. > > Personally, I'd just do something like the following since it > reflects what others do and implements the de-facto meaning of the > MBR: > > Index: sys/geom/part/g_part_mbr.c > =================================================================== > --- sys/geom/part/g_part_mbr.c (revision 222183) +++ > sys/geom/part/g_part_mbr.c (working copy) @@ -473,7 +473,7 @@ > > basetable->gpt_entries = NDOSPART; basetable->gpt_first = > basetable->gpt_sectors; - basetable->gpt_last = msize - (msize % > basetable->gpt_sectors) - 1; + basetable->gpt_last = msize - 1;
There was report with different problem, the user created a partition that is aligned to 4KB sector size and it starts from the inside of first track, i.e. start offset < 63. And as he said, he did use sysinstall(8) for partitioning. -- WBR, Andrey V. Elsukov
signature.asc
Description: OpenPGP digital signature
