diff options
author | Darrick J. Wong <djwong@kernel.org> | 2023-08-04 14:34:19 -0700 |
---|---|---|
committer | Zorro Lang <zlang@kernel.org> | 2023-08-05 21:53:09 +0800 |
commit | 0ca1d4fbb2e9a492968f2951df101f24477f7991 (patch) | |
tree | 4fb44fd2eaa39a270897f6fe2da2b968efddf2a0 | |
parent | d542fbc4ccef64887c5750a500b00318ffc3a7ff (diff) |
xfs: skip fragmentation tests when alwayscow mode is enabled, part 2v2023.08.06
If the always_cow debugging flag is enabled, all file writes turn into
copy writes. This dramatically ramps up fragmentation in the filesystem
(intentionally!) so there's no point in complaining about fragmentation.
I missed these two in the original commit because readahead for md5sum
would create large folios at the start of the file. This resulted in
the fdatatasync after the random writes issuing writeback for the whole
large folio, which reduced file fragmentation to the point where this
test started passing.
With Ritesh's patchset implementing sub-folio dirty tracking, this test
goes back to failing due to high fragmentation (as it did before large
folios) so we need to mask these off too.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>
-rwxr-xr-x | tests/xfs/180 | 1 | ||||
-rwxr-xr-x | tests/xfs/208 | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/tests/xfs/180 b/tests/xfs/180 index cfea2020..d2fac03a 100755 --- a/tests/xfs/180 +++ b/tests/xfs/180 @@ -23,6 +23,7 @@ _require_scratch_reflink _require_cp_reflink _require_xfs_io_command "fiemap" _require_xfs_io_command "cowextsize" +_require_no_xfs_always_cow echo "Format and mount" _scratch_mkfs > $seqres.full 2>&1 diff --git a/tests/xfs/208 b/tests/xfs/208 index 9a71b74f..1e7734b8 100755 --- a/tests/xfs/208 +++ b/tests/xfs/208 @@ -26,6 +26,7 @@ _require_scratch_reflink _require_cp_reflink _require_xfs_io_command "fiemap" _require_xfs_io_command "cowextsize" +_require_no_xfs_always_cow echo "Format and mount" _scratch_mkfs > $seqres.full 2>&1 |