Re: [PATCH] xfs/{050,299}: clear quota warnings in between checks

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

 



On Thu, Jul 30, 2020 at 11:34:38AM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> 
> Both of these quota tests contain the output of the xfs_quota repquota
> command in the golden output.  Unfortunately, the output was recorded
> before quota soft warnings were implemented, which means they'll regress
> the output when we make quota warning work.  Fix this by resetting the
> warning count to zero before generating output.
> 
> While we're at it, use $XFS_QUOTA_PROG instead of xfs_quota.
> 
> Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>

Looks good.
Reviewed-by: Bill O'Donnell <billodo@xxxxxxxxxx>

> ---
> note that these changes are necessary because we fixed a longstanding
> warning counter bug for 5.9...
> ---
>  tests/xfs/050 |   23 ++++++++++++++++-------
>  tests/xfs/299 |   23 ++++++++++++++++-------
>  2 files changed, 32 insertions(+), 14 deletions(-)
> 
> diff --git a/tests/xfs/050 b/tests/xfs/050
> index 788ed7f1..c765f00b 100755
> --- a/tests/xfs/050
> +++ b/tests/xfs/050
> @@ -61,6 +61,7 @@ _filter_report()
>  			$val = $ENV{'NUM_SPACE_FILES'};
>  		}
>  		s/(^\[ROOT\] \S+ \S+ \S+ \S+ \[--------\] )(\S+)/$1@{[$2 - $val]}/g' |
> +	sed -e 's/ 65535 \[--------\]/ 00 \[--------\]/g' |
>  	perl -npe '
>  		s|^(.*?) (\d+) (\d+) (\d+)|$1 @{[$2 * 1024 /'$bsize']} @{[$3 * 1024 /'$bsize']} @{[$4 * 1024 /'$bsize']}|'
>  }
> @@ -128,9 +129,11 @@ _exercise()
>  
>  	echo "Using type=$type id=$id" >>$seqres.full
>  
> +	$XFS_QUOTA_PROG -x -c "warn -$type 65535 -d" $SCRATCH_DEV
> +
>  	echo
>  	echo "*** report no quota settings" | tee -a $seqres.full
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -139,11 +142,11 @@ _exercise()
>  	_file_as_id $SCRATCH_MNT/initme $id $type 1024 0
>  	echo "ls -l $SCRATCH_MNT" >>$seqres.full
>  	ls -l $SCRATCH_MNT >>$seqres.full
> -	xfs_quota -D $tmp.projects -P $temp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $temp.projid -x \
>  		-c "limit -$type bsoft=${bsoft} bhard=${bhard} $id" \
>  		-c "limit -$type isoft=$isoft ihard=$ihard $id" \
>  		$SCRATCH_DEV
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -154,7 +157,8 @@ _exercise()
>  	_file_as_id $SCRATCH_MNT/softie3 $id $type 1024 0
>  	_file_as_id $SCRATCH_MNT/softie4 $id $type 1024 0
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -i -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -162,7 +166,9 @@ _exercise()
>  	echo "*** push past the soft block limit" | tee -a $seqres.full
>  	_file_as_id $SCRATCH_MNT/softie $id $type $bsize 300
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -i -$type 0 $id" \
> +		-c "warn -b -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -174,7 +180,9 @@ _exercise()
>  		_file_as_id $SCRATCH_MNT/hard$i $id $type 1024 0
>  	done
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x  -c "warn -b -$type 0 $id" \
> +		-c "warn -i -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -185,7 +193,8 @@ _exercise()
>  	echo "ls -l $SCRATCH_MNT" >>$seqres.full
>  	ls -l $SCRATCH_MNT >>$seqres.full
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -b -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_and_check_blks | LC_COLLATE=POSIX sort -ru
>  
> diff --git a/tests/xfs/299 b/tests/xfs/299
> index adcf0e41..574a93b9 100755
> --- a/tests/xfs/299
> +++ b/tests/xfs/299
> @@ -54,6 +54,7 @@ _filter_report()
>  			$val = $ENV{'NUM_SPACE_FILES'};
>  		}
>  		s/(^\[ROOT\] \S+ \S+ \S+ \S+ \[--------\] )(\S+)/$1@{[$2 - $val]}/g' |
> +	sed -e 's/ 65535 \[--------\]/ 00 \[--------\]/g' |
>  	perl -npe '
>  		s|^(.*?) (\d+) (\d+) (\d+)|$1 @{[$2 * 1024 /'$bsize']} @{[$3 * 1024 /'$bsize']} @{[$4 * 1024 /'$bsize']}|'
>  }
> @@ -114,9 +115,11 @@ _exercise()
>  
>  	echo "Using type=$type id=$id" >>$seqres.full
>  
> +	$XFS_QUOTA_PROG -x -c "warn -$type 65535 -d" $SCRATCH_DEV
> +
>  	echo
>  	echo "*** report no quota settings" | tee -a $seqres.full
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -125,11 +128,11 @@ _exercise()
>  	_file_as_id $SCRATCH_MNT/initme $id $type 1024 0
>  	echo "ls -l $SCRATCH_MNT" >>$seqres.full
>  	ls -l $SCRATCH_MNT >>$seqres.full
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "limit -$type bsoft=${bsoft} bhard=${bhard} $id" \
>  		-c "limit -$type isoft=$isoft ihard=$ihard $id" \
>  		$SCRATCH_DEV
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -140,7 +143,8 @@ _exercise()
>  	_file_as_id $SCRATCH_MNT/softie3 $id $type 1024 0
>  	_file_as_id $SCRATCH_MNT/softie4 $id $type 1024 0
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -i -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -148,7 +152,9 @@ _exercise()
>  	echo "*** push past the soft block limit" | tee -a $seqres.full
>  	_file_as_id $SCRATCH_MNT/softie $id $type $bsize 200
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -i -$type 0 $id" \
> +		-c "warn -b -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -160,7 +166,9 @@ _exercise()
>  		_file_as_id $SCRATCH_MNT/hard$i $id $type 1024 0
>  	done
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x  -c "warn -b -$type 0 $id" \
> +		-c "warn -i -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_report | LC_COLLATE=POSIX sort -ru
>  
> @@ -171,7 +179,8 @@ _exercise()
>  	echo "ls -l $SCRATCH_MNT" >>$seqres.full
>  	ls -l $SCRATCH_MNT >>$seqres.full
>  	_qmount
> -	xfs_quota -D $tmp.projects -P $tmp.projid -x \
> +	$XFS_QUOTA_PROG -x -c "warn -b -$type 0 $id" $SCRATCH_DEV
> +	$XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
>  		-c "repquota -birnN -$type" $SCRATCH_DEV |
>  		_filter_and_check_blks | LC_COLLATE=POSIX sort -ru
>  
> 




[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux