From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Chandan Babu R <chandanrlinux@gmail.com>
Cc: fstests@vger.kernel.org, linux-xfs@vger.kernel.org,
guaneryu@gmail.com, zlang@redhat.com
Subject: Re: [PATCH 1/2] xfs: Add realtime group
Date: Wed, 16 Sep 2020 09:51:42 -0700 [thread overview]
Message-ID: <20200916165142.GD7954@magnolia> (raw)
In-Reply-To: <20200916053407.2036-1-chandanrlinux@gmail.com>
On Wed, Sep 16, 2020 at 11:04:06AM +0530, Chandan Babu R wrote:
> This commit adds a new group to classify tests that can work with
> realtime devices.
>
> Signed-off-by: Chandan Babu R <chandanrlinux@gmail.com>
> ---
> tests/xfs/group | 52 ++++++++++++++++++++++++-------------------------
> 1 file changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/tests/xfs/group b/tests/xfs/group
> index ed0d389e..3bb0f674 100644
> --- a/tests/xfs/group
> +++ b/tests/xfs/group
> @@ -67,7 +67,7 @@
> 067 acl attr auto quick
> 068 auto stress dump
> 069 ioctl auto quick
> -070 auto quick repair
> +070 auto quick repair realtime
This test has an open-coded call to repair + rt volume, but is not
itself a test of rt functionality.
> 071 rw auto
> 072 rw auto prealloc quick
> 073 copy auto
> @@ -87,11 +87,11 @@
> 087 fuzzers
> 088 fuzzers
> 089 fuzzers
> -090 rw auto
> +090 rw auto realtime
> 091 fuzzers
> 092 other auto quick
> 093 fuzzers
> -094 metadata dir ioctl auto
> +094 metadata dir ioctl auto realtime
> 095 log v2log auto
> 096 mkfs v2log auto quick
> 097 fuzzers
> @@ -119,7 +119,7 @@
> 119 log v2log auto freeze
> 120 fuzzers
> 121 shutdown log auto quick
> -122 other auto quick clone
> +122 other auto quick clone realtime
This is an ondisk structure size check. It doesn't test rt
functionality, but I guess it doesn't really harm things to throw it on
the 'realtime' pile. I'm not objecting to this; it's just a funny
thought I had while reading this patch.
(Not sure why it's in 'clone' either...)
> 123 fuzzers
> 124 fuzzers
> 125 fuzzers
> @@ -128,7 +128,7 @@
> 128 auto quick clone fsr
> 129 auto quick clone
> 130 fuzzers clone
> -131 auto quick clone
> +131 auto quick clone realtime
> 132 auto quick
> 133 dangerous_fuzzers
> 134 dangerous_fuzzers
> @@ -188,7 +188,7 @@
> 188 ci dir auto
> 189 mount auto quick
> 190 rw auto quick
> -191-input-validation auto quick mkfs
> +191-input-validation auto quick mkfs realtime
> 192 auto quick clone
> 193 auto quick clone
> 194 rw auto
> @@ -272,7 +272,7 @@
> 273 auto rmap fsmap
> 274 auto quick rmap fsmap
> 275 auto quick rmap fsmap
> -276 auto quick rmap fsmap
> +276 auto quick rmap fsmap realtime
> 277 auto quick rmap fsmap
> 278 repair auto
> 279 auto mkfs
> @@ -287,7 +287,7 @@
> 288 auto quick repair fuzzers
> 289 growfs auto quick
> 290 auto rw prealloc quick ioctl zero
> -291 auto repair
> +291 auto repair realtime
This is a directory repair test, which doesn't exercise any rt volume
functionality.
...
FWIW I checked all the other tests that you added to the realtime group,
and the changes I don't have any comments about all look ok to me.
--D
> 292 auto mkfs quick
> 293 auto quick
> 294 auto dir metadata
> @@ -329,17 +329,17 @@
> 330 auto quick clone fsr quota
> 331 auto quick rmap clone
> 332 auto quick rmap clone collapse punch insert zero
> -333 auto quick rmap
> -334 auto quick rmap
> -335 auto rmap
> -336 auto rmap
> -337 fuzzers rmap
> -338 auto quick rmap
> -339 auto quick rmap
> -340 auto quick rmap
> -341 auto quick rmap
> -342 auto quick rmap
> -343 auto quick rmap collapse punch insert zero
> +333 auto quick rmap realtime
> +334 auto quick rmap realtime
> +335 auto rmap realtime
> +336 auto rmap realtime
> +337 fuzzers rmap realtime
> +338 auto quick rmap realtime
> +339 auto quick rmap realtime
> +340 auto quick rmap realtime
> +341 auto quick rmap realtime
> +342 auto quick rmap realtime
> +343 auto quick rmap collapse punch insert zero realtime
> 344 auto quick clone
> 345 auto quick clone
> 346 auto quick clone
> @@ -402,10 +402,10 @@
> 403 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> 404 dangerous_fuzzers dangerous_scrub dangerous_repair
> 405 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> -406 dangerous_fuzzers dangerous_scrub dangerous_repair
> -407 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> -408 dangerous_fuzzers dangerous_scrub dangerous_repair
> -409 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> +406 dangerous_fuzzers dangerous_scrub dangerous_repair realtime
> +407 dangerous_fuzzers dangerous_scrub dangerous_online_repair realtime
> +408 dangerous_fuzzers dangerous_scrub dangerous_repair realtime
> +409 dangerous_fuzzers dangerous_scrub dangerous_online_repair realtime
> 410 dangerous_fuzzers dangerous_scrub dangerous_repair
> 411 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> 412 dangerous_fuzzers dangerous_scrub dangerous_repair
> @@ -415,7 +415,7 @@
> 416 dangerous_fuzzers dangerous_scrub dangerous_repair
> 417 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> 418 dangerous_fuzzers dangerous_scrub dangerous_repair
> -419 auto quick swap
> +419 auto quick swap realtime
> 420 auto quick clone punch seek
> 421 auto quick clone punch seek
> 422 dangerous_scrub dangerous_online_repair
> @@ -477,8 +477,8 @@
> 478 dangerous_fuzzers dangerous_norepair
> 479 dangerous_fuzzers dangerous_norepair
> 480 dangerous_fuzzers dangerous_norepair
> -481 dangerous_fuzzers dangerous_norepair
> -482 dangerous_fuzzers dangerous_norepair
> +481 dangerous_fuzzers dangerous_norepair realtime
> +482 dangerous_fuzzers dangerous_norepair realtime
> 483 dangerous_fuzzers dangerous_norepair
> 484 dangerous_fuzzers dangerous_norepair
> 485 dangerous_fuzzers dangerous_norepair
> --
> 2.28.0
>
next prev parent reply other threads:[~2020-09-16 20:53 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-16 5:34 [PATCH 1/2] xfs: Add realtime group Chandan Babu R
2020-09-16 5:34 ` [PATCH V2 2/2] xfs: Check if rt summary/bitmap buffers are logged with correct xfs_buf type Chandan Babu R
2020-09-16 13:11 ` Chandan Babu R
2020-09-16 16:53 ` Darrick J. Wong
2020-09-17 3:50 ` Chandan Babu R
2020-09-16 16:51 ` Darrick J. Wong [this message]
2020-09-17 4:10 ` [PATCH 1/2] xfs: Add realtime group Chandan Babu R
2020-09-17 4:28 Chandan Babu R
2020-09-17 4:31 ` Chandan Babu R
2020-09-17 16:41 ` 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=20200916165142.GD7954@magnolia \
--to=darrick.wong@oracle.com \
--cc=chandanrlinux@gmail.com \
--cc=fstests@vger.kernel.org \
--cc=guaneryu@gmail.com \
--cc=linux-xfs@vger.kernel.org \
--cc=zlang@redhat.com \
/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).