* [PATCH xfsprogs manual] Inconsistency between the code and the manual page
@ 2019-06-12 5:20 Alvin Zheng
2019-06-12 6:47 ` Darrick J. Wong
2019-06-12 22:19 ` Dave Chinner
0 siblings, 2 replies; 4+ messages in thread
From: Alvin Zheng @ 2019-06-12 5:20 UTC (permalink / raw
To: linux-xfs, darrick.wong; +Cc: caspar
Hi,
The manual page of mkfs.xfs (xfsprogs-5.0.0) says "When specifying
parameters in units of sectors or filesystem blocks, the -s option or
the -b option first needs to be added to the command line. Failure to
specify the size of the units will result in illegal value errors when
parameters are quantified in those units". However, I read the code and
found that if the size of the block and sector is not specified, the
default size (block: 4k, sector: 512B) will be used. Therefore, the
following commands can work normally in xfsprogs-5.0.0.
mkfs.xfs -n size=2b /dev/vdc
mkfs.xfs -d agsize=8192b /dev/vdc
So I think the manual of mkfs.xfs should be updated as follows. Any
ideas?
diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
index 4b8c78c..45d7a84 100644
--- a/man/man8/mkfs.xfs.8
+++ b/man/man8/mkfs.xfs.8
@@ -115,9 +115,7 @@ When specifying parameters in units of sectors or
filesystem blocks, the
.B \-s
option or the
.B \-b
-option first needs to be added to the command line.
-Failure to specify the size of the units will result in illegal value
errors
-when parameters are quantified in those units.
+option can be used to specify the size of the sector or block. If the
size of the block or sector is not specified, the default size (block:
4KiB, sector: 512B) will be used.
.PP
Many feature options allow an optional argument of 0 or 1, to explicitly
disable or enable the functionality.
@@ -136,10 +134,6 @@ The filesystem block size is specified with a
in bytes. The default value is 4096 bytes (4 KiB), the minimum is 512,
and the
maximum is 65536 (64 KiB).
.IP
-To specify any options on the command line in units of filesystem
blocks, this
-option must be specified first so that the filesystem block size is
-applied consistently to all options.
-.IP
Although
.B mkfs.xfs
will accept any of these values and create a valid filesystem,
@@ -894,10 +888,6 @@ is 512 bytes. The minimum value for sector size is
.I sector_size
must be a power of 2 size and cannot be made larger than the
filesystem block size.
-.IP
-To specify any options on the command line in units of sectors, this
-option must be specified first so that the sector size is
-applied consistently to all options.
.RE
.TP
.BI \-L " label"
Best regards,
Alvin
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH xfsprogs manual] Inconsistency between the code and the manual page
2019-06-12 5:20 [PATCH xfsprogs manual] Inconsistency between the code and the manual page Alvin Zheng
@ 2019-06-12 6:47 ` Darrick J. Wong
2019-06-12 22:19 ` Dave Chinner
1 sibling, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2019-06-12 6:47 UTC (permalink / raw
To: Alvin Zheng; +Cc: linux-xfs, caspar
On Wed, Jun 12, 2019 at 01:20:46PM +0800, Alvin Zheng wrote:
> Hi,
>
> The manual page of mkfs.xfs (xfsprogs-5.0.0) says "When specifying
> parameters in units of sectors or filesystem blocks, the -s option or the -b
> option first needs to be added to the command line. Failure to specify the
> size of the units will result in illegal value errors when parameters are
> quantified in those units". However, I read the code and found that if the
> size of the block and sector is not specified, the default size (block: 4k,
> sector: 512B) will be used. Therefore, the following commands can work
> normally in xfsprogs-5.0.0.
>
> mkfs.xfs -n size=2b /dev/vdc
> mkfs.xfs -d agsize=8192b /dev/vdc
>
> So I think the manual of mkfs.xfs should be updated as follows. Any
> ideas?
/me vaguely remembers some sort of discussion years ago around unease
that one can specify a block count without specifying the size of those
blocks...
...but to be honest, I don't really see why that would be a problem.
This seems fine to me, though I defer to Eric the Real Maintainer. :)
--D
> diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
> index 4b8c78c..45d7a84 100644
> --- a/man/man8/mkfs.xfs.8
> +++ b/man/man8/mkfs.xfs.8
> @@ -115,9 +115,7 @@ When specifying parameters in units of sectors or
> filesystem blocks, the
> .B \-s
> option or the
> .B \-b
> -option first needs to be added to the command line.
> -Failure to specify the size of the units will result in illegal value
> errors
> -when parameters are quantified in those units.
> +option can be used to specify the size of the sector or block. If the size
> of the block or sector is not specified, the default size (block: 4KiB,
> sector: 512B) will be used.
> .PP
> Many feature options allow an optional argument of 0 or 1, to explicitly
> disable or enable the functionality.
> @@ -136,10 +134,6 @@ The filesystem block size is specified with a
> in bytes. The default value is 4096 bytes (4 KiB), the minimum is 512, and
> the
> maximum is 65536 (64 KiB).
> .IP
> -To specify any options on the command line in units of filesystem blocks,
> this
> -option must be specified first so that the filesystem block size is
> -applied consistently to all options.
> -.IP
> Although
> .B mkfs.xfs
> will accept any of these values and create a valid filesystem,
> @@ -894,10 +888,6 @@ is 512 bytes. The minimum value for sector size is
> .I sector_size
> must be a power of 2 size and cannot be made larger than the
> filesystem block size.
> -.IP
> -To specify any options on the command line in units of sectors, this
> -option must be specified first so that the sector size is
> -applied consistently to all options.
> .RE
> .TP
> .BI \-L " label"
>
>
> Best regards,
>
> Alvin
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH xfsprogs manual] Inconsistency between the code and the manual page
2019-06-12 5:20 [PATCH xfsprogs manual] Inconsistency between the code and the manual page Alvin Zheng
2019-06-12 6:47 ` Darrick J. Wong
@ 2019-06-12 22:19 ` Dave Chinner
2019-06-13 10:23 ` Alvin Zheng
1 sibling, 1 reply; 4+ messages in thread
From: Dave Chinner @ 2019-06-12 22:19 UTC (permalink / raw
To: Alvin Zheng; +Cc: linux-xfs, darrick.wong, caspar
On Wed, Jun 12, 2019 at 01:20:46PM +0800, Alvin Zheng wrote:
> Hi,
>
> The manual page of mkfs.xfs (xfsprogs-5.0.0) says "When specifying
> parameters in units of sectors or filesystem blocks, the -s option or the -b
> option first needs to be added to the command line. Failure to specify the
> size of the units will result in illegal value errors when parameters are
> quantified in those units". However, I read the code and found that if the
> size of the block and sector is not specified, the default size (block: 4k,
> sector: 512B) will be used. Therefore, the following commands can work
> normally in xfsprogs-5.0.0.
>
> mkfs.xfs -n size=2b /dev/vdc
> mkfs.xfs -d agsize=8192b /dev/vdc
>
> So I think the manual of mkfs.xfs should be updated as follows. Any
> ideas?
The intent of the wording in the mkfs man page is "when using a
custom sector or block size, it must be specified before any
parameter that uses units of sector or block sizes." So just
removing the "it must be specified first" wording is incorrect
because mkfs should throw errors is it is not specified first.
> diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
> index 4b8c78c..45d7a84 100644
> --- a/man/man8/mkfs.xfs.8
> +++ b/man/man8/mkfs.xfs.8
> @@ -115,9 +115,7 @@ When specifying parameters in units of sectors or
> filesystem blocks, the
> .B \-s
> option or the
> .B \-b
> -option first needs to be added to the command line.
> -Failure to specify the size of the units will result in illegal value
> errors
> -when parameters are quantified in those units.
> +option can be used to specify the size of the sector or block. If the size
> of the block or sector is not specified, the default size (block: 4KiB,
> sector: 512B) will be used.
That's fine to remove.
> .PP
> Many feature options allow an optional argument of 0 or 1, to explicitly
> disable or enable the functionality.
> @@ -136,10 +134,6 @@ The filesystem block size is specified with a
> in bytes. The default value is 4096 bytes (4 KiB), the minimum is 512, and
> the
> maximum is 65536 (64 KiB).
> .IP
> -To specify any options on the command line in units of filesystem blocks,
> this
> -option must be specified first so that the filesystem block size is
> -applied consistently to all options.
"If a non-default filesystem block size is specified, the option
must be specified before any options that use filesystem block size
units so that the non-default filesystem block size is applied
consistently to all options."
> -.IP
> Although
> .B mkfs.xfs
> will accept any of these values and create a valid filesystem,
> @@ -894,10 +888,6 @@ is 512 bytes. The minimum value for sector size is
> .I sector_size
> must be a power of 2 size and cannot be made larger than the
> filesystem block size.
> -.IP
> -To specify any options on the command line in units of sectors, this
> -option must be specified first so that the sector size is
> -applied consistently to all options.
Same wording as for the filesystem block size applies to sector
sizes as well.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH xfsprogs manual] Inconsistency between the code and the manual page
2019-06-12 22:19 ` Dave Chinner
@ 2019-06-13 10:23 ` Alvin Zheng
0 siblings, 0 replies; 4+ messages in thread
From: Alvin Zheng @ 2019-06-13 10:23 UTC (permalink / raw
To: Dave Chinner; +Cc: linux-xfs, darrick.wong, caspar
On 2019/6/13 6:19, Dave Chinner wrote:
> On Wed, Jun 12, 2019 at 01:20:46PM +0800, Alvin Zheng wrote:
>> Hi,
>>
>> The manual page of mkfs.xfs (xfsprogs-5.0.0) says "When specifying
>> parameters in units of sectors or filesystem blocks, the -s option or the -b
>> option first needs to be added to the command line. Failure to specify the
>> size of the units will result in illegal value errors when parameters are
>> quantified in those units". However, I read the code and found that if the
>> size of the block and sector is not specified, the default size (block: 4k,
>> sector: 512B) will be used. Therefore, the following commands can work
>> normally in xfsprogs-5.0.0.
>>
>> mkfs.xfs -n size=2b /dev/vdc
>> mkfs.xfs -d agsize=8192b /dev/vdc
>>
>> So I think the manual of mkfs.xfs should be updated as follows. Any
>> ideas?
> The intent of the wording in the mkfs man page is "when using a
> custom sector or block size, it must be specified before any
> parameter that uses units of sector or block sizes." So just
> removing the "it must be specified first" wording is incorrect
> because mkfs should throw errors is it is not specified first.
>
>> diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
>> index 4b8c78c..45d7a84 100644
>> --- a/man/man8/mkfs.xfs.8
>> +++ b/man/man8/mkfs.xfs.8
>> @@ -115,9 +115,7 @@ When specifying parameters in units of sectors or
>> filesystem blocks, the
>> .B \-s
>> option or the
>> .B \-b
>> -option first needs to be added to the command line.
>> -Failure to specify the size of the units will result in illegal value
>> errors
>> -when parameters are quantified in those units.
>> +option can be used to specify the size of the sector or block. If the size
>> of the block or sector is not specified, the default size (block: 4KiB,
>> sector: 512B) will be used.
> That's fine to remove.
>
>> .PP
>> Many feature options allow an optional argument of 0 or 1, to explicitly
>> disable or enable the functionality.
>> @@ -136,10 +134,6 @@ The filesystem block size is specified with a
>> in bytes. The default value is 4096 bytes (4 KiB), the minimum is 512, and
>> the
>> maximum is 65536 (64 KiB).
>> .IP
>> -To specify any options on the command line in units of filesystem blocks,
>> this
>> -option must be specified first so that the filesystem block size is
>> -applied consistently to all options.
> "If a non-default filesystem block size is specified, the option
> must be specified before any options that use filesystem block size
> units so that the non-default filesystem block size is applied
> consistently to all options."
>
>> -.IP
>> Although
>> .B mkfs.xfs
>> will accept any of these values and create a valid filesystem,
>> @@ -894,10 +888,6 @@ is 512 bytes. The minimum value for sector size is
>> .I sector_size
>> must be a power of 2 size and cannot be made larger than the
>> filesystem block size.
>> -.IP
>> -To specify any options on the command line in units of sectors, this
>> -option must be specified first so that the sector size is
>> -applied consistently to all options.
> Same wording as for the filesystem block size applies to sector
> sizes as well.
>
> Cheers,
>
> Dave.
Thanks for your suggestions, I will send a updated patch. :)
Best regards,
Alvin
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-13 10:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-12 5:20 [PATCH xfsprogs manual] Inconsistency between the code and the manual page Alvin Zheng
2019-06-12 6:47 ` Darrick J. Wong
2019-06-12 22:19 ` Dave Chinner
2019-06-13 10:23 ` Alvin Zheng
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.