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@xxxxxxxxx> > --- > 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 >
