Re: [PATCH 7/6] xfs_bmap: fix -n documentation in manpage

["Darrick J. Wong" <darrick.wong@oracle.com>]

On Thu, Aug 24, 2017 at 03:09:32PM -0500, Eric Sandeen wrote:
> xfs_bmap's manpage mis-describes the behavior from the
> -n option.  xfs_io's fiemap command references the xfs_bmap
> manpage, and has the same problem:
> 
> -n does not change the query batch size, it limits the number
> of extents displayed.
> 
> This has been true for 15+ years, so change the documentation
> to match reality.
> 
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>

For patches 6 & 7,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>

--D

> ---
> 
> diff --git a/man/man8/xfs_bmap.8 b/man/man8/xfs_bmap.8
> index 098cfae..c725519 100644
> --- a/man/man8/xfs_bmap.8
> +++ b/man/man8/xfs_bmap.8
> @@ -77,12 +77,11 @@ option is used.
>  .BI \-n " num_extents"
>  If this option is given,
>  .B xfs_bmap
> -obtains the extent list of the file in groups of
> +will display at most
>  .I num_extents
>  extents. In the absence of
>  .BR \-n ", " xfs_bmap
> -queries the system for the number of extents in the file and uses that
> -value to compute the group size.
> +will display all extents in the file.
>  .TP
>  .B \-p
>  If this option is used,
> 
> --
> 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