xfs: verify invalid metadata LSN detection
authorBrian Foster <bfoster@redhat.com>
Mon, 21 Dec 2015 07:00:29 +0000 (18:00 +1100)
committerDave Chinner <david@fromorbit.com>
Mon, 21 Dec 2015 07:00:29 +0000 (18:00 +1100)
XFS v5 superblock fs' use metadata LSN tracking to determine when an
on-disk structure was last written to disk. This is used to ensure log
recovery operates correctly after an unclean shutdown. To work
correctly, the on-disk metadata LSNs must always remain behind the
current LSN with respect to the log.

Historically, xfs_repair had a problem where it incorrectly formats the
log to an LSN that is potentially behind existing metadata LSNs. As
such, xfs_repair and the kernel have been updated to prevent, detect and
recover from the problem. Add a test that intentionally formats the log
incorrectly and verifies that the fs fails to mount and that xfs_repair
detects the invalid metadata LSNs.

Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
tests/xfs/137 [new file with mode: 0755]
tests/xfs/137.out [new file with mode: 0644]

diff --git a/tests/xfs/137 b/tests/xfs/137
new file mode 100755 (executable)
index 0000000..f162264
--- /dev/null
@@ -0,0 +1,90 @@
+#! /bin/bash
+# FS QA Test No. 137
+# XFS v5 supers carry an LSN in various on-disk structures to track when
+# associated metadata was last written to disk. These metadata LSNs must always
+# be behind the current LSN as dictated by the log to ensure log recovery
+# correctness after a potential crash. This test uses xfs_db to intentionally
+# put the current LSN behind metadata LSNs and verifies that the kernel and
+# xfs_repair detect the problem.
+# Copyright (c) 2015 Red Hat, Inc. All Rights Reserved.
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# GNU General Public License for more details.
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+seq=`basename $0`
+echo "QA output created by $seq"
+status=1       # failure is the default!
+trap "_cleanup; exit \$status" 0 1 2 3 15
+       cd /
+       rm -f $tmp.*
+rm -f $seqres.full
+# get standard environment, filters and checks
+. ./common/rc
+# real QA test starts here
+# Modify as appropriate.
+_supported_fs xfs
+_supported_os Linux
+_require_xfs_db_command "logformat"
+_scratch_mkfs >> $seqres.full 2>&1 || _fail "mkfs failed"
+# push the log cycle ahead so we have room to move it backwards later
+$XFS_DB_PROG -x -c "logformat -c 3" $SCRATCH_DEV >> $seqres.full 2>&1
+# do some work on the fs to update metadata LSNs
+$FSSTRESS_PROG -d $SCRATCH_MNT -n 999 -p 4 -w >> $seqres.full 2>&1
+# Reformat to the current cycle and try to mount. This fails in most cases
+# because the sb LSN is ahead of the current LSN. If it doesn't fail, push the
+# cycle back further and try again.
+$XFS_DB_PROG -x -c "logformat" $SCRATCH_DEV >> $seqres.full 2>&1
+_scratch_mount >> $seqres.full 2>&1
+if [ $? != 0 ]; then
+       echo mount failure detected
+       _scratch_unmount
+       $XFS_DB_PROG -x -c "logformat -c 2" $SCRATCH_DEV >> $seqres.full 2>&1
+       _scratch_mount >> $seqres.full 2>&1 || echo mount failure detected
+# verify that repair detects invalid LSNs as well
+$XFS_REPAIR_PROG -n $SCRATCH_DEV >> $seqres.full 2>&1 || \
+       echo repair failure detected
+# repair for real so the post-test check can verify repair fixed things up
+$XFS_REPAIR_PROG $SCRATCH_DEV >> $seqres.full 2>&1
+# success, all done
diff --git a/tests/xfs/137.out b/tests/xfs/137.out
new file mode 100644 (file)
index 0000000..b9e4bee
--- /dev/null
@@ -0,0 +1,3 @@
+QA output created by 137
+mount failure detected
+repair failure detected
index 6550702897119b5f2c7c42d0c4a5184eee742293..dd206efc634c06dc4fee893547373162f7b576fc 100644 (file)
 134 quota auto quick
 135 auto logprint quick v2log
 136 attr2
+137 auto metadata v2log
 142 dmapi
 143 dmapi
 144 dmapi