mirror of
https://github.com/linux-apfs/apfstests.git
synced 2026-05-01 15:01:44 -07:00
xfs/098: fix xfs_repair on newer xfsprogs
1) use _repair_scratch_fs instead of xfs_repair The obsolete xfs_repair always cleared the log regardless of whether it is corrupted and current xfs_repair only cleared the log when -L option is specified. xfs_repair -L option should be used to clear it if xfs_repair failed to clear log. 2) catch non-zero return value instead of 2 It can be applied to both the old return value 1 and the new return value 2 3) add filter_xfs_dmesg to ignore mount related warnings If we corrupt log and mount on a CONFIG_XFS_WARN build, there will be mount related warnings in dmesg as expected. Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> Reviewed-by: Zorro Lang <zlang@redhat.com> Signed-off-by: Eryu Guan <eguan@redhat.com>
This commit is contained in:
@@ -1152,7 +1152,7 @@ _repair_scratch_fs()
|
|||||||
xfs)
|
xfs)
|
||||||
_scratch_xfs_repair "$@" 2>&1
|
_scratch_xfs_repair "$@" 2>&1
|
||||||
res=$?
|
res=$?
|
||||||
if [ "$res" -eq 2 ]; then
|
if [ "$res" -ne 0 ]; then
|
||||||
echo "xfs_repair returns $res; replay log?"
|
echo "xfs_repair returns $res; replay log?"
|
||||||
_scratch_mount
|
_scratch_mount
|
||||||
res=$?
|
res=$?
|
||||||
|
|||||||
+15
-1
@@ -54,6 +54,17 @@ _require_xfs_db_blocktrash_z_command
|
|||||||
test -z "${FUZZ_ARGS}" && FUZZ_ARGS="-n 8 -3"
|
test -z "${FUZZ_ARGS}" && FUZZ_ARGS="-n 8 -3"
|
||||||
|
|
||||||
rm -f $seqres.full
|
rm -f $seqres.full
|
||||||
|
|
||||||
|
# If we corrupt log on a CONFIG_XFS_WARN build, there will be mount related
|
||||||
|
# WARNINGs in dmesg as expected. We don't want to simply _disable_dmesg_check
|
||||||
|
# which could miss other potential bugs, so filter out the intentional WARNINGs,
|
||||||
|
# make sure test doesn't fail because of this warning and fails on other WARNINGs.
|
||||||
|
filter_xfs_dmesg()
|
||||||
|
{
|
||||||
|
local warn="WARNING:.*fs/xfs/xfs_message\.c:.*asswarn.*"
|
||||||
|
sed -e "s#$warn#Intentional warnings in asswarn#"
|
||||||
|
}
|
||||||
|
|
||||||
TESTDIR="${SCRATCH_MNT}/scratchdir"
|
TESTDIR="${SCRATCH_MNT}/scratchdir"
|
||||||
TESTFILE="${TESTDIR}/testfile"
|
TESTFILE="${TESTDIR}/testfile"
|
||||||
|
|
||||||
@@ -93,7 +104,10 @@ echo "+ mount image"
|
|||||||
_scratch_mount 2>/dev/null && _fail "mount should not succeed"
|
_scratch_mount 2>/dev/null && _fail "mount should not succeed"
|
||||||
|
|
||||||
echo "+ repair fs"
|
echo "+ repair fs"
|
||||||
_scratch_xfs_repair >> $seqres.full 2>&1
|
_repair_scratch_fs >> $seqres.full 2>&1
|
||||||
|
|
||||||
|
# mount may trigger related WARNINGs, so filter them.
|
||||||
|
_check_dmesg filter_xfs_dmesg
|
||||||
|
|
||||||
echo "+ mount image (2)"
|
echo "+ mount image (2)"
|
||||||
_scratch_mount
|
_scratch_mount
|
||||||
|
|||||||
Reference in New Issue
Block a user