在 2021/11/15 11:48, Eryu Guan 写道:
On Mon, Nov 15, 2021 at 11:41:06AM +0800, Shiyang Ruan wrote:Test for races or FS corruption between reflink and mmap reading the target file. (MMAP version of generic/164,165) Signed-off-by: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx> --- common/reflink | 11 +++++++ tests/generic/930 | 72 +++++++++++++++++++++++++++++++++++++++++++ tests/generic/930.out | 5 +++ 3 files changed, 88 insertions(+) create mode 100755 tests/generic/930 create mode 100644 tests/generic/930.out diff --git a/common/reflink b/common/reflink index 85b586cb..215a8a48 100644 --- a/common/reflink +++ b/common/reflink @@ -186,6 +186,17 @@ _read_range() { $XFS_IO_PROG $xfs_io_args -f -c "pread -q -v $offset $len" "$file" | cut -d ' ' -f '3-18' }+# Prints a range of a file as a hex dump+_mread_range() { + file="$1" + offset="$2" + len="$3" + xfs_io_args="$4" + + $XFS_IO_PROG $xfs_io_args -f -c "mmap -rw 0 $((offset + len))" \ + -c "mread -v $offset $len" "$file" | cut -d ' ' -f '3-18' +} + # Compare ranges of two files _compare_range() { file1="$1" diff --git a/tests/generic/930 b/tests/generic/930 new file mode 100755 index 00000000..c259ae45 --- /dev/null +++ b/tests/generic/930 @@ -0,0 +1,72 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# +# FS QA Test No. xxxThe test number in test should be 930 as well, otherwise I have to manually edit it after applying & before running ./tools/mvtest, as 'xxx' won't be matched & replaced by the mvtest script. The same applies to other tests as well.
Ok. I didn't know that... I'll fix it. -- Thanks, Ruan.
Thanks, Eryu+# +# Test for races or FS corruption between reflink and mmap reading the +# target file. (MMAP version of generic/164,165) +# +. ./common/preamble +_begin_fstest auto clone + +_register_cleanup "_cleanup" BUS + +# Import common functions. +. ./common/filter +. ./common/reflink + +# real QA test starts here +_require_scratch_reflink +_require_cp_reflink + +echo "Format and mount" +_scratch_mkfs > $seqres.full 2>&1 +_scratch_mount >> $seqres.full 2>&1 + +testdir=$SCRATCH_MNT/test-$seq +finished_file=/tmp/finished +rm -rf $finished_file +mkdir $testdir + +loops=512 +nr_loops=$((loops - 1)) +blksz=65536 + +echo "Initialize files" +echo >> $seqres.full +_pwrite_byte 0x61 0 $((loops * blksz)) $testdir/file1 >> $seqres.full +_pwrite_byte 0x62 0 $((loops * blksz)) $testdir/file2 >> $seqres.full +_cp_reflink $testdir/file1 $testdir/file3 +_scratch_cycle_mount + +fbytes() { + egrep -v '(61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61|62 62 62 62 62 62 62 62 62 62 62 62 62 62 62 62)' +} + +reader() { + while [ ! -e $finished_file ]; do + _mread_range $testdir/file3 0 $((loops * blksz)) | fbytes + done +} + +echo "Reflink and mmap reread the files!" +reader & +for i in `seq 1 2`; do + seq $nr_loops -1 0 | while read i; do + _reflink_range $testdir/file1 $((i * blksz)) \ + $testdir/file3 $((i * blksz)) $blksz >> $seqres.full + [ $? -ne 0 ] && break + done + seq $nr_loops -1 0 | while read i; do + _reflink_range $testdir/file2 $((i * blksz)) \ + $testdir/file3 $((i * blksz)) $blksz >> $seqres.full + [ $? -ne 0 ] && break + done +done +echo "Finished reflinking" +touch $finished_file +wait + +# success, all done +status=0 +exit diff --git a/tests/generic/930.out b/tests/generic/930.out new file mode 100644 index 00000000..f8e0a1a3 --- /dev/null +++ b/tests/generic/930.out @@ -0,0 +1,5 @@ +QA output created by 930 +Format and mount +Initialize files +Reflink and mmap reread the files! +Finished reflinking -- 2.33.0
