From: Eryu Guan <eguan@redhat.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: ocfs2-devel@oss.oracle.com, fstests@vger.kernel.org
Subject: Re: [PATCH 5/7] reflink: make error reporting consistent
Date: Mon, 12 Dec 2016 18:47:43 +0800 [thread overview]
Message-ID: <20161212104630.GB29149@eguan.usersys.redhat.com> (raw)
In-Reply-To: <148149319659.31093.17539238147279139276.stgit@birch.djwong.org>
On Sun, Dec 11, 2016 at 01:53:16PM -0800, Darrick J. Wong wrote:
> When we're using dm-error to simulate failed devices, we don't really
> know if the write or the fdatasync is going to receive the EIO. So,
> capture the entire output and just look for the word error instead
> of enshrining it in the golden output.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> tests/generic/265 | 6 +++++-
> tests/generic/265.out | 1 -
> tests/generic/266 | 5 ++++-
> tests/generic/266.out | 1 -
> tests/generic/267 | 2 +-
> tests/generic/268 | 5 ++++-
> tests/generic/268.out | 1 -
> tests/generic/271 | 5 +++--
> tests/generic/271.out | 1 -
> tests/generic/272 | 5 +++--
> tests/generic/272.out | 1 -
> tests/generic/276 | 2 +-
> tests/generic/276.out | 1 -
> tests/generic/278 | 5 +++--
> tests/generic/278.out | 1 -
> tests/generic/279 | 4 ++--
> tests/generic/279.out | 1 -
> tests/generic/281 | 4 ++--
> tests/generic/281.out | 1 -
> tests/generic/283 | 6 ++++--
> tests/generic/283.out | 1 -
> 21 files changed, 32 insertions(+), 27 deletions(-)
>
>
> diff --git a/tests/generic/265 b/tests/generic/265
> index 8e9d5bc..ceddfbe 100755
> --- a/tests/generic/265
> +++ b/tests/generic/265
> @@ -80,7 +80,11 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" -c "fdatasync" $testdir/file2 >> $seqres.full
> +urk=$($XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" \
> + -c "fdatasync" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "pwrite did not fail"
> +
> _dmerror_load_working_table
> _dmerror_unmount
> _dmerror_mount
> diff --git a/tests/generic/265.out b/tests/generic/265.out
> index 1b67114..31eb4e9 100644
> --- a/tests/generic/265.out
> +++ b/tests/generic/265.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-265/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-265/file2
> CoW and unmount
> -fdatasync: Input/output error
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-265/file1
> Check for damage
> diff --git a/tests/generic/266 b/tests/generic/266
> index 4f9816a..09541c8 100755
> --- a/tests/generic/266
> +++ b/tests/generic/266
> @@ -80,7 +80,10 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" -c "fdatasync" $testdir/file2 >> $seqres.full
> +urk=$($XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" \
> + -c "fdatasync" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "pwrite did not fail"
>
> echo "Clean up the mess"
> _dmerror_unmount
> diff --git a/tests/generic/266.out b/tests/generic/266.out
> index dd34ad3..1641654 100644
> --- a/tests/generic/266.out
> +++ b/tests/generic/266.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-266/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-266/file2
> CoW and unmount
> -fdatasync: Input/output error
> Clean up the mess
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-266/file1
> diff --git a/tests/generic/267 b/tests/generic/267
> index e3a6b0c..2ec6ad9 100755
> --- a/tests/generic/267
> +++ b/tests/generic/267
> @@ -80,7 +80,7 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 >> $seqres.full
> +$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 >> $seqres.full 2>&1
> _dmerror_load_working_table
> rm -rf $testdir/file2 >> $seqres.full 2>&1
> _dmerror_unmount
> diff --git a/tests/generic/268 b/tests/generic/268
> index c7dcd57..b7d16ab 100755
> --- a/tests/generic/268
> +++ b/tests/generic/268
> @@ -81,7 +81,10 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" -c "fdatasync" $testdir/file2 >> $seqres.full
> +urk=$($XFS_IO_PROG -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" \
> + -c "fdatasync" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "pwrite did not fail"
> _dmerror_load_working_table
>
> echo "Rewrite"
> diff --git a/tests/generic/268.out b/tests/generic/268.out
> index 234e8be..1a3b39a 100644
> --- a/tests/generic/268.out
> +++ b/tests/generic/268.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-268/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-268/file2
> CoW and unmount
> -fdatasync: Input/output error
> Rewrite
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-268/file1
> diff --git a/tests/generic/271 b/tests/generic/271
> index ded8854..9439327 100755
> --- a/tests/generic/271
> +++ b/tests/generic/271
> @@ -81,8 +81,9 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 \
> - 2>&1 >> $seqres.full | _filter_xfs_io_error
> +urk=$($XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "dio pwrite did not fail"
> _dmerror_load_working_table
> _dmerror_unmount
> _dmerror_mount
> diff --git a/tests/generic/271.out b/tests/generic/271.out
> index 54d5b0d..9b44ee7 100644
> --- a/tests/generic/271.out
> +++ b/tests/generic/271.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-271/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-271/file2
> CoW and unmount
> -pwrite: Input/output error
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-271/file1
> Check for damage
> diff --git a/tests/generic/272 b/tests/generic/272
> index 5bc5b39..243adf5 100755
> --- a/tests/generic/272
> +++ b/tests/generic/272
> @@ -81,8 +81,9 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 \
> - 2>&1 >> $seqres.full | _filter_xfs_io_error
> +urk=$($XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "dio pwrite did not fail"
>
> echo "Clean up the mess"
> _dmerror_unmount
> diff --git a/tests/generic/272.out b/tests/generic/272.out
> index 0b8bdca..10ecda9 100644
> --- a/tests/generic/272.out
> +++ b/tests/generic/272.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-272/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-272/file2
> CoW and unmount
> -pwrite: Input/output error
> Clean up the mess
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-272/file1
> diff --git a/tests/generic/276 b/tests/generic/276
> index 83b902e..34b0029 100755
> --- a/tests/generic/276
> +++ b/tests/generic/276
> @@ -82,7 +82,7 @@ echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> $XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 \
> - 2>&1 >> $seqres.full | _filter_xfs_io_error
> + >> $seqres.full 2>&1
This seems different, the error is ignored after this update and test
passes even if there's no error happened. Is this intentional?
The same question applies to generic/279 and generic/281.
Thanks,
Eryu
> _dmerror_load_working_table
> rm -rf $testdir/file2 >> $seqres.full 2>&1
> _dmerror_unmount
> diff --git a/tests/generic/276.out b/tests/generic/276.out
> index 88a0162..a080dd0 100644
> --- a/tests/generic/276.out
> +++ b/tests/generic/276.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-276/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-276/file2
> CoW and unmount
> -pwrite: Input/output error
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-276/file1
> Check for damage
> diff --git a/tests/generic/278 b/tests/generic/278
> index 415742a..d751f01 100755
> --- a/tests/generic/278
> +++ b/tests/generic/278
> @@ -82,8 +82,9 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -$XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 \
> - 2>&1 >> $seqres.full | _filter_xfs_io_error
> +urk=$($XFS_IO_PROG -d -f -c "pwrite -S 0x63 -b $bufsize 0 $filesize" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "dio pwrite did not fail"
> _dmerror_load_working_table
>
> echo "Rewrite"
> diff --git a/tests/generic/278.out b/tests/generic/278.out
> index 9ead4ac..cd22fcd 100644
> --- a/tests/generic/278.out
> +++ b/tests/generic/278.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-278/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-278/file2
> CoW and unmount
> -pwrite: Input/output error
> Rewrite
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-278/file1
> diff --git a/tests/generic/279 b/tests/generic/279
> index 2f1ec4f..4541de3 100755
> --- a/tests/generic/279
> +++ b/tests/generic/279
> @@ -80,8 +80,8 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -urk=$($XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" -c "msync -s 0 $filesize" $testdir/file2 > $TEST_DIR/mwrite.out 2>&1)
> -cat $TEST_DIR/mwrite.out | tee -a $seqres.full
> +$XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" \
> + -c "msync -s 0 $filesize" $testdir/file2 >> $seqres.full 2>&1
> _dmerror_load_working_table
> _dmerror_unmount
> _dmerror_mount
> diff --git a/tests/generic/279.out b/tests/generic/279.out
> index f34c2b2..009f05e 100644
> --- a/tests/generic/279.out
> +++ b/tests/generic/279.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-279/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-279/file2
> CoW and unmount
> -msync: Input/output error
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-279/file1
> Check for damage
> diff --git a/tests/generic/281 b/tests/generic/281
> index c95acf2..e8e1661 100755
> --- a/tests/generic/281
> +++ b/tests/generic/281
> @@ -80,8 +80,8 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -urk=$($XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" -c "msync -s 0 $filesize" $testdir/file2 > $TEST_DIR/mwrite.out 2>&1)
> -cat $TEST_DIR/mwrite.out | tee -a $seqres.full
> +$XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" \
> + -c "msync -s 0 $filesize" $testdir/file2 >> $seqres.full 2>&1
>
> echo "Clean up the mess"
> _dmerror_unmount
> diff --git a/tests/generic/281.out b/tests/generic/281.out
> index a2d8e3f..1bae187 100644
> --- a/tests/generic/281.out
> +++ b/tests/generic/281.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-281/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-281/file2
> CoW and unmount
> -msync: Input/output error
> Clean up the mess
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-281/file1
> diff --git a/tests/generic/283 b/tests/generic/283
> index 8d56ac3..7bfdb4d 100755
> --- a/tests/generic/283
> +++ b/tests/generic/283
> @@ -81,8 +81,10 @@ md5sum $testdir/file2 | _filter_scratch
> echo "CoW and unmount"
> sync
> _dmerror_load_error_table
> -urk=$($XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" -c "msync -s 0 $filesize" $testdir/file2 > $TEST_DIR/mwrite.out 2>&1)
> -cat $TEST_DIR/mwrite.out | tee -a $seqres.full
> +urk=$($XFS_IO_PROG -f -c "mmap -rw 0 $filesize" -c "mwrite -S 0x63 0 $filesize" \
> + -c "msync -s 0 $filesize" $testdir/file2 2>&1)
> +echo $urk >> $seqres.full
> +echo "$urk" | grep -q "error" || _fail "mwrite did not fail"
> _dmerror_load_working_table
>
> echo "Rewrite"
> diff --git a/tests/generic/283.out b/tests/generic/283.out
> index f9fd5c8..b475054 100644
> --- a/tests/generic/283.out
> +++ b/tests/generic/283.out
> @@ -5,7 +5,6 @@ Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-283/file1
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-283/file2
> CoW and unmount
> -msync: Input/output error
> Rewrite
> Compare files
> 1886e67cf8783e89ce6ddc5bb09a3944 SCRATCH_MNT/test-283/file1
>
next prev parent reply other threads:[~2016-12-12 10:47 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-11 21:52 [PATCH 0/7] xfstests: misc reflink test fixes Darrick J. Wong
2016-12-11 21:52 ` [PATCH 1/7] ocfs2: test reflinking to inline data files Darrick J. Wong
2016-12-12 9:01 ` Eryu Guan
2016-12-12 18:09 ` Darrick J. Wong
2016-12-13 3:20 ` Eryu Guan
2016-12-13 7:11 ` Darrick J. Wong
2016-12-13 21:35 ` Darrick J. Wong
2016-12-14 7:32 ` Eric Ren
2016-12-11 21:52 ` [PATCH 2/7] ocfs2/reflink: fix file block size reporting Darrick J. Wong
2016-12-12 9:48 ` Eryu Guan
2016-12-12 23:08 ` Darrick J. Wong
2016-12-11 21:53 ` [PATCH 3/7] reflink: fix quota tests to work properly Darrick J. Wong
2016-12-12 10:06 ` Eryu Guan
2016-12-12 23:08 ` Darrick J. Wong
2016-12-11 21:53 ` [PATCH 4/7] reflink: fix space consumption tests Darrick J. Wong
2016-12-12 10:25 ` Eryu Guan
2016-12-12 23:03 ` Darrick J. Wong
2016-12-13 3:22 ` Eryu Guan
2016-12-11 21:53 ` [PATCH 5/7] reflink: make error reporting consistent Darrick J. Wong
2016-12-12 10:47 ` Eryu Guan [this message]
2016-12-12 23:06 ` Darrick J. Wong
2016-12-11 21:53 ` [PATCH 6/7] reflink: don't test disjoint block sharing sets Darrick J. Wong
2016-12-11 21:53 ` [PATCH 7/7] xfs/ext4: check negative inode size Darrick J. Wong
2016-12-12 11:07 ` Eryu Guan
2016-12-13 21:49 ` Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161212104630.GB29149@eguan.usersys.redhat.com \
--to=eguan@redhat.com \
--cc=darrick.wong@oracle.com \
--cc=fstests@vger.kernel.org \
--cc=ocfs2-devel@oss.oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox