On Thu, Nov 25, 2021 at 03:47:47PM +0800, Shiyang Ruan wrote:
> Ensuring that copy on write in buffered mode works when the CoW
> range originally covers multiple extents, mixed with reflinked, unwritten,
> hole, regular and delalloc blocks.
>
> Signed-off-by: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx>
> ---
> common/reflink | 54 +++++++++++++++++++++++++++++++++++
> tests/generic/910 | 66 +++++++++++++++++++++++++++++++++++++++++++
> tests/generic/910.out | 12 ++++++++
> 3 files changed, 132 insertions(+)
> create mode 100755 tests/generic/910
> create mode 100644 tests/generic/910.out
>
> diff --git a/common/reflink b/common/reflink
> index e304ddc8..cca43393 100644
> --- a/common/reflink
> +++ b/common/reflink
> @@ -435,3 +435,57 @@ _sweave_reflink_holes_delalloc() {
> $FILEFRAG_PROG -v $sfile 2>&1 | grep -q delalloc || \
> _notrun "test requires delayed allocation writes"
> }
> +
> +# Create a file of interleaved holes, unwritten blocks, regular blocks, and
> +# reflinked blocks
> +_sweave_reflink_rainbow() {
> + blksz=$1
> + nr=$2
> + sfile=$3
> + dfile=$4
> +
> + $XFS_IO_PROG -f -c "truncate $((blksz * nr))" $sfile
> + _pwrite_byte 0x00 0 $((blksz * nr)) $sfile.chk
> + _pwrite_byte 0x61 0 $((blksz * nr)) $dfile
> + seq 0 5 $((nr - 1)) | while read i; do
> + _pwrite_byte 0x61 $((blksz * i)) $blksz $sfile
> + _pwrite_byte 0x61 $((blksz * i)) $blksz $sfile.chk
> + done
> + # 0 blocks are reflinked
> + seq 0 5 $((nr - 1)) | while read i; do
> + _reflink_range $sfile $((blksz * i)) $dfile $((blksz * i)) $blksz
> + _pwrite_byte 0x61 $((blksz * i)) $blksz $sfile.chk
> + done
> + # 1 blocks are unwritten
> + seq 1 5 $((nr - 1)) | while read i; do
> + $XFS_IO_PROG -f -c "falloc $((blksz * i)) $blksz" $sfile
> + _pwrite_byte 0x00 $((blksz * i)) $blksz $sfile.chk
> + done
> + # 2 blocks are holes
> + seq 2 5 $((nr - 1)) | while read i; do
> + _pwrite_byte 0x00 $((blksz * i)) $blksz $sfile.chk
> + done
> + # 3 blocks are regular
> + seq 3 5 $((nr - 1)) | while read i; do
> + _pwrite_byte 0x71 $((blksz * i)) $blksz $sfile
> + _pwrite_byte 0x71 $((blksz * i)) $blksz $sfile.chk
> + done
> + # 4 blocks will be delalloc later
> +}
> +
> +# For a file created with _sweave_reflink_rainbow, fill the holes with delalloc
> +# extents
> +_sweave_reflink_rainbow_delalloc() {
> + blksz=$1
> + nr=$2
> + dfile=$3
> +
> + # 4 blocks are delalloc (do later)
> + seq 4 5 $((nr - 1)) | while read i; do
> + _pwrite_byte 0x62 $((blksz * i)) $blksz $sfile
> + _pwrite_byte 0x62 $((blksz * i)) $blksz $sfile.chk
> + done
> + # make sure we actually got delalloc block
> + $FILEFRAG_PROG -v $sfile 2>&1 | grep -q delalloc || \
> + _notrun "test requires delayed allocation writes"
> +}
> diff --git a/tests/generic/910 b/tests/generic/910
> new file mode 100755
> index 00000000..8d731a2d
> --- /dev/null
> +++ b/tests/generic/910
> @@ -0,0 +1,66 @@
> +#! /bin/bash
> +# SPDX-License-Identifier: GPL-2.0
> +#
> +# FS QA Test No. 910
> +#
> +# Ensuring that copy on write in buffered mode works when the CoW
> +# range originally covers multiple extents, mixed with reflinked, unwritten,
> +# hole, regular and delalloc blocks.
> +# - Create a file with the following repeating sequence of blocks:
> +# 1. reflinked
> +# 2. unwritten
> +# 3. hole
> +# 4. regular block
> +# 5. delalloc
> +# - CoW across the halfway mark, starting with the unwritten extent.
> +# - Check that the files are now different where we say they're different.
> +#
> +. ./common/preamble
> +_begin_fstest auto quick clone punch
> +
> +# Import common functions.
> +. ./common/filter
> +. ./common/reflink
> +
> +# real QA test starts here
> +_require_scratch_reflink
> +_require_xfs_io_command "falloc"
> +_require_xfs_io_command "fpunch"
> +_require_cp_reflink
> +_require_odirect
I don't see any directio going on in this test?
This test is also missing a _require_command $FILEFRAG_PROG filefrag.
The rest looks good. Thanks for adding a new combination!
--D
> +
> +echo "Format and mount"
> +_scratch_mkfs > $seqres.full 2>&1
> +_scratch_mount >> $seqres.full 2>&1
> +
> +testdir=$SCRATCH_MNT/test-$seq
> +mkdir $testdir
> +
> +echo "Create the original files"
> +blksz=65536
> +nr=64
> +filesize=$((blksz * nr))
> +_sweave_reflink_rainbow $blksz $nr $testdir/file1 $testdir/file3 >> $seqres.full
> +_scratch_cycle_mount
> +
> +echo "Compare files"
> +md5sum $testdir/file1 | _filter_scratch
> +md5sum $testdir/file3 | _filter_scratch
> +md5sum $testdir/file1.chk | _filter_scratch
> +
> +echo "CoW across the transition"
> +cowoff=$((filesize / 4))
> +cowsz=$((filesize / 2))
> +_sweave_reflink_rainbow_delalloc $blksz $nr $testdir/file1 >> $seqres.full
> +# now cow
> +$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $cowsz $cowoff $cowsz" $testdir/file1 >> $seqres.full
> +_pwrite_byte 0x63 $cowoff $cowsz $testdir/file1.chk >> $seqres.full
> +_scratch_cycle_mount
> +
> +echo "Compare files"
> +md5sum $testdir/file1 | _filter_scratch
> +md5sum $testdir/file3 | _filter_scratch
> +md5sum $testdir/file1.chk | _filter_scratch
> +
> +# success, all done
> +status=0
> diff --git a/tests/generic/910.out b/tests/generic/910.out
> new file mode 100644
> index 00000000..12956584
> --- /dev/null
> +++ b/tests/generic/910.out
> @@ -0,0 +1,12 @@
> +QA output created by 910
> +Format and mount
> +Create the original files
> +Compare files
> +6366fd359371414186688a0ef6988893 SCRATCH_MNT/test-910/file1
> +bdbcf02ee0aa977795a79d25fcfdccb1 SCRATCH_MNT/test-910/file3
> +6366fd359371414186688a0ef6988893 SCRATCH_MNT/test-910/file1.chk
> +CoW across the transition
> +Compare files
> +26aa3a0749b867ec58363c8539ee5471 SCRATCH_MNT/test-910/file1
> +bdbcf02ee0aa977795a79d25fcfdccb1 SCRATCH_MNT/test-910/file3
> +26aa3a0749b867ec58363c8539ee5471 SCRATCH_MNT/test-910/file1.chk
> --
> 2.34.0
>
>
>
