Re: [PATCH 5/9] xfs/327: fix inode reflink flag checking

Christoph Hellwig
2020-10-28T21:51:56Z

On Tue, Oct 27, 2020 at 12:02:02PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> This is a regression test that tried to make sure that repair correctly
> clears the XFS inode reflink flag when it detects files that do not
> share any blocks. However, it does this checking by looking at the
> (online) lsattr output. This worked fine during development when we
> exposed the reflink state via the stat ioctls, but that has long since
> been removed. Now the only way to check is via xfs_db, so switch it to
> use that.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>

Looks good,

Reviewed-by: Christoph Hellwig <hch@lst.de>