From: Chandan Babu R <chandanrlinux@gmail.com>
To: "Darrick J. Wong" <djwong@kernel.org>
Cc: guaneryu@gmail.com, linux-xfs@vger.kernel.org,
fstests@vger.kernel.org, guan@eryu.me
Subject: Re: [PATCH 05/10] common/filter: refactor quota report filtering
Date: Thu, 11 Mar 2021 15:12:30 +0530 [thread overview]
Message-ID: <87k0qec9h5.fsf@garuda> (raw)
In-Reply-To: <161526483109.1214319.14555094406560973318.stgit@magnolia>
On 09 Mar 2021 at 10:10, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
>
> xfs/299 and xfs/050 share the same function to filter quota reporting
> into a format suitable for the golden output. Refactor this so that we
> can use it in a new test.
>
Looks good to me.
Reviewed-by: Chandan Babu R <chandanrlinux@gmail.com>
> Signed-off-by: Darrick J. Wong <djwong@kernel.org>
> ---
> common/filter | 24 ++++++++++++++++++++++++
> tests/xfs/050 | 30 ++++++------------------------
> tests/xfs/299 | 30 ++++++------------------------
> 3 files changed, 36 insertions(+), 48 deletions(-)
>
>
> diff --git a/common/filter b/common/filter
> index 2f3277f7..2efbbd99 100644
> --- a/common/filter
> +++ b/common/filter
> @@ -637,5 +637,29 @@ _filter_getcap()
> sed -e "s/= //" -e "s/\+/=/g"
> }
>
> +# Filter user/group/project id numbers out of quota reports, and standardize
> +# the block counts to use filesystem block size. Callers must set the id and
> +# bsize variables before calling this function.
> +_filter_quota_report()
> +{
> + test -n "$id" || echo "id must be set"
> + test -n "$bsize" || echo "block size must be set"
> +
> + tr -s '[:space:]' | \
> + perl -npe '
> + s/^\#'$id' /[NAME] /g;
> + s/^\#0 \d+ /[ROOT] 0 /g;
> + s/6 days/7 days/g' |
> + perl -npe '
> + $val = 0;
> + if ($ENV{'LARGE_SCRATCH_DEV'}) {
> + $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']}|'
> +}
> +
> # make sure this script returns success
> /bin/true
> diff --git a/tests/xfs/050 b/tests/xfs/050
> index 53412a13..1df97537 100755
> --- a/tests/xfs/050
> +++ b/tests/xfs/050
> @@ -47,24 +47,6 @@ bhard=$(( 1000 * $bsize ))
> isoft=4
> ihard=10
>
> -_filter_report()
> -{
> - tr -s '[:space:]' | \
> - perl -npe '
> - s/^\#'$id' /[NAME] /g;
> - s/^\#0 \d+ /[ROOT] 0 /g;
> - s/6 days/7 days/g' |
> - perl -npe '
> - $val = 0;
> - if ($ENV{'LARGE_SCRATCH_DEV'}) {
> - $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']}|'
> -}
> -
> # The actual point at which limit enforcement takes place for the
> # hard block limit is variable depending on filesystem blocksize,
> # and iosize. What we want to test is that the limit is enforced
> @@ -84,7 +66,7 @@ _filter_and_check_blks()
> }
> s/^(\#'$id'\s+)(\d+)/\1 =OK=/g;
> }
> - ' | _filter_report
> + ' | _filter_quota_report
> }
>
> _qsetup()
> @@ -134,7 +116,7 @@ _exercise()
> echo "*** report no quota settings" | tee -a $seqres.full
> $XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
> -c "repquota -birnN -$type" $SCRATCH_DEV |
> - _filter_report | LC_COLLATE=POSIX sort -ru
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** report initial settings" | tee -a $seqres.full
> @@ -147,7 +129,7 @@ _exercise()
> $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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** push past the soft inode limit" | tee -a $seqres.full
> @@ -159,7 +141,7 @@ _exercise()
> $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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** push past the soft block limit" | tee -a $seqres.full
> @@ -169,7 +151,7 @@ _exercise()
> -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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> # Note: for quota accounting (not enforcement), EDQUOT is not expected
> @@ -183,7 +165,7 @@ _exercise()
> -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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> # Note: for quota accounting (not enforcement), EDQUOT is not expected
> diff --git a/tests/xfs/299 b/tests/xfs/299
> index 15e0edf6..b862e67e 100755
> --- a/tests/xfs/299
> +++ b/tests/xfs/299
> @@ -40,24 +40,6 @@ _require_xfs_quota
> _require_xfs_mkfs_crc
> _require_xfs_crc
>
> -_filter_report()
> -{
> - tr -s '[:space:]' | \
> - perl -npe '
> - s/^\#'$id' /[NAME] /g;
> - s/^\#0 \d+ /[ROOT] 0 /g;
> - s/6 days/7 days/g' |
> - perl -npe '
> - $val = 0;
> - if ($ENV{'LARGE_SCRATCH_DEV'}) {
> - $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']}|'
> -}
> -
> # The actual point at which limit enforcement takes place for the
> # hard block limit is variable depending on filesystem blocksize,
> # and iosize. What we want to test is that the limit is enforced
> @@ -77,7 +59,7 @@ _filter_and_check_blks()
> }
> s/^(\#'$id'\s+)(\d+)/\1 =OK=/g;
> }
> - ' | _filter_report
> + ' | _filter_quota_report
> }
>
> _qsetup()
> @@ -120,7 +102,7 @@ _exercise()
> echo "*** report no quota settings" | tee -a $seqres.full
> $XFS_QUOTA_PROG -D $tmp.projects -P $tmp.projid -x \
> -c "repquota -birnN -$type" $SCRATCH_DEV |
> - _filter_report | LC_COLLATE=POSIX sort -ru
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** report initial settings" | tee -a $seqres.full
> @@ -133,7 +115,7 @@ _exercise()
> $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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** push past the soft inode limit" | tee -a $seqres.full
> @@ -145,7 +127,7 @@ _exercise()
> $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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> echo "*** push past the soft block limit" | tee -a $seqres.full
> @@ -155,7 +137,7 @@ _exercise()
> -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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> # Note: for quota accounting (not enforcement), EDQUOT is not expected
> @@ -169,7 +151,7 @@ _exercise()
> -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
> + _filter_quota_report | LC_COLLATE=POSIX sort -ru
>
> echo
> # Note: for quota accounting (not enforcement), EDQUOT is not expected
--
chandan
next prev parent reply other threads:[~2021-03-11 9:43 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-09 4:40 [PATCHSET 00/10] fstests: test kernel regressions fixed in 5.12 Darrick J. Wong
2021-03-09 4:40 ` [PATCH 01/10] xfs: test regression in xfs_bmap_validate_extent Darrick J. Wong
2021-03-09 4:40 ` [PATCH 02/10] generic: test reflink and copy_file_range behavior with O_SYNC and FS_XFLAG_SYNC files Darrick J. Wong
2021-03-09 4:40 ` [PATCH 03/10] xfs: test rtalloc alignment and math errors Darrick J. Wong
2021-03-11 7:58 ` Chandan Babu R
2021-03-23 4:15 ` Darrick J. Wong
2021-03-25 7:52 ` Chandan Babu R
2021-03-09 4:40 ` [PATCH 04/10] xfs: test mkfs min log size calculation w/ rt volumes Darrick J. Wong
2021-03-11 9:32 ` Chandan Babu R
2021-03-14 14:51 ` Eryu Guan
2021-03-14 16:39 ` Eryu Guan
2021-03-09 4:40 ` [PATCH 05/10] common/filter: refactor quota report filtering Darrick J. Wong
2021-03-11 9:42 ` Chandan Babu R [this message]
2021-03-09 4:40 ` [PATCH 06/10] xfs: test quota softlimit warning functionality Darrick J. Wong
2021-03-11 11:10 ` Chandan Babu R
2021-03-12 4:25 ` Chandan Babu R
2021-03-09 4:40 ` [PATCH 07/10] xfs/122: fix test for xfs_attr_shortform_t conversion Darrick J. Wong
2021-03-11 12:58 ` Christoph Hellwig
2021-03-09 4:40 ` [PATCH 08/10] generic: test file writers racing with FIDEDUPERANGE Darrick J. Wong
2021-03-12 7:13 ` Chandan Babu R
2021-03-09 4:40 ` [PATCH 09/10] generic: test a deadlock in xfs_rename when whiteing out files Darrick J. Wong
2021-03-14 18:06 ` Eryu Guan
2021-03-15 16:54 ` Darrick J. Wong
2021-03-09 4:40 ` [PATCH 10/10] xfs: test delalloc quota leak when chprojid fails Darrick J. Wong
2021-03-12 8:49 ` Chandan Babu R
2021-03-14 18:07 ` [PATCHSET 00/10] fstests: test kernel regressions fixed in 5.12 Eryu Guan
2021-03-14 21:36 ` Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k0qec9h5.fsf@garuda \
--to=chandanrlinux@gmail.com \
--cc=djwong@kernel.org \
--cc=fstests@vger.kernel.org \
--cc=guan@eryu.me \
--cc=guaneryu@gmail.com \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).