Re: [PATCH] generic: test FIEMAP on extended attribute blocks

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



On Fri, Apr 07, 2017 at 05:26:16PM +0800, Eryu Guan wrote:
> On Thu, Apr 06, 2017 at 09:01:30AM -0700, Darrick J. Wong wrote:
> > Make sure that FIEMAP produces some output when we add enough xattrs
> > to force the xattrs to be stored in an external block.
> > 
> > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> 
> Test itself looks fine to me. But given that there's disagreement on
> fiemap on xattr (still an reasonable test?) and test crashes debug built
> & 512b block size XFS, I'll hold this test for now.

ext4 also implements fiemap on xattr, so at the very least we still need
to test that even if XFS ends up dropping xattr fiemap.

I'll push out a v2 of the xfs patch soon...

--D

> 
> Thanks,
> Eryu
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" 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 fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux