svn commit: r219573 - head/sbin/geom/class/part

[Ben Kaduk]

On Sat, Mar 12, 2011 at 12:05 PM, Andrey V. Elsukov <ae at freebsd.org> wrote:
> Author: ae
> Date: Sat Mar 12 17:05:18 2011
> New Revision: 219573
> URL: http://svn.freebsd.org/changeset/base/219573
>
> Log:
>  Document GEOM_PART_EBR_COMPAT option.
>
>  MFC after:    2 weeks
>
> Modified:
>  head/sbin/geom/class/part/gpart.8
>
> Modified: head/sbin/geom/class/part/gpart.8
> ==============================================================================
> --- head/sbin/geom/class/part/gpart.8   Sat Mar 12 17:03:41 2011        (r219572)
> +++ head/sbin/geom/class/part/gpart.8   Sat Mar 12 17:05:18 2011        (r219573)
> @@ -24,7 +24,7 @@
>  .\"
>  .\" $FreeBSD$
>  .\"
> -.Dd March 9, 2011
> +.Dd March 12, 2011
>  .Dt GPART 8
>  .Os
>  .Sh NAME
> @@ -38,6 +38,7 @@ lines in your kernel configuration file:
>  .Cd "options GEOM_PART_APM"
>  .Cd "options GEOM_PART_BSD"
>  .Cd "options GEOM_PART_EBR"
> +.Cd "options GEOM_PART_EBR_COMPAT"
>  .Cd "options GEOM_PART_GPT"
>  .Cd "options GEOM_PART_MBR"
>  .Cd "options GEOM_PART_PC98"
> @@ -58,6 +59,11 @@ The
>  option adds support for the Extended Boot Record (EBR),
>  which is used to define a logical partition.
>  The
> +.Dv GEOM_PART_EBR_COMPAT
> +option enables the backward compatibility for the partition names

I think just "enables backward compatibility" is more standard English
usage.  Not knowing much about the EBR scheme, I would also guess that
"for partition names in the EBR scheme" is what is intended here, but
that would be something of a guess.

> +of the EBR scheme. Also it makes impossible any types of actions
> +with such partitions.

Judging by, e.g. r216755 of g_part_ebr.c, it seems that the sense of
"impossible" is reversed.  I think "It also allows such partitions to
be modified" might be what is intended?

-Ben Kaduk

> +The
>  .Dv GEOM_PART_GPT
>  option adds support for the GUID Partition Table (GPT)
>  found on Intel Itanium computers and Intel-based Macintosh computers.