Re: [PATCH 05/13] xfs: add a _require_xfs_data_csum helper

["Darrick J. Wong" <djwong@kernel.org>]

On Thu, Sep 24, 2026 at 12:07:46PM +0200, Christoph Hellwig wrote:
> Add a helper to check if a given mountpoint supports (RT) data checksums
> on XFS.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  common/xfs | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/common/xfs b/common/xfs
> index 8254e48603af..e9f6877c98cd 100644
> --- a/common/xfs
> +++ b/common/xfs
> @@ -2448,3 +2448,16 @@ _require_xfs_healer()
>  
>  	_scratch_mkfs_xfs_supported "$arg" concurrency=0
>  }
> +
> +# require (RT) data checksum support on the passed mountpoint
> +_require_xfs_data_csum()
> +{
> +	mnt=$1
> +
> +	csumval=$($XFS_IO_PROG -c 'statfs -g' $mnt | \
> +		grep "geom.rtcsum_type" | awk '{print $3}')
> +	echo "csumval: $csumval" >> $seqres.full
> +	if [ -e "$csumval" -o "$csumval" -eq "0" ]; then
> +		_notrun "requires XFS data checsums"

		                           checksums

Any chance we could export file data checksum type via statx or
somewhere?

With the typo fixed,
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>

--D

> +	fi
> +}
> -- 
> 2.53.0
> 
>