Re: [PATCH] fiemap: add a nosync option

Christoph Hellwig <[email protected]>
Newsgroups org.kernel.vger.linux-xfs
Message-ID <[email protected]>
On Tue, Jul 14, 2026 at 12:53:14PM +0200, Pankaj Raghav wrote:
> diff --git a/io/fiemap.c b/io/fiemap.c
> index b41f71bf..7aa8868d 100644
> --- a/io/fiemap.c
> +++ b/io/fiemap.c
> @@ -36,6 +36,7 @@ fiemap_help(void)
>  " -a -- prints the attribute fork map instead of the data fork.\n"
>  " -l -- also displays the length of each extent in 512-byte blocks.\n"
>  " -n -- query n extents.\n"
> +" -s -- do not use FIEMAP_FLAG_SYNC.\n"

The xfs_io manpage refers to the xfs_bmap man page for the argument,
and xfs_bmap already supports a -e option to skip the sync.  Maybe we
should match that for consistency?

Otherwise this looks good.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.