Re: [PATCH 3/5] common/btrfs: quota rescan helpers

[Josef Bacik <josef@toxicpanda.com>]

On Wed, Jul 05, 2023 at 04:42:25PM -0700, Boris Burkov wrote:
> Many btrfs tests explicitly trigger quota rescan. This is not a
> meaningful operation for simple quotas, so we wrap it in a helper that
> doesn't blow up quite so badly and lets us run those tests where the
> rescan is a qgroup detail.
> 
> Signed-off-by: Boris Burkov <boris@bur.io>
> ---
>  common/btrfs | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
> 
> diff --git a/common/btrfs b/common/btrfs
> index 66c065a10..d88feaded 100644
> --- a/common/btrfs
> +++ b/common/btrfs
> @@ -715,6 +715,31 @@ _qgroup_mode()
>  	fi
>  }
>  
> +_check_regular_qgroup()
> +{
> +	local mnt=$1
> +
> +	_qgroup_mode $mnt | grep -q 'qgroup'
> +}
> +
> +_qgroup_rescan()
> +{
> +	local mnt=$1
> +
> +	_check_regular_qgroup $mnt || return 1
> +	_run_btrfs_util_prog quota rescan -w $mnt
> +}
> +
> +_require_qgroup_rescan()
> +{
> +	_scratch_mkfs >>$seqres.full 2>&1
> +	_scratch_mount
> +    _run_btrfs_util_prog quota enable $SCRATCH_MNT
> +    $BTRFS_UTIL_PROG quota rescan -w $SCRATCH_MNT || \
> +         _notrun "not able to run quota rescan"
> +	_scratch_unmount
> +}

Looks like whitespace errors here.  Thanks,

Josef