Re: XFS fiemap issue with Linux 3.10.0-327.el7.x86_64 (CentOS 7)

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

 



On 2/15/16 11:26 AM, Jim Wilcoxson wrote:
> I'm developing a backup program, HashBackup, and recently added sparse
> "hole skipping" using fiemap.  

That is probably not safe.  Mapping can change immediately after fiemap
returns, and you could miss data.  cp went down that path a few years ago,
and it caused pain.

SEEK_HOLE/SEEK_DATA would be a better choice.

> Today I ran into a weird issue with xfs
> on Linux 3.10.
> 
> A test program creates a sparse file with 10000*(4K data, 4M hole).
> HB calls fiemap with start=0, length=0xFFFF...FFFF, mapped_extents=0
> to get the number of extents.    Fiemap is coming back with
> mapped_extents=1364 instead of 10000.

Take a look at xfs_bmap -v output to get a clear picture of what
is actually on disk.  It's possible that some combination of
speculative preallocation or other things has merged extents.

In general, the kernel has plenty of leeway in terms of how it
chooses to lay out a file.  "seek/write" doesn't necessarily
guarantee a hole.

> The fiemap.txt file says:
> 
> "If fm_extent_count is zero, then the fm_extents[] array is ignored
> (no extents will be returned), and the fm_mapped_extents count will
> hold the number of extents needed in fm_extents[] to hold the file's
> current mapping."
> 
> It doesn't say the filesystem can choose to return fewer extents if it
> wants, but maybe xfs interprets it this way and fiemap has to be
> called in a loop until the extent_last flag is set.  If that's the
> case, fiemap.txt should be updated.

It's not clear that it is returning fewer extents.  See what xfs_bmap
says, and compare.

But also, don't use fiemap results for anything that requires
data integrity...

-Eric

> Thanks,
> Jim
> (not on the xfs mailing list)
> 

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs



[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux