]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
generic: test recovery of extended attribute updates
authorDarrick J. Wong <djwong@kernel.org>
Thu, 20 Jun 2024 20:57:11 +0000 (13:57 -0700)
committerZorro Lang <zlang@kernel.org>
Sun, 23 Jun 2024 15:04:36 +0000 (23:04 +0800)
Fork generic/475 to test recovery of extended attribute modifications
and log recovery.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Zorro Lang <zlang@kernel.org>
tests/generic/753 [new file with mode: 0755]
tests/generic/753.out [new file with mode: 0644]

diff --git a/tests/generic/753 b/tests/generic/753
new file mode 100755 (executable)
index 0000000..bb547a6
--- /dev/null
@@ -0,0 +1,93 @@
+#! /bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (c) 2022-2024 Oracle, Inc.  All Rights Reserved.
+#
+# FS QA Test No. 753
+#
+# Test log recovery with repeated (simulated) disk failures.  We kick
+# off fsstress on the scratch fs to exercise extended attribute operations,
+# then switch out the underlying device with dm-error to see what happens when
+# the disk goes down.  Having taken down the fs in this manner, remount it and
+# repeat.
+#
+. ./common/preamble
+_begin_fstest shutdown auto log metadata eio recoveryloop attr
+
+# Override the default cleanup function.
+_cleanup()
+{
+       cd /
+       rm -f $tmp.*
+       $KILLALL_PROG -9 fsstress > /dev/null 2>&1
+       _dmerror_unmount
+       _dmerror_cleanup
+}
+
+# Import common functions.
+. ./common/dmerror
+
+# Modify as appropriate.
+_supported_fs generic
+
+_require_scratch
+_require_dm_target error
+_require_command "$KILLALL_PROG" "killall"
+
+echo "Silence is golden."
+
+_scratch_mkfs >> $seqres.full 2>&1
+_require_metadata_journaling $SCRATCH_DEV
+_dmerror_init
+_dmerror_mount
+
+args=('-z' '-S' 'c')
+
+# Do some directory tree modifications, but the bulk of this is geared towards
+# exercising the xattr code, especially attr_set which can do up to 10k values.
+for verb in unlink rmdir; do
+       args+=('-f' "${verb}=50")
+done
+for verb in creat mkdir; do
+       args+=('-f' "${verb}=2")
+done
+for verb in getfattr listfattr; do
+       args+=('-f' "${verb}=3")
+done
+for verb in attr_remove removefattr; do
+       args+=('-f' "${verb}=4")
+done
+args+=('-f' "setfattr=20")
+args+=('-f' "attr_set=60")     # sets larger xattrs
+
+while _soak_loop_running $((50 * TIME_FACTOR)); do
+       ($FSSTRESS_PROG "${args[@]}" $FSSTRESS_AVOID -d $SCRATCH_MNT -n 999999 -p $((LOAD_FACTOR * 4)) >> $seqres.full &) \
+               > /dev/null 2>&1
+
+       # purposely include 0 second sleeps to test shutdown immediately after
+       # recovery
+       sleep $((RANDOM % 3))
+
+       # This test aims to simulate sudden disk failure, which means that we
+       # do not want to quiesce the filesystem or otherwise give it a chance
+       # to flush its logs.  Therefore we want to call dmsetup with the
+       # --nolockfs parameter; to make this happen we must call the load
+       # error table helper *without* 'lockfs'.
+       _dmerror_load_error_table
+
+       ps -e | grep fsstress > /dev/null 2>&1
+       while [ $? -eq 0 ]; do
+               $KILLALL_PROG -9 fsstress > /dev/null 2>&1
+               wait > /dev/null 2>&1
+               ps -e | grep fsstress > /dev/null 2>&1
+       done
+
+       # Mount again to replay log after loading working table, so we have a
+       # consistent XFS after test.
+       _dmerror_unmount || _fail "unmount failed"
+       _dmerror_load_working_table
+       _dmerror_mount || _fail "mount failed"
+done
+
+# success, all done
+status=0
+exit
diff --git a/tests/generic/753.out b/tests/generic/753.out
new file mode 100644 (file)
index 0000000..88cbfe1
--- /dev/null
@@ -0,0 +1,2 @@
+QA output created by 753
+Silence is golden.