| Message ID | 20220223033751.97913-2-catherine.hoang@oracle.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | xfstests: add log attribute replay test | expand |
On 2/22/22 8:37 PM, Catherine Hoang wrote: > From: Allison Henderson <allison.henderson@oracle.com> > > This patch adds tests to exercise the log attribute error > inject and log replay. These tests aim to cover cases where attributes > are added, removed, and overwritten in each format (shortform, leaf, > node). Error inject is used to replay these operations from the log. > > Signed-off-by: Allison Henderson <allison.henderson@oracle.com> > Signed-off-by: Catherine Hoang <catherine.hoang@oracle.com> This looks good to me, though I can't give an rvb as a co-author. We'll need to get the kernel/user space features merged before the test can run though. It seems Darrick has found a bug in generic/467, so we'll need to get that weeded out before the rest can follow. We'll chat more in our meeting today. In the meantime, I encourgage reviews for the testcase since we will eventually need it. Thanks all! Allison > --- > tests/xfs/543 | 176 ++++++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/543.out | 149 +++++++++++++++++++++++++++++++++++++++ > 2 files changed, 325 insertions(+) > create mode 100755 tests/xfs/543 > create mode 100644 tests/xfs/543.out > > diff --git a/tests/xfs/543 b/tests/xfs/543 > new file mode 100755 > index 00000000..06f16f21 > --- /dev/null > +++ b/tests/xfs/543 > @@ -0,0 +1,176 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0 > +# Copyright (c) 2022, Oracle and/or its affiliates. All Rights Reserved. > +# > +# FS QA Test 543 > +# > +# Log attribute replay test > +# > +. ./common/preamble > +_begin_fstest auto quick attr > + > +# get standard environment, filters and checks > +. ./common/filter > +. ./common/attr > +. ./common/inject > + > +_cleanup() > +{ > + rm -rf $tmp.* $testdir > + test -w /sys/fs/xfs/debug/larp && \ > + echo 0 > /sys/fs/xfs/debug/larp > +} > + > +test_attr_replay() > +{ > + testfile=$testdir/$1 > + attr_name=$2 > + attr_value=$3 > + flag=$4 > + error_tag=$5 > + > + # Inject error > + _scratch_inject_error $error_tag > + > + # Set attribute > + echo "$attr_value" | ${ATTR_PROG} -$flag "$attr_name" $testfile 2>&1 | \ > + _filter_scratch > + > + # FS should be shut down, touch will fail > + touch $testfile 2>&1 | _filter_scratch > + > + # Remount to replay log > + _scratch_remount_dump_log >> $seqres.full > + > + # FS should be online, touch should succeed > + touch $testfile > + > + # Verify attr recovery > + { $ATTR_PROG -g $attr_name $testfile | md5sum; } 2>&1 | _filter_scratch > + > + echo "" > +} > + > +create_test_file() > +{ > + filename=$testdir/$1 > + count=$2 > + attr_value=$3 > + > + touch $filename > + > + for i in `seq $count` > + do > + $ATTR_PROG -s "attr_name$i" -V $attr_value $filename >> \ > + $seqres.full > + done > +} > + > +# real QA test starts here > +_supported_fs xfs > + > +_require_scratch > +_require_attrs > +_require_xfs_io_error_injection "larp" > +_require_xfs_io_error_injection "da_leaf_split" > +_require_xfs_io_error_injection "attr_leaf_to_node" > +_require_xfs_sysfs debug/larp > +test -w /sys/fs/xfs/debug/larp || _notrun "larp knob not writable" > + > +# turn on log attributes > +echo 1 > /sys/fs/xfs/debug/larp > + > +attr16="0123456789ABCDEF" > +attr64="$attr16$attr16$attr16$attr16" > +attr256="$attr64$attr64$attr64$attr64" > +attr1k="$attr256$attr256$attr256$attr256" > +attr4k="$attr1k$attr1k$attr1k$attr1k" > +attr8k="$attr4k$attr4k" > +attr16k="$attr8k$attr8k" > +attr32k="$attr16k$attr16k" > +attr64k="$attr32k$attr32k" > + > +echo "*** mkfs" > +_scratch_mkfs >/dev/null > + > +echo "*** mount FS" > +_scratch_mount > + > +testdir=$SCRATCH_MNT/testdir > +mkdir $testdir > + > +# empty, inline > +create_test_file empty_file1 0 > +test_attr_replay empty_file1 "attr_name" $attr64 "s" "larp" > +test_attr_replay empty_file1 "attr_name" $attr64 "r" "larp" > + > +# empty, internal > +create_test_file empty_file2 0 > +test_attr_replay empty_file2 "attr_name" $attr1k "s" "larp" > +test_attr_replay empty_file2 "attr_name" $attr1k "r" "larp" > + > +# empty, remote > +create_test_file empty_file3 0 > +test_attr_replay empty_file3 "attr_name" $attr64k "s" "larp" > +test_attr_replay empty_file3 "attr_name" $attr64k "r" "larp" > + > +# inline, inline > +create_test_file inline_file1 1 $attr16 > +test_attr_replay inline_file1 "attr_name2" $attr64 "s" "larp" > +test_attr_replay inline_file1 "attr_name2" $attr64 "r" "larp" > + > +# inline, internal > +create_test_file inline_file2 1 $attr16 > +test_attr_replay inline_file2 "attr_name2" $attr1k "s" "larp" > +test_attr_replay inline_file2 "attr_name2" $attr1k "r" "larp" > + > +# inline, remote > +create_test_file inline_file3 1 $attr16 > +test_attr_replay inline_file3 "attr_name2" $attr64k "s" "larp" > +test_attr_replay inline_file3 "attr_name2" $attr64k "r" "larp" > + > +# extent, internal > +create_test_file extent_file1 1 $attr1k > +test_attr_replay extent_file1 "attr_name2" $attr1k "s" "larp" > +test_attr_replay extent_file1 "attr_name2" $attr1k "r" "larp" > + > +# extent, inject error on split > +create_test_file extent_file2 3 $attr1k > +test_attr_replay extent_file2 "attr_name4" $attr1k "s" "da_leaf_split" > + > +# extent, inject error on fork transition > +create_test_file extent_file3 3 $attr1k > +test_attr_replay extent_file3 "attr_name4" $attr1k "s" "attr_leaf_to_node" > + > +# extent, remote > +create_test_file extent_file4 1 $attr1k > +test_attr_replay extent_file4 "attr_name2" $attr64k "s" "larp" > +test_attr_replay extent_file4 "attr_name2" $attr64k "r" "larp" > + > +# remote, internal > +create_test_file remote_file1 1 $attr64k > +test_attr_replay remote_file1 "attr_name2" $attr1k "s" "larp" > +test_attr_replay remote_file1 "attr_name2" $attr1k "r" "larp" > + > +# remote, remote > +create_test_file remote_file2 1 $attr64k > +test_attr_replay remote_file2 "attr_name2" $attr64k "s" "larp" > +test_attr_replay remote_file2 "attr_name2" $attr64k "r" "larp" > + > +# replace shortform > +create_test_file sf_file 2 $attr64 > +test_attr_replay sf_file "attr_name2" $attr64 "s" "larp" > + > +# replace leaf > +create_test_file leaf_file 2 $attr1k > +test_attr_replay leaf_file "attr_name2" $attr1k "s" "larp" > + > +# replace node > +create_test_file node_file 1 $attr64k > +$ATTR_PROG -s "attr_name2" -V $attr1k $testdir/node_file \ > + >> $seqres.full > +test_attr_replay node_file "attr_name2" $attr1k "s" "larp" > + > +echo "*** done" > +status=0 > +exit > diff --git a/tests/xfs/543.out b/tests/xfs/543.out > new file mode 100644 > index 00000000..1c74e795 > --- /dev/null > +++ b/tests/xfs/543.out > @@ -0,0 +1,149 @@ > +QA output created by 543 > +*** mkfs > +*** mount FS > +attr_set: Input/output error > +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error > +21d850f99c43cc13abbe34838a8a3c8a - > + > +attr_remove: Input/output error > +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error > +attr_get: No data available > +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file2': Input/output error > +2ff89c2935debc431745ec791be5421a - > + > +attr_remove: Input/output error > +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file2': Input/output error > +attr_get: No data available > +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file2 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file3 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file3': Input/output error > +5d24b314242c52176c98ac4bd685da8b - > + > +attr_remove: Input/output error > +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file3 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file3': Input/output error > +attr_get: No data available > +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file3 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file1': Input/output error > +5a7b559a70d8e92b4f3c6f7158eead08 - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file1': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file1 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file2': Input/output error > +5717d5e66c70be6bdb00ecbaca0b7749 - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file2': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file2 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file3 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file3': Input/output error > +5c929964efd1b243aa8cceb6524f4810 - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file3 > +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file3': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file3 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/extent_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file1': Input/output error > +51ccb5cdfc9082060f0f94a8a108fea0 - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/extent_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file1': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/extent_file1 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name4" for SCRATCH_MNT/testdir/extent_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file2': Input/output error > +8d530bbe852d8bca83b131d5b3e497f5 - > + > +attr_set: Input/output error > +Could not set "attr_name4" for SCRATCH_MNT/testdir/extent_file3 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file3': Input/output error > +5d77c4d3831a35bcbbd6e7677119ce9a - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/extent_file4 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file4': Input/output error > +6707ec2431e4dbea20e17da0816520bb - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/extent_file4 > +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file4': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/extent_file4 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/remote_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file1': Input/output error > +767ebca3e4a6d24170857364f2bf2a3c - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/remote_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file1': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/remote_file1 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/remote_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file2': Input/output error > +fd84ddec89237e6d34a1703639efaebf - > + > +attr_remove: Input/output error > +Could not remove "attr_name2" for SCRATCH_MNT/testdir/remote_file2 > +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file2': Input/output error > +attr_get: No data available > +Could not get "attr_name2" for SCRATCH_MNT/testdir/remote_file2 > +d41d8cd98f00b204e9800998ecf8427e - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/sf_file > +touch: cannot touch 'SCRATCH_MNT/testdir/sf_file': Input/output error > +34aaa49662bafb46c76e377454685071 - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/leaf_file > +touch: cannot touch 'SCRATCH_MNT/testdir/leaf_file': Input/output error > +664e95ec28830ffb367c0950026e0d21 - > + > +attr_set: Input/output error > +Could not set "attr_name2" for SCRATCH_MNT/testdir/node_file > +touch: cannot touch 'SCRATCH_MNT/testdir/node_file': Input/output error > +bb37a78ce26472eeb711e3559933db42 - > + > +*** done
On Wed, Feb 23, 2022 at 03:37:51AM +0000, Catherine Hoang wrote: > From: Allison Henderson <allison.henderson@oracle.com> > > This patch adds tests to exercise the log attribute error > inject and log replay. These tests aim to cover cases where attributes > are added, removed, and overwritten in each format (shortform, leaf, > node). Error inject is used to replay these operations from the log. > > Signed-off-by: Allison Henderson <allison.henderson@oracle.com> > Signed-off-by: Catherine Hoang <catherine.hoang@oracle.com> > --- > tests/xfs/543 | 176 ++++++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/543.out | 149 +++++++++++++++++++++++++++++++++++++++ > 2 files changed, 325 insertions(+) > create mode 100755 tests/xfs/543 > create mode 100644 tests/xfs/543.out > > diff --git a/tests/xfs/543 b/tests/xfs/543 > new file mode 100755 > index 00000000..06f16f21 > --- /dev/null > +++ b/tests/xfs/543 > @@ -0,0 +1,176 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0 > +# Copyright (c) 2022, Oracle and/or its affiliates. All Rights Reserved. > +# > +# FS QA Test 543 > +# > +# Log attribute replay test > +# > +. ./common/preamble > +_begin_fstest auto quick attr > + > +# get standard environment, filters and checks > +. ./common/filter > +. ./common/attr > +. ./common/inject > + > +_cleanup() > +{ > + rm -rf $tmp.* $testdir > + test -w /sys/fs/xfs/debug/larp && \ > + echo 0 > /sys/fs/xfs/debug/larp This is problematic - I set larp=1 before I start running fstests so all tests exercise the LARP paths. This will turn off LARP unconditionally, so after this runs nothing will exercise the LARP paths, even if that's what I want the tests to do. Also, this does not replicate what the generic _cleanup() function does. Also, no need to remove $testdir - it's on the scratch device. Also, don't call things "testdir" because "test directory" has specific meaning - i.e. the *test device mount* - and this is too easy to confuse when reading the test code. > +} > + > +test_attr_replay() > +{ > + testfile=$testdir/$1 > + attr_name=$2 > + attr_value=$3 > + flag=$4 > + error_tag=$5 > + > + # Inject error > + _scratch_inject_error $error_tag > + > + # Set attribute > + echo "$attr_value" | ${ATTR_PROG} -$flag "$attr_name" $testfile 2>&1 | \ > + _filter_scratch > + > + # FS should be shut down, touch will fail > + touch $testfile 2>&1 | _filter_scratch > + > + # Remount to replay log > + _scratch_remount_dump_log >> $seqres.full > + > + # FS should be online, touch should succeed > + touch $testfile > + > + # Verify attr recovery > + { $ATTR_PROG -g $attr_name $testfile | md5sum; } 2>&1 | _filter_scratch > + > + echo "" > +} > + > +create_test_file() > +{ > + filename=$testdir/$1 > + count=$2 > + attr_value=$3 > + > + touch $filename > + > + for i in `seq $count` > + do > + $ATTR_PROG -s "attr_name$i" -V $attr_value $filename >> \ > + $seqres.full > + done > +} > + > +# real QA test starts here > +_supported_fs xfs > + > +_require_scratch > +_require_attrs > +_require_xfs_io_error_injection "larp" > +_require_xfs_io_error_injection "da_leaf_split" > +_require_xfs_io_error_injection "attr_leaf_to_node" > +_require_xfs_sysfs debug/larp These go first, before any code. > +test -w /sys/fs/xfs/debug/larp || _notrun "larp knob not writable" When is the sysfs not writeable? > +# turn on log attributes > +echo 1 > /sys/fs/xfs/debug/larp Needs to store the previous value so it can be restored at cleanup. > +attr16="0123456789ABCDEF" > +attr64="$attr16$attr16$attr16$attr16" > +attr256="$attr64$attr64$attr64$attr64" > +attr1k="$attr256$attr256$attr256$attr256" > +attr4k="$attr1k$attr1k$attr1k$attr1k" > +attr8k="$attr4k$attr4k" > +attr16k="$attr8k$attr8k" > +attr32k="$attr16k$attr16k" > +attr64k="$attr32k$attr32k" > + > +echo "*** mkfs" > +_scratch_mkfs >/dev/null > + > +echo "*** mount FS" > +_scratch_mount There's no need for echo lines like this in the golden output - it's obvious what failed from the diff... > +testdir=$SCRATCH_MNT/testdir > +mkdir $testdir Why not just use $SCRATCH_MNT directly? > + > +# empty, inline > +create_test_file empty_file1 0 > +test_attr_replay empty_file1 "attr_name" $attr64 "s" "larp" > +test_attr_replay empty_file1 "attr_name" $attr64 "r" "larp" > + > +# empty, internal > +create_test_file empty_file2 0 > +test_attr_replay empty_file2 "attr_name" $attr1k "s" "larp" > +test_attr_replay empty_file2 "attr_name" $attr1k "r" "larp" > + > +# empty, remote > +create_test_file empty_file3 0 > +test_attr_replay empty_file3 "attr_name" $attr64k "s" "larp" > +test_attr_replay empty_file3 "attr_name" $attr64k "r" "larp" single attr insert/remove. OK. > + > +# inline, inline > +create_test_file inline_file1 1 $attr16 > +test_attr_replay inline_file1 "attr_name2" $attr64 "s" "larp" > +test_attr_replay inline_file1 "attr_name2" $attr64 "r" "larp" ... Insert/remove of a second attr and the format conversions they cause. OK. Doesn't check that the first xattr is retained and uncorrupted anywhere. > +# replace shortform > +create_test_file sf_file 2 $attr64 > +test_attr_replay sf_file "attr_name2" $attr64 "s" "larp" This only replaces with same size. We also need coverage of replace with smaller size and larger size, as well as replace causing sf -> leaf and sf -> remote attr format conversions. > +# replace leaf > +create_test_file leaf_file 2 $attr1k > +test_attr_replay leaf_file "attr_name2" $attr1k "s" "larp" Need replace causing leaf -> sf (smaller) and well as leaf -> remote attr (larger). Also leaf w/ remote attr -> sf (much smaller!). > +# replace node > +create_test_file node_file 1 $attr64k > +$ATTR_PROG -s "attr_name2" -V $attr1k $testdir/node_file \ > + >> $seqres.full > +test_attr_replay node_file "attr_name2" $attr1k "s" "larp" Need node -> leaf, node -> sf and node w/ remote attr -> sf > --- /dev/null > +++ b/tests/xfs/543.out > @@ -0,0 +1,149 @@ > +QA output created by 543 > +*** mkfs > +*** mount FS > +attr_set: Input/output error > +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error We don't really need to capture the error injection errors, all we care about is that the recovered md5sum matches the expected md5sum: > +21d850f99c43cc13abbe34838a8a3c8a - That's the recovered md5sum, what was the expected md5sum of the original attr value that we stored? i.e. how do we validate taht we got the correct attr value? > + > +attr_remove: Input/output error > +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error > +attr_get: No data available > +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file1 > +d41d8cd98f00b204e9800998ecf8427e - Why do we have md5sum output for an attr that does not exist? We should be capturing the ENODATA error here, as that is the correct response after recovery of a remove operation. Cheers, Dave.
diff --git a/tests/xfs/543 b/tests/xfs/543 new file mode 100755 index 00000000..06f16f21 --- /dev/null +++ b/tests/xfs/543 @@ -0,0 +1,176 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2022, Oracle and/or its affiliates. All Rights Reserved. +# +# FS QA Test 543 +# +# Log attribute replay test +# +. ./common/preamble +_begin_fstest auto quick attr + +# get standard environment, filters and checks +. ./common/filter +. ./common/attr +. ./common/inject + +_cleanup() +{ + rm -rf $tmp.* $testdir + test -w /sys/fs/xfs/debug/larp && \ + echo 0 > /sys/fs/xfs/debug/larp +} + +test_attr_replay() +{ + testfile=$testdir/$1 + attr_name=$2 + attr_value=$3 + flag=$4 + error_tag=$5 + + # Inject error + _scratch_inject_error $error_tag + + # Set attribute + echo "$attr_value" | ${ATTR_PROG} -$flag "$attr_name" $testfile 2>&1 | \ + _filter_scratch + + # FS should be shut down, touch will fail + touch $testfile 2>&1 | _filter_scratch + + # Remount to replay log + _scratch_remount_dump_log >> $seqres.full + + # FS should be online, touch should succeed + touch $testfile + + # Verify attr recovery + { $ATTR_PROG -g $attr_name $testfile | md5sum; } 2>&1 | _filter_scratch + + echo "" +} + +create_test_file() +{ + filename=$testdir/$1 + count=$2 + attr_value=$3 + + touch $filename + + for i in `seq $count` + do + $ATTR_PROG -s "attr_name$i" -V $attr_value $filename >> \ + $seqres.full + done +} + +# real QA test starts here +_supported_fs xfs + +_require_scratch +_require_attrs +_require_xfs_io_error_injection "larp" +_require_xfs_io_error_injection "da_leaf_split" +_require_xfs_io_error_injection "attr_leaf_to_node" +_require_xfs_sysfs debug/larp +test -w /sys/fs/xfs/debug/larp || _notrun "larp knob not writable" + +# turn on log attributes +echo 1 > /sys/fs/xfs/debug/larp + +attr16="0123456789ABCDEF" +attr64="$attr16$attr16$attr16$attr16" +attr256="$attr64$attr64$attr64$attr64" +attr1k="$attr256$attr256$attr256$attr256" +attr4k="$attr1k$attr1k$attr1k$attr1k" +attr8k="$attr4k$attr4k" +attr16k="$attr8k$attr8k" +attr32k="$attr16k$attr16k" +attr64k="$attr32k$attr32k" + +echo "*** mkfs" +_scratch_mkfs >/dev/null + +echo "*** mount FS" +_scratch_mount + +testdir=$SCRATCH_MNT/testdir +mkdir $testdir + +# empty, inline +create_test_file empty_file1 0 +test_attr_replay empty_file1 "attr_name" $attr64 "s" "larp" +test_attr_replay empty_file1 "attr_name" $attr64 "r" "larp" + +# empty, internal +create_test_file empty_file2 0 +test_attr_replay empty_file2 "attr_name" $attr1k "s" "larp" +test_attr_replay empty_file2 "attr_name" $attr1k "r" "larp" + +# empty, remote +create_test_file empty_file3 0 +test_attr_replay empty_file3 "attr_name" $attr64k "s" "larp" +test_attr_replay empty_file3 "attr_name" $attr64k "r" "larp" + +# inline, inline +create_test_file inline_file1 1 $attr16 +test_attr_replay inline_file1 "attr_name2" $attr64 "s" "larp" +test_attr_replay inline_file1 "attr_name2" $attr64 "r" "larp" + +# inline, internal +create_test_file inline_file2 1 $attr16 +test_attr_replay inline_file2 "attr_name2" $attr1k "s" "larp" +test_attr_replay inline_file2 "attr_name2" $attr1k "r" "larp" + +# inline, remote +create_test_file inline_file3 1 $attr16 +test_attr_replay inline_file3 "attr_name2" $attr64k "s" "larp" +test_attr_replay inline_file3 "attr_name2" $attr64k "r" "larp" + +# extent, internal +create_test_file extent_file1 1 $attr1k +test_attr_replay extent_file1 "attr_name2" $attr1k "s" "larp" +test_attr_replay extent_file1 "attr_name2" $attr1k "r" "larp" + +# extent, inject error on split +create_test_file extent_file2 3 $attr1k +test_attr_replay extent_file2 "attr_name4" $attr1k "s" "da_leaf_split" + +# extent, inject error on fork transition +create_test_file extent_file3 3 $attr1k +test_attr_replay extent_file3 "attr_name4" $attr1k "s" "attr_leaf_to_node" + +# extent, remote +create_test_file extent_file4 1 $attr1k +test_attr_replay extent_file4 "attr_name2" $attr64k "s" "larp" +test_attr_replay extent_file4 "attr_name2" $attr64k "r" "larp" + +# remote, internal +create_test_file remote_file1 1 $attr64k +test_attr_replay remote_file1 "attr_name2" $attr1k "s" "larp" +test_attr_replay remote_file1 "attr_name2" $attr1k "r" "larp" + +# remote, remote +create_test_file remote_file2 1 $attr64k +test_attr_replay remote_file2 "attr_name2" $attr64k "s" "larp" +test_attr_replay remote_file2 "attr_name2" $attr64k "r" "larp" + +# replace shortform +create_test_file sf_file 2 $attr64 +test_attr_replay sf_file "attr_name2" $attr64 "s" "larp" + +# replace leaf +create_test_file leaf_file 2 $attr1k +test_attr_replay leaf_file "attr_name2" $attr1k "s" "larp" + +# replace node +create_test_file node_file 1 $attr64k +$ATTR_PROG -s "attr_name2" -V $attr1k $testdir/node_file \ + >> $seqres.full +test_attr_replay node_file "attr_name2" $attr1k "s" "larp" + +echo "*** done" +status=0 +exit diff --git a/tests/xfs/543.out b/tests/xfs/543.out new file mode 100644 index 00000000..1c74e795 --- /dev/null +++ b/tests/xfs/543.out @@ -0,0 +1,149 @@ +QA output created by 543 +*** mkfs +*** mount FS +attr_set: Input/output error +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error +21d850f99c43cc13abbe34838a8a3c8a - + +attr_remove: Input/output error +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file1': Input/output error +attr_get: No data available +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file1 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file2': Input/output error +2ff89c2935debc431745ec791be5421a - + +attr_remove: Input/output error +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file2': Input/output error +attr_get: No data available +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file2 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name" for SCRATCH_MNT/testdir/empty_file3 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file3': Input/output error +5d24b314242c52176c98ac4bd685da8b - + +attr_remove: Input/output error +Could not remove "attr_name" for SCRATCH_MNT/testdir/empty_file3 +touch: cannot touch 'SCRATCH_MNT/testdir/empty_file3': Input/output error +attr_get: No data available +Could not get "attr_name" for SCRATCH_MNT/testdir/empty_file3 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file1': Input/output error +5a7b559a70d8e92b4f3c6f7158eead08 - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file1': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file1 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file2': Input/output error +5717d5e66c70be6bdb00ecbaca0b7749 - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file2': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file2 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/inline_file3 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file3': Input/output error +5c929964efd1b243aa8cceb6524f4810 - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/inline_file3 +touch: cannot touch 'SCRATCH_MNT/testdir/inline_file3': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/inline_file3 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/extent_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file1': Input/output error +51ccb5cdfc9082060f0f94a8a108fea0 - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/extent_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file1': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/extent_file1 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name4" for SCRATCH_MNT/testdir/extent_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file2': Input/output error +8d530bbe852d8bca83b131d5b3e497f5 - + +attr_set: Input/output error +Could not set "attr_name4" for SCRATCH_MNT/testdir/extent_file3 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file3': Input/output error +5d77c4d3831a35bcbbd6e7677119ce9a - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/extent_file4 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file4': Input/output error +6707ec2431e4dbea20e17da0816520bb - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/extent_file4 +touch: cannot touch 'SCRATCH_MNT/testdir/extent_file4': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/extent_file4 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/remote_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file1': Input/output error +767ebca3e4a6d24170857364f2bf2a3c - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/remote_file1 +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file1': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/remote_file1 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/remote_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file2': Input/output error +fd84ddec89237e6d34a1703639efaebf - + +attr_remove: Input/output error +Could not remove "attr_name2" for SCRATCH_MNT/testdir/remote_file2 +touch: cannot touch 'SCRATCH_MNT/testdir/remote_file2': Input/output error +attr_get: No data available +Could not get "attr_name2" for SCRATCH_MNT/testdir/remote_file2 +d41d8cd98f00b204e9800998ecf8427e - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/sf_file +touch: cannot touch 'SCRATCH_MNT/testdir/sf_file': Input/output error +34aaa49662bafb46c76e377454685071 - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/leaf_file +touch: cannot touch 'SCRATCH_MNT/testdir/leaf_file': Input/output error +664e95ec28830ffb367c0950026e0d21 - + +attr_set: Input/output error +Could not set "attr_name2" for SCRATCH_MNT/testdir/node_file +touch: cannot touch 'SCRATCH_MNT/testdir/node_file': Input/output error +bb37a78ce26472eeb711e3559933db42 - + +*** done