* [PATCH] fstests: update the punch, collapse, insert, and zero groups
@ 2018-05-29 19:01 Theodore Ts'o
2018-05-31 15:21 ` Eryu Guan
0 siblings, 1 reply; 2+ messages in thread
From: Theodore Ts'o @ 2018-05-29 19:01 UTC (permalink / raw)
To: fstests; +Cc: Theodore Ts'o
Update the group files to annotate those tests which have a
_require_xfs_io_command for punch, collapse, insert, and zero. This
makes it easier to exclude tests that use one of these fallocate
commands. Or if you want to specifically test for those fallocate
commands you can do this.
I maintain an out-of-tree xfstests patch which uses an XFS_IO_AVOID
environment variable to suppress running tests that use punch,
collapse, insert, etc. This was rejected because of the claim that it
could be done using groups. The reason why I still keep it as an
out-of-tree patch is because I was sure that when people added or
remove tests, they would forget to update the groups file with these
groups.
This commit is a demonstration of why my reason for keeping support
for XFS_IO_AVOID as an out-of-tree patch was prescient. :-)
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
---
tests/btrfs/group | 4 ++--
tests/generic/group | 10 +++++-----
tests/xfs/group | 12 ++++++------
3 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/tests/btrfs/group b/tests/btrfs/group
index f04ee8d5..487f3e10 100644
--- a/tests/btrfs/group
+++ b/tests/btrfs/group
@@ -110,7 +110,7 @@
105 auto quick send
106 auto quick clone compress
107 auto quick prealloc
-108 auto quick send clone punch
+108 auto quick send clone punch collapse
109 auto quick send clone compress
110 auto quick send
111 auto quick send
@@ -161,5 +161,5 @@
156 auto quick trim
157 auto quick raid
158 auto quick raid scrub
-159 auto quick
+159 auto quick collapse
160 auto quick
diff --git a/tests/generic/group b/tests/generic/group
index d0d42e97..349a66a6 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -350,7 +350,7 @@
345 auto
346 auto quick rw
347 auto quick rw thin
-348 auto quick metadata
+348 auto quick metadata insert
349 blockdev rw zero
350 blockdev rw punch
351 blockdev rw punch collapse insert zero
@@ -394,7 +394,7 @@
389 auto quick acl
390 auto freeze stress
391 auto quick rw
-392 shutdown auto quick metadata
+392 shutdown auto quick metadata punch
393 auto quick rw
394 auto quick
395 auto quick encrypt
@@ -448,7 +448,7 @@
443 auto quick rw
444 auto quick acl
445 auto quick rw
-446 auto quick rw
+446 auto quick rw punch
447 auto clone
448 auto quick rw
449 auto quick acl enospc
@@ -460,7 +460,7 @@
455 auto log replay
456 auto quick metadata
457 auto log replay clone
-458 auto quick clone
+458 auto quick clone collapse zero
459 auto thin
460 auto quick rw
461 auto shutdown stress
@@ -495,7 +495,7 @@
491 auto quick freeze mount
492 auto quick
493 auto quick swap
-494 auto quick swap
+494 auto quick swap punch
495 auto quick swap
900 auto quick verity
901 auto quick verity
diff --git a/tests/xfs/group b/tests/xfs/group
index e5fd1c6d..2319f58f 100644
--- a/tests/xfs/group
+++ b/tests/xfs/group
@@ -111,7 +111,7 @@
111 ioctl
112 fuzzers
113 fuzzers
-114 auto quick clone rmap
+114 auto quick clone rmap collapse insert
115 auto quick fuzzers
116 quota auto quick
117 fuzzers
@@ -329,7 +329,7 @@
329 auto quick clone fsr
330 auto quick clone fsr quota
331 auto quick rmap clone
-332 auto quick rmap clone
+332 auto quick rmap clone collapse punch insert zero
333 auto quick rmap
334 auto quick rmap
335 auto rmap
@@ -340,7 +340,7 @@
340 auto quick rmap
341 auto quick rmap
342 auto quick rmap
-343 auto quick rmap
+343 auto quick rmap collapse punch insert zero
344 auto quick clone
345 auto quick clone
346 auto quick clone
@@ -417,8 +417,8 @@
417 dangerous_fuzzers dangerous_scrub dangerous_online_repair
418 dangerous_fuzzers dangerous_scrub dangerous_repair
419 auto quick swap
-420 auto quick clone dedupe
-421 auto quick clone dedupe
+420 auto quick clone dedupe punch
+421 auto quick clone dedupe punch
422 dangerous_scrub dangerous_online_repair
423 dangerous_scrub
424 auto quick db
@@ -440,7 +440,7 @@
440 auto quick clone quota
441 auto quick clone quota
442 auto stress clone quota
-443 auto quick ioctl fsr
+443 auto quick ioctl fsr punch
444 auto quick
445 auto quick filestreams
446 auto quick
--
2.16.1.72.g5be1f00a9a
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] fstests: update the punch, collapse, insert, and zero groups
2018-05-29 19:01 [PATCH] fstests: update the punch, collapse, insert, and zero groups Theodore Ts'o
@ 2018-05-31 15:21 ` Eryu Guan
0 siblings, 0 replies; 2+ messages in thread
From: Eryu Guan @ 2018-05-31 15:21 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: fstests
On Tue, May 29, 2018 at 03:01:28PM -0400, Theodore Ts'o wrote:
> Update the group files to annotate those tests which have a
> _require_xfs_io_command for punch, collapse, insert, and zero. This
> makes it easier to exclude tests that use one of these fallocate
> commands. Or if you want to specifically test for those fallocate
> commands you can do this.
>
> I maintain an out-of-tree xfstests patch which uses an XFS_IO_AVOID
> environment variable to suppress running tests that use punch,
> collapse, insert, etc. This was rejected because of the claim that it
> could be done using groups. The reason why I still keep it as an
> out-of-tree patch is because I was sure that when people added or
> remove tests, they would forget to update the groups file with these
> groups.
I'll also pay more attention on this.
Thanks,
Eryu
>
> This commit is a demonstration of why my reason for keeping support
> for XFS_IO_AVOID as an out-of-tree patch was prescient. :-)
>
> Signed-off-by: Theodore Ts'o <tytso@mit.edu>
> ---
> tests/btrfs/group | 4 ++--
> tests/generic/group | 10 +++++-----
> tests/xfs/group | 12 ++++++------
> 3 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/tests/btrfs/group b/tests/btrfs/group
> index f04ee8d5..487f3e10 100644
> --- a/tests/btrfs/group
> +++ b/tests/btrfs/group
> @@ -110,7 +110,7 @@
> 105 auto quick send
> 106 auto quick clone compress
> 107 auto quick prealloc
> -108 auto quick send clone punch
> +108 auto quick send clone punch collapse
> 109 auto quick send clone compress
> 110 auto quick send
> 111 auto quick send
> @@ -161,5 +161,5 @@
> 156 auto quick trim
> 157 auto quick raid
> 158 auto quick raid scrub
> -159 auto quick
> +159 auto quick collapse
> 160 auto quick
> diff --git a/tests/generic/group b/tests/generic/group
> index d0d42e97..349a66a6 100644
> --- a/tests/generic/group
> +++ b/tests/generic/group
> @@ -350,7 +350,7 @@
> 345 auto
> 346 auto quick rw
> 347 auto quick rw thin
> -348 auto quick metadata
> +348 auto quick metadata insert
> 349 blockdev rw zero
> 350 blockdev rw punch
> 351 blockdev rw punch collapse insert zero
> @@ -394,7 +394,7 @@
> 389 auto quick acl
> 390 auto freeze stress
> 391 auto quick rw
> -392 shutdown auto quick metadata
> +392 shutdown auto quick metadata punch
> 393 auto quick rw
> 394 auto quick
> 395 auto quick encrypt
> @@ -448,7 +448,7 @@
> 443 auto quick rw
> 444 auto quick acl
> 445 auto quick rw
> -446 auto quick rw
> +446 auto quick rw punch
> 447 auto clone
> 448 auto quick rw
> 449 auto quick acl enospc
> @@ -460,7 +460,7 @@
> 455 auto log replay
> 456 auto quick metadata
> 457 auto log replay clone
> -458 auto quick clone
> +458 auto quick clone collapse zero
> 459 auto thin
> 460 auto quick rw
> 461 auto shutdown stress
> @@ -495,7 +495,7 @@
> 491 auto quick freeze mount
> 492 auto quick
> 493 auto quick swap
> -494 auto quick swap
> +494 auto quick swap punch
> 495 auto quick swap
> 900 auto quick verity
> 901 auto quick verity
> diff --git a/tests/xfs/group b/tests/xfs/group
> index e5fd1c6d..2319f58f 100644
> --- a/tests/xfs/group
> +++ b/tests/xfs/group
> @@ -111,7 +111,7 @@
> 111 ioctl
> 112 fuzzers
> 113 fuzzers
> -114 auto quick clone rmap
> +114 auto quick clone rmap collapse insert
> 115 auto quick fuzzers
> 116 quota auto quick
> 117 fuzzers
> @@ -329,7 +329,7 @@
> 329 auto quick clone fsr
> 330 auto quick clone fsr quota
> 331 auto quick rmap clone
> -332 auto quick rmap clone
> +332 auto quick rmap clone collapse punch insert zero
> 333 auto quick rmap
> 334 auto quick rmap
> 335 auto rmap
> @@ -340,7 +340,7 @@
> 340 auto quick rmap
> 341 auto quick rmap
> 342 auto quick rmap
> -343 auto quick rmap
> +343 auto quick rmap collapse punch insert zero
> 344 auto quick clone
> 345 auto quick clone
> 346 auto quick clone
> @@ -417,8 +417,8 @@
> 417 dangerous_fuzzers dangerous_scrub dangerous_online_repair
> 418 dangerous_fuzzers dangerous_scrub dangerous_repair
> 419 auto quick swap
> -420 auto quick clone dedupe
> -421 auto quick clone dedupe
> +420 auto quick clone dedupe punch
> +421 auto quick clone dedupe punch
> 422 dangerous_scrub dangerous_online_repair
> 423 dangerous_scrub
> 424 auto quick db
> @@ -440,7 +440,7 @@
> 440 auto quick clone quota
> 441 auto quick clone quota
> 442 auto stress clone quota
> -443 auto quick ioctl fsr
> +443 auto quick ioctl fsr punch
> 444 auto quick
> 445 auto quick filestreams
> 446 auto quick
> --
> 2.16.1.72.g5be1f00a9a
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-05-31 15:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-29 19:01 [PATCH] fstests: update the punch, collapse, insert, and zero groups Theodore Ts'o
2018-05-31 15:21 ` Eryu Guan
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.