[PATCH] xfs: flag panicking tests as dangerous

Donald Douwsma <[email protected]>
Newsgroups org.kernel.vger.linux-xfs,org.kernel.vger.fstests
Message-ID <[email protected]>
xfs/649 can derail test runs as its not flagged as dangerous.
Fix that and pick up other tests that can also BUG or hang.

Signed-off-by: Donald Douwsma <[email protected]>
---
Note: xfs/270 mentions a potential hang/panic, but its unclear if that's
likely or expected. If not then we shouldn't flag it, thoughts?
---
 tests/xfs/051 | 2 +-
 tests/xfs/081 | 2 +-
 tests/xfs/270 | 2 +-
 tests/xfs/306 | 2 +-
 tests/xfs/311 | 2 +-
 tests/xfs/431 | 2 +-
 tests/xfs/649 | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/tests/xfs/051 b/tests/xfs/051
index 95c89bbab..a46959375 100755
--- a/tests/xfs/051
+++ b/tests/xfs/051
@@ -9,7 +9,7 @@
 # causes BUG() errors or a panic.
 #
 . ./common/preamble
-_begin_fstest shutdown auto log metadata
+_begin_fstest shutdown auto log metadata dangerous
 
 _cleanup()
 {
diff --git a/tests/xfs/081 b/tests/xfs/081
index 14c0d13e5..6f85bd8a6 100755
--- a/tests/xfs/081
+++ b/tests/xfs/081
@@ -13,7 +13,7 @@
 # removexattr will hang.
 #
 . ./common/preamble
-_begin_fstest auto quick attr
+_begin_fstest auto quick attr dangerous
 
 # Import common functions.
 . ./common/filter
diff --git a/tests/xfs/270 b/tests/xfs/270
index d3bce386a..71430dd15 100755
--- a/tests/xfs/270
+++ b/tests/xfs/270
@@ -10,7 +10,7 @@
 # ro-compat features), and this case is the regression testcase.
 #
 . ./common/preamble
-_begin_fstest auto quick mount
+_begin_fstest auto quick mount dangerous
 
 # Import common functions.
 . ./common/filter
diff --git a/tests/xfs/306 b/tests/xfs/306
index d48b75363..8430aea84 100755
--- a/tests/xfs/306
+++ b/tests/xfs/306
@@ -13,7 +13,7 @@
 # CIL (i.e., when the log is subsequently flushed).
 #
 . ./common/preamble
-_begin_fstest auto quick punch
+_begin_fstest auto quick punch dangerous
 
 # Import common functions.
 
diff --git a/tests/xfs/311 b/tests/xfs/311
index 8b806fc29..766d8bad4 100755
--- a/tests/xfs/311
+++ b/tests/xfs/311
@@ -11,7 +11,7 @@
 # panic.
 #
 . ./common/preamble
-_begin_fstest auto quick
+_begin_fstest auto quick dangerous
 
 # Override the default cleanup function.
 _cleanup()
diff --git a/tests/xfs/431 b/tests/xfs/431
index 9764d2437..5fb76ccd1 100755
--- a/tests/xfs/431
+++ b/tests/xfs/431
@@ -12,7 +12,7 @@
 # See CVE-2017-14340 for more information.
 #
 . ./common/preamble
-_begin_fstest auto quick
+_begin_fstest auto quick dangerous
 
 # Import common functions.
 . ./common/filter
diff --git a/tests/xfs/649 b/tests/xfs/649
index 56277c629..2f735fb0e 100755
--- a/tests/xfs/649
+++ b/tests/xfs/649
@@ -23,7 +23,7 @@
 #
 
 . ./common/preamble
-_begin_fstest auto quick attr
+_begin_fstest auto quick attr dangerous
 
 _fixed_by_kernel_commit ae668cd567a6 \
 	"xfs: do not propagate ENODATA disk errors into xattr code"
-- 
2.52.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.