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.