| Message ID | bc2a4277-cbe2-c3e7-ad92-e2a67f9102cf@sandeen.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Jun 14, 2018 at 10:43:37AM -0500, Eric Sandeen wrote: > A bit more clarification in the CONFIGURATION FILE FORMAT > section of the mkfs.xfs manpage. > > Primarily, map the commandline switches to the section names > explicitly. > > "optional default" sounds pretty weird, so call it the > "defaults" configuration file instead. > > Document the comment format by example. > > Fix the sparse inode default setting in the example. > > Change the .BI tags to .B (there is no bold/italic switching). > > Signed-off-by: Eric Sandeen <sandeen@redhat.com> > --- > > diff --git a/man/man8/mkfs.xfs.8.in b/man/man8/mkfs.xfs.8.in > index cf4bdf8..9c0c1a8 100644 > --- a/man/man8/mkfs.xfs.8.in > +++ b/man/man8/mkfs.xfs.8.in > @@ -957,53 +957,61 @@ Do not attempt to discard blocks at mkfs time. > .B \-V > Prints the version number and exits. > .SH CONFIGURATION FILE FORMAT > -The optional default configuration file in > +The optional defaults configuration file in > .B @sysconfdir@/xfs/mkfs/default > as well as any alternate configuration file specified via the > .B \-c > -option follow a simple ini-style format as shown below. > -Available options consist of a small subset of the parameters available > -via the > -.BR mkfs.xfs (8) > -command line. > -Currently all default parameters can only be either enabled or disabled, > +option follow a simple ini-style format as shown below, with section names > +corresponding to command-line options as follows: > +.BR "\-d [data]" , > +.BR "\-i [inode]" , > +.BR "\-l [log]" , > +.BR "\-m [metadata]" , > +.BR "\-n [naming]" , > +.BR "\-r [rtdev]" . > +.br > +In each of these sections, a subset of the command-line suboptions may be > +specified to override built-in default settings. > +Currently all parameters can only be either enabled or disabled, > with a value of 1 to enable or 0 to disable. > See below for a list of all supported configuration parameters and their > current built-in default settings. > .PP > -.BI [data] > +.B # Comments and blank lines are allowed > +.br > +.B [data] > .br > -.BI noalign=0 > +.B noalign=0 > .PP > -.BI [inode] > +.B [inode] > .br > -.BI align=1 > +.B align=1 > .br > -.BI projid32bit=1 > +.B projid32bit=1 > .br > -.BI sparse=0 > +.B sparse=1 > .PP > -.BI [log] > +.B [log] > .br > -.BI lazy-count=1 > +.B lazy-count=1 > .PP > -.BI [metadata] > +.B [metadata] > .br > -.BI crc=1 > +.B crc=1 > .br > -.BI finobt=1 > +.B finobt=1 > .br > -.BI rmapbt=0 > +.B rmapbt=0 > .br > -.BI reflink=0 > +.B reflink=0 > .PP > -.BI [naming] > +.B [naming] > .br > -.BI ftype=1 > +.B ftype=1 > .PP > -.BI [rtdev] > +.B [rtdev] > .br > -.BI noalign=0 > +.B noalign=0 I would add a note that configuration files are still considered an experimental feature. With that, Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> --D > .PP > .SH SEE ALSO > .BR xfs (5), > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/man/man8/mkfs.xfs.8.in b/man/man8/mkfs.xfs.8.in index cf4bdf8..9c0c1a8 100644 --- a/man/man8/mkfs.xfs.8.in +++ b/man/man8/mkfs.xfs.8.in @@ -957,53 +957,61 @@ Do not attempt to discard blocks at mkfs time. .B \-V Prints the version number and exits. .SH CONFIGURATION FILE FORMAT -The optional default configuration file in +The optional defaults configuration file in .B @sysconfdir@/xfs/mkfs/default as well as any alternate configuration file specified via the .B \-c -option follow a simple ini-style format as shown below. -Available options consist of a small subset of the parameters available -via the -.BR mkfs.xfs (8) -command line. -Currently all default parameters can only be either enabled or disabled, +option follow a simple ini-style format as shown below, with section names +corresponding to command-line options as follows: +.BR "\-d [data]" , +.BR "\-i [inode]" , +.BR "\-l [log]" , +.BR "\-m [metadata]" , +.BR "\-n [naming]" , +.BR "\-r [rtdev]" . +.br +In each of these sections, a subset of the command-line suboptions may be +specified to override built-in default settings. +Currently all parameters can only be either enabled or disabled, with a value of 1 to enable or 0 to disable. See below for a list of all supported configuration parameters and their current built-in default settings. .PP -.BI [data] +.B # Comments and blank lines are allowed +.br +.B [data] .br -.BI noalign=0 +.B noalign=0 .PP -.BI [inode] +.B [inode] .br -.BI align=1 +.B align=1 .br -.BI projid32bit=1 +.B projid32bit=1 .br -.BI sparse=0 +.B sparse=1 .PP -.BI [log] +.B [log] .br -.BI lazy-count=1 +.B lazy-count=1 .PP -.BI [metadata] +.B [metadata] .br -.BI crc=1 +.B crc=1 .br -.BI finobt=1 +.B finobt=1 .br -.BI rmapbt=0 +.B rmapbt=0 .br -.BI reflink=0 +.B reflink=0 .PP -.BI [naming] +.B [naming] .br -.BI ftype=1 +.B ftype=1 .PP -.BI [rtdev] +.B [rtdev] .br -.BI noalign=0 +.B noalign=0 .PP .SH SEE ALSO .BR xfs (5),
A bit more clarification in the CONFIGURATION FILE FORMAT section of the mkfs.xfs manpage. Primarily, map the commandline switches to the section names explicitly. "optional default" sounds pretty weird, so call it the "defaults" configuration file instead. Document the comment format by example. Fix the sparse inode default setting in the example. Change the .BI tags to .B (there is no bold/italic switching). Signed-off-by: Eric Sandeen <sandeen@redhat.com> --- -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html