On Wed, May 12, 2021 at 07:32:26AM +0800, Gao Xiang wrote:
> In order to detect whether the current kernel supports XFS shrinking.
>
> Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx>
> Signed-off-by: Gao Xiang <hsiangkao@xxxxxxxxxx>
> ---
> common/xfs | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/common/xfs b/common/xfs
> index 69f76d6e..a0a4032a 100644
> --- a/common/xfs
> +++ b/common/xfs
> @@ -766,6 +766,26 @@ _require_xfs_mkfs_without_validation()
> fi
> }
>
> +_require_xfs_scratch_shrink()
I renamed this to _require_scratch_xfs_shrink(), as most of other
helpers are in this format as _require_scratch_xfs_bigtime(), the only
exception is _require_xfs_scratch_rmapbt(), I think we should change
that too in another patch.
> +{
> + _require_scratch
> + _require_command "$XFS_GROWFS_PROG" xfs_growfs
> +
> + _scratch_mkfs_xfs | _filter_mkfs 2>$tmp.mkfs >/dev/null
> + . $tmp.mkfs
> + _scratch_mount
> + # here just to check if kernel supports, no need do more extra work
> + errmsg=$($XFS_GROWFS_PROG -D$((dblocks-1)) "$SCRATCH_MNT" 2>&1)
Also make it as local.
Thanks,
Eryu
> + if [ "$?" -ne 0 ]; then
> + echo "$errmsg" | grep 'XFS_IOC_FSGROWFSDATA xfsctl failed: Invalid argument' > /dev/null && \
> + _notrun "kernel does not support shrinking"
> + echo "$errmsg" | grep 'data size .* too small, old size is ' > /dev/null && \
> + _notrun "xfsprogs does not support shrinking"
> + _fail "$XFS_GROWFS_PROG failed unexpectedly: $errmsg"
> + fi
> + _scratch_unmount
> +}
> +
> # XFS ability to change UUIDs on V5/CRC filesystems
> #
> _require_meta_uuid()
> --
> 2.27.0
