generic/711,xfs/537: actually fork these tests for exchange-range

Fork g/711 to g/752, x/537 to x/615. These tests to check the same
things with exchange-range as they do for swapext, since the code
porting swapext to commit-range has been dropped.

I was going to fork xfs/789 as well, but it turns out that generic/714
covers this sufficiently so for that one, we just strike fiexchange
from the group tag.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Zorro Lang <zlang@kernel.org>

diff --git a/tests/generic/711 b/tests/generic/711
index b107f976ef9049df8436614266a40acf96baaaee..792136306cdd857631449019cef2796a271a9aa4 100755 (executable)
--- a/tests/generic/711
+++ b/tests/generic/711
@@ -7,7 +7,7 @@
 # Make sure that swapext won't touch a swap file.
 
 . ./common/preamble
-_begin_fstest auto quick fiexchange swapext
+_begin_fstest auto quick swapext
 
 # Override the default cleanup function.
 _cleanup()

diff --git a/tests/generic/752 b/tests/generic/752
new file mode 100755 (executable)
index 0000000..2cdb2c7
--- /dev/null
+++ b/tests/generic/752
@@ -0,0 +1,45 @@
+#! /bin/bash
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2022 Oracle.  All Rights Reserved.
+#
+# FS QA Test No. 752
+#
+# Make sure that exchangerange won't touch a swap file.
+
+. ./common/preamble
+_begin_fstest auto quick fiexchange
+
+# Override the default cleanup function.
+_cleanup()
+{
+       cd /
+       test -e "$dir/a" && swapoff $dir/a
+       rm -r -f $tmp.* $dir
+}
+
+# Import common functions.
+. ./common/filter
+
+# real QA test starts here
+_require_xfs_io_command exchangerange
+_require_test
+
+dir=$TEST_DIR/test-$seq
+mkdir -p $dir
+
+# Set up a fragmented swapfile and a dummy donor file.
+$XFS_IO_PROG -f -c 'pwrite -S 0x58 0 32m -b 1m' -c fsync $dir/a >> $seqres.full
+$here/src/punch-alternating $dir/a
+$XFS_IO_PROG -f -c 'pwrite -S 0x58 0 32m -b 1m' -c fsync $dir/a >> $seqres.full
+$MKSWAP_PROG $dir/a >> $seqres.full
+
+$XFS_IO_PROG -f -c 'pwrite -S 0x59 0 32m -b 1m' $dir/b >> $seqres.full
+
+swapon $dir/a || _notrun 'failed to swapon'
+
+# Now try to exchangerange.
+$XFS_IO_PROG -c "exchangerange $dir/b" $dir/a
+
+# success, all done
+status=0
+exit

diff --git a/tests/generic/752.out b/tests/generic/752.out
new file mode 100644 (file)
index 0000000..4769edf
--- /dev/null
+++ b/tests/generic/752.out
@@ -0,0 +1,2 @@
+QA output created by 752
+exchangerange: Text file busy

diff --git a/tests/xfs/615 b/tests/xfs/615
new file mode 100755 (executable)
index 0000000..783af4e
--- /dev/null
+++ b/tests/xfs/615
@@ -0,0 +1,89 @@
+#! /bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (c) 2024 Oracle.  All Rights Reserved.
+#
+# FS QA Test No. 615
+#
+# Verify that XFS does not cause inode fork's extent count to overflow when
+# exchanging ranges between files
+. ./common/preamble
+_begin_fstest auto quick collapse fiexchange
+
+# Import common functions.
+. ./common/filter
+. ./common/inject
+
+# real QA test starts here
+
+_supported_fs xfs
+_require_scratch
+_require_xfs_debug
+_require_xfs_scratch_rmapbt
+_require_xfs_io_command "fcollapse"
+_require_xfs_io_command "exchangerange"
+_require_xfs_io_error_injection "reduce_max_iextents"
+
+echo "* Exchange extent forks"
+
+echo "Format and mount fs"
+_scratch_mkfs >> $seqres.full
+_scratch_mount >> $seqres.full
+
+bsize=$(_get_file_block_size $SCRATCH_MNT)
+
+srcfile=${SCRATCH_MNT}/srcfile
+donorfile=${SCRATCH_MNT}/donorfile
+
+echo "Create \$donorfile having an extent of length 67 blocks"
+$XFS_IO_PROG -f -s -c "pwrite -b $((17 * bsize)) 0 $((17 * bsize))" $donorfile \
+       >> $seqres.full
+
+# After the for loop the donor file will have the following extent layout
+# | 0-4 | 5 | 6 | 7 | 8 | 9 | 10 |
+echo "Fragment \$donorfile"
+for i in $(seq 5 10); do
+       start_offset=$((i * bsize))
+       $XFS_IO_PROG -f -c "fcollapse $start_offset $bsize" $donorfile >> $seqres.full
+done
+
+echo "Create \$srcfile having an extent of length 18 blocks"
+$XFS_IO_PROG -f -s -c "pwrite -b $((18 * bsize)) 0 $((18 * bsize))" $srcfile \
+       >> $seqres.full
+
+echo "Fragment \$srcfile"
+# After the for loop the src file will have the following extent layout
+# | 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7-10 |
+for i in $(seq 1 7); do
+       start_offset=$((i * bsize))
+       $XFS_IO_PROG -f -c "fcollapse $start_offset $bsize" $srcfile >> $seqres.full
+done
+
+echo "Collect \$donorfile's extent count"
+donor_nr_exts=$(_xfs_get_fsxattr nextents $donorfile)
+
+echo "Collect \$srcfile's extent count"
+src_nr_exts=$(_xfs_get_fsxattr nextents $srcfile)
+
+echo "Inject reduce_max_iextents error tag"
+_scratch_inject_error reduce_max_iextents 1
+
+echo "Exchange \$srcfile's and \$donorfile's extent forks"
+$XFS_IO_PROG -f -c "exchangerange $donorfile" $srcfile >> $seqres.full 2>&1
+
+echo "Check for \$donorfile's extent count overflow"
+nextents=$(_xfs_get_fsxattr nextents $donorfile)
+
+if (( $nextents == $src_nr_exts )); then
+       echo "\$donorfile: Extent count overflow check failed"
+fi
+
+echo "Check for \$srcfile's extent count overflow"
+nextents=$(_xfs_get_fsxattr nextents $srcfile)
+
+if (( $nextents == $donor_nr_exts )); then
+       echo "\$srcfile: Extent count overflow check failed"
+fi
+
+# success, all done
+status=0
+exit

diff --git a/tests/xfs/615.out b/tests/xfs/615.out
new file mode 100644 (file)
index 0000000..ab9b31b
--- /dev/null
+++ b/tests/xfs/615.out
@@ -0,0 +1,13 @@
+QA output created by 615
+* Exchange extent forks
+Format and mount fs
+Create $donorfile having an extent of length 67 blocks
+Fragment $donorfile
+Create $srcfile having an extent of length 18 blocks
+Fragment $srcfile
+Collect $donorfile's extent count
+Collect $srcfile's extent count
+Inject reduce_max_iextents error tag
+Exchange $srcfile's and $donorfile's extent forks
+Check for $donorfile's extent count overflow
+Check for $srcfile's extent count overflow

diff --git a/tests/xfs/789 b/tests/xfs/789
index 00b98020f25e4190de4a0e3118098fad33091daf..e3a332d7cf03bb86a188bdc7f2b6aced99519f02 100755 (executable)
--- a/tests/xfs/789
+++ b/tests/xfs/789
@@ -7,7 +7,7 @@
 # Simple tests of the old xfs swapext ioctl
 
 . ./common/preamble
-_begin_fstest auto quick fiexchange swapext
+_begin_fstest auto quick swapext
 
 # Override the default cleanup function.
 _cleanup()