Re: [PATCH] xfs_io: add missed inode command into man page

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/20/17 12:23 PM, Darrick J. Wong wrote:
> On Fri, Apr 21, 2017 at 01:06:57AM +0800, Zorro Lang wrote:
>> There's an "inode" command in xfs_io, it's used to query physical
>> information about an inode. But there's not any information about
>> it in xfs_io and other related man pages. So document this command
>> in the xfs_io man page now.
>>
>> Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
>> ---
>>  man/man8/xfs_io.8 | 17 +++++++++++++++++
>>  1 file changed, 17 insertions(+)
>>
>> diff --git a/man/man8/xfs_io.8 b/man/man8/xfs_io.8
>> index 19e1ae4..ceb6cf4 100644
>> --- a/man/man8/xfs_io.8
>> +++ b/man/man8/xfs_io.8
>> @@ -650,6 +650,23 @@ sec uses UNIX timestamp notation and is the seconds elapsed since
>>  nsec is the nanoseconds since the sec. This value needs to be in
>>  the range 0-999999999 with UTIME_NOW and UTIME_OMIT being exceptions.
>>  Each (sec, nsec) pair constitutes a single timestamp value.
>> +.RE
>> +.PD
>> +.TP
>> +.BI " inode [ [ -n ] " number " ] [ -v ]"
>> +The inode command querys physical information about an inode, it
> 
> "queries"
> 
>> +returns 1 if any inode number greater than 32 bits exists in the
> 
> Technically, it returns 1 if any inode number would overflow 32 bits,
> which is to say that it returns 1 if any inode number is greater than
> 2^32-1.
> 
>> +filesystem, or 0 if none exist by default. The
> 
> You could leave out "by default".

I think that the intent here was to say that by default (i.e. with no
other options given, this is what the command does - return 1 or 0 based on
whether or not any any inode number exceeding 32 bits is currently in use.

>> +.I number
>> +is used to specify an inode number, then inode command returns this
>> +inode number if it's in use, or 0 if not. But if use
> 
> "If an inode /number/ is given, this command returns that inode number if
> the inode is in use, or zero if not."?
> 
>> +.B \-n
>> +with
>> +.I number
>> +, the next used inode number after this number will be returned, or 0
>> +if can't find the next one.
> 
> "If -n is given with /number/, the next inode number in use will be
> returned, or zero if there aren't any more"?
> 
>> +.B \-v
>> +for verbose mode, the inode number's size in bits will be displayed.

This is how I would describe the behavior, feel free to use as much or
as little as you like:

With no arguments, the inode command will return 1 or 0, indicating whether
or not any inode numbers greater than 32 bits are currently in use in the
filesystem.

If given an inode /number/ as an argument, the command will return /number/ or
0, depending on whether or not inode /number/ is in use.

With -n /number/, the command will return the next inode number after /number/
which is in use.

With -v, the command will also report the number of bits (32 or 64) used
by the inode number printed in the result; if no inode /number/ was specified
on the commandline, the maximum possible inode number in the system will be
printed along with its size.

> (What a strange command...)

yup.

-Eric

> --D
> 
>>  
>>  .SH MEMORY MAPPED I/O COMMANDS
>>  .TP
>> -- 
>> 2.7.4
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> 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@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux