Ensuring that copy on write in mmap mode works when the CoW range originally covers multiple extents, some delalloc, some not. (MMAP version of generic/195,194) Also, add a check to make sure the delalloc block is accutally exist. Signed-off-by: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx> --- common/reflink | 3 +++ tests/generic/904 | 61 +++++++++++++++++++++++++++++++++++++++++++ tests/generic/904.out | 12 +++++++++ 3 files changed, 76 insertions(+) create mode 100755 tests/generic/904 create mode 100644 tests/generic/904.out diff --git a/common/reflink b/common/reflink index ee60398e..38210673 100644 --- a/common/reflink +++ b/common/reflink @@ -296,6 +296,9 @@ _weave_reflink_holes_delalloc() { _pwrite_byte 0x62 $((blksz * i)) $blksz $dfile _pwrite_byte 0x62 $((blksz * i)) $blksz $dfile.chk done + # make sure we actually got delalloc block + $FILEFRAG_PROG -v $dfile 2>&1 | grep -q delalloc || \ + _notrun "test requires delayed allocation writes" } # Create a file of interleaved regular blocks and reflinked blocks diff --git a/tests/generic/904 b/tests/generic/904 new file mode 100755 index 00000000..69703d70 --- /dev/null +++ b/tests/generic/904 @@ -0,0 +1,61 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# +# FS QA Test No. 904 +# +# Ensuring that copy on write in mmap mode works when the CoW +# range originally covers multiple extents, some delalloc, some not. +# (MMAP version of generic/195,194) +# - Create a file. +# - Reflink the odd blocks of the first file into the second file. +# - Buffered write the even blocks of the second file. +# - mmap 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 + +# Import common functions. +. ./common/filter +. ./common/reflink + +# real QA test starts here +_require_scratch_reflink +_require_xfs_io_command "falloc" + +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)) +_weave_reflink_holes $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/file3.chk | _filter_scratch + +echo "mmap CoW across the transition" +cowoff=$((filesize / 4)) +cowsz=$((filesize / 2)) +_weave_reflink_holes_delalloc $blksz $nr $testdir/file3 >> $seqres.full +mmapsz=$((cowoff + cowsz)) +_mwrite_byte 0x63 $cowoff $cowsz $mmapsz $testdir/file3 >> $seqres.full +_mwrite_byte 0x63 $cowoff $cowsz $mmapsz $testdir/file3.chk >> $seqres.full +_scratch_cycle_mount + +echo "Compare files" +md5sum $testdir/file1 | _filter_scratch +md5sum $testdir/file3 | _filter_scratch +md5sum $testdir/file3.chk | _filter_scratch + +# success, all done +status=0 +exit diff --git a/tests/generic/904.out b/tests/generic/904.out new file mode 100644 index 00000000..cc993394 --- /dev/null +++ b/tests/generic/904.out @@ -0,0 +1,12 @@ +QA output created by 904 +Format and mount +Create the original files +Compare files +bdbcf02ee0aa977795a79d25fcfdccb1 SCRATCH_MNT/test-904/file1 +fa50dba51826899c372464a153cb2117 SCRATCH_MNT/test-904/file3 +fa50dba51826899c372464a153cb2117 SCRATCH_MNT/test-904/file3.chk +mmap CoW across the transition +Compare files +bdbcf02ee0aa977795a79d25fcfdccb1 SCRATCH_MNT/test-904/file1 +55968a95dfc0120df4e8485576514320 SCRATCH_MNT/test-904/file3 +55968a95dfc0120df4e8485576514320 SCRATCH_MNT/test-904/file3.chk -- 2.34.0