Re: [PATCH 4/4] xfs/260: Move xfs/260 to generic

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On 2020/6/10 0:42, Darrick J. Wong wrote:
On Wed, Jun 03, 2020 at 10:01:15PM +0800, Xiao Yang wrote:
Both ext4 and xfs support per-inode DAX flag now so move it to generic.

Signed-off-by: Xiao Yang<yangx.jy@xxxxxxxxxxxxxx>
---
  tests/{xfs/260 =>  generic/602} | 12 ++++++++----
  tests/generic/602.out          |  2 ++
  tests/generic/group            |  1 +
  tests/xfs/260.out              |  2 --
  tests/xfs/group                |  1 -
  5 files changed, 11 insertions(+), 7 deletions(-)
  rename tests/{xfs/260 =>  generic/602} (91%)
  create mode 100644 tests/generic/602.out
  delete mode 100644 tests/xfs/260.out

diff --git a/tests/xfs/260 b/tests/generic/602
similarity index 91%
rename from tests/xfs/260
rename to tests/generic/602
index fbdc4cd8..0cd20f31 100755
--- a/tests/xfs/260
+++ b/tests/generic/602
@@ -2,7 +2,7 @@
  # SPDX-License-Identifier: GPL-2.0
  # Copyright (c) 2017 Red Hat Inc.  All Rights Reserved.
  #
-# FS QA Test 260
+# FS QA Test 602
  #
  # Test per-inode DAX flag by mmap direct/buffered IO.
  #
@@ -28,7 +28,7 @@ _cleanup()
  # remove previous $seqres.full before test
  rm -f $seqres.full

-_supported_fs xfs
+_supported_fs generic
  _supported_os Linux
  _require_scratch_dax_mountopt "dax"
  _require_test_program "feature"
@@ -121,8 +121,12 @@ do_tests()
  	t_dax_flag_mmap_dio $((64 * 1024 * 1024))
  }

-# make xfs 2Mb aligned for PMD fault testing
-_scratch_mkfs "-d su=2m,sw=1">  /dev/null 2>&1
+# make 2Mb aligned for PMD fault testing
+mkfs_opts=""
+[ "$FSTYP" = "ext4" ]&&  mkfs_opts="-E stride=512,stripe_width=1"
+[ "$FSTYP" = "xfs" ]&&  mkfs_opts="-d su=2m,sw=1"

/me wonders if this ought to be smart enough to sniff out the hugepage
size on the test machine...
Hi Darrick,

PMD page is always 2M, I am not sure why we need to sniff out the hugepage size? Could you give me an example?

Best Regards,
Xiao Yang

--D

+
+_scratch_mkfs "$mkfs_opts">  /dev/null 2>&1

  # mount with dax option
  _scratch_mount "-o dax"
diff --git a/tests/generic/602.out b/tests/generic/602.out
new file mode 100644
index 00000000..61976e5e
--- /dev/null
+++ b/tests/generic/602.out
@@ -0,0 +1,2 @@
+QA output created by 602
+Silence is golden
diff --git a/tests/generic/group b/tests/generic/group
index c6ce029c..4bda3772 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -604,3 +604,4 @@
  599 auto quick remount shutdown
  600 auto quick quota
  601 auto quick quota
+602 auto attr quick dax
diff --git a/tests/xfs/260.out b/tests/xfs/260.out
deleted file mode 100644
index 18ca517c..00000000
--- a/tests/xfs/260.out
+++ /dev/null
@@ -1,2 +0,0 @@
-QA output created by 260
-Silence is golden
diff --git a/tests/xfs/group b/tests/xfs/group
index daf54add..71c30898 100644
--- a/tests/xfs/group
+++ b/tests/xfs/group
@@ -257,7 +257,6 @@
  257 auto quick clone
  258 auto quick clone
  259 auto quick
-260 auto attr quick dax
  261 auto quick quota
  262 dangerous_fuzzers dangerous_scrub dangerous_online_repair
  263 auto quick quota
--
2.21.0





.







[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux