From: Darrick J. Wong <djwong@xxxxxxxxxx> Some of the older blocktrash-based fuzz tests cause the fs to go down due to the corrupted image and fail to remount. Offline repair fails because _repair_scratch_fs is the helper that is smart enough to call xfs_repair -L, not _scratch_xfs_repair. Fix these instances. Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> --- tests/xfs/083 | 8 ++++---- tests/xfs/085 | 2 +- tests/xfs/086 | 4 ++-- tests/xfs/087 | 2 +- tests/xfs/088 | 4 ++-- tests/xfs/089 | 4 ++-- tests/xfs/091 | 4 ++-- tests/xfs/093 | 2 +- tests/xfs/097 | 2 +- tests/xfs/099 | 4 ++-- tests/xfs/100 | 4 ++-- tests/xfs/101 | 4 ++-- tests/xfs/102 | 4 ++-- tests/xfs/105 | 4 ++-- tests/xfs/112 | 4 ++-- tests/xfs/113 | 4 ++-- tests/xfs/117 | 2 +- tests/xfs/120 | 2 +- tests/xfs/123 | 2 +- tests/xfs/124 | 4 ++-- tests/xfs/125 | 4 ++-- tests/xfs/126 | 4 ++-- tests/xfs/130 | 2 +- tests/xfs/235 | 2 +- 24 files changed, 41 insertions(+), 41 deletions(-) diff --git a/tests/xfs/083 b/tests/xfs/083 index a3f32cb7..14a36416 100755 --- a/tests/xfs/083 +++ b/tests/xfs/083 @@ -44,7 +44,7 @@ scratch_repair() { FSCK_LOG="${tmp}-fuzz-${fsck_pass}.log" echo "++ fsck pass ${fsck_pass}" > "${FSCK_LOG}" - _scratch_xfs_repair >> "${FSCK_LOG}" 2>&1 + _repair_scratch_fs >> "${FSCK_LOG}" 2>&1 res=$? if [ "${res}" -eq 0 ]; then echo "++ allegedly fixed, reverify" >> "${FSCK_LOG}" @@ -105,7 +105,7 @@ echo "+ populate fs image" >> $seqres.full _scratch_populate >> $seqres.full echo "+ check fs" >> $seqres.full -_scratch_xfs_repair >> $seqres.full 2>&1 || _fail "should pass initial fsck" +_repair_scratch_fs >> $seqres.full 2>&1 || _fail "should pass initial fsck" echo "++ corrupt image" >> $seqres.full _scratch_xfs_db -x -c blockget -c "blocktrash ${FUZZ_ARGS}" >> $seqres.full 2>&1 @@ -129,7 +129,7 @@ done echo "+ fsck loop returns ${fsck_loop_ret}" >> $seqres.full echo "++ check fs for round 2" >> $seqres.full -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 ROUND2_LOG="${tmp}-round2-${fsck_pass}.log" echo "++ mount image (2)" >> $ROUND2_LOG @@ -150,7 +150,7 @@ umount "${SCRATCH_MNT}" >> $ROUND2_LOG 2>&1 cat "$ROUND2_LOG" >> $seqres.full echo "++ check fs (2)" >> $seqres.full -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 egrep -q '(did not fix|makes no progress)' $seqres.full && echo "xfs_repair failed" | tee -a $seqres.full if [ "$(wc -l < "$ROUND2_LOG")" -ne 8 ]; then diff --git a/tests/xfs/085 b/tests/xfs/085 index 560b5a24..5d898088 100755 --- a/tests/xfs/085 +++ b/tests/xfs/085 @@ -75,7 +75,7 @@ echo "+ mount image" _try_scratch_mount 2>/dev/null && _fail "mount should not succeed" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/086 b/tests/xfs/086 index f94c26b4..f4cf950d 100755 --- a/tests/xfs/086 +++ b/tests/xfs/086 @@ -86,7 +86,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount @@ -109,7 +109,7 @@ done umount "${SCRATCH_MNT}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount diff --git a/tests/xfs/087 b/tests/xfs/087 index 967791dd..e7b06e09 100755 --- a/tests/xfs/087 +++ b/tests/xfs/087 @@ -86,7 +86,7 @@ fi echo "broken: ${broken}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/088 b/tests/xfs/088 index 62360ca8..42a186be 100755 --- a/tests/xfs/088 +++ b/tests/xfs/088 @@ -86,7 +86,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount @@ -109,7 +109,7 @@ done umount "${SCRATCH_MNT}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount diff --git a/tests/xfs/089 b/tests/xfs/089 index 79167a57..7d8af7ce 100755 --- a/tests/xfs/089 +++ b/tests/xfs/089 @@ -86,7 +86,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount @@ -110,7 +110,7 @@ done umount "${SCRATCH_MNT}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount diff --git a/tests/xfs/091 b/tests/xfs/091 index db6bb0b2..5fa98328 100755 --- a/tests/xfs/091 +++ b/tests/xfs/091 @@ -86,7 +86,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount @@ -110,7 +110,7 @@ done umount "${SCRATCH_MNT}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image" _scratch_mount diff --git a/tests/xfs/093 b/tests/xfs/093 index 3bdbff4d..9a61cc3e 100755 --- a/tests/xfs/093 +++ b/tests/xfs/093 @@ -86,7 +86,7 @@ fi echo "broken: ${broken}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/097 b/tests/xfs/097 index f8ea4676..98648c9e 100755 --- a/tests/xfs/097 +++ b/tests/xfs/097 @@ -88,7 +88,7 @@ fi echo "broken: ${broken}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/099 b/tests/xfs/099 index 0cf19682..9a1408b8 100755 --- a/tests/xfs/099 +++ b/tests/xfs/099 @@ -74,8 +74,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/100 b/tests/xfs/100 index 44d175cc..277f26ec 100755 --- a/tests/xfs/100 +++ b/tests/xfs/100 @@ -79,8 +79,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/101 b/tests/xfs/101 index 023cc349..6eb303ad 100755 --- a/tests/xfs/101 +++ b/tests/xfs/101 @@ -74,8 +74,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/102 b/tests/xfs/102 index 907b8592..23326ecf 100755 --- a/tests/xfs/102 +++ b/tests/xfs/102 @@ -79,8 +79,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/105 b/tests/xfs/105 index bb7e93e1..7aeee7f0 100755 --- a/tests/xfs/105 +++ b/tests/xfs/105 @@ -79,8 +79,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/112 b/tests/xfs/112 index cf0a36d0..085f21ee 100755 --- a/tests/xfs/112 +++ b/tests/xfs/112 @@ -79,11 +79,11 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 if [ $? -eq 2 ]; then _scratch_mount umount "${SCRATCH_MNT}" - _scratch_xfs_repair >> $seqres.full 2>&1 + _repair_scratch_fs >> $seqres.full 2>&1 fi echo "+ mount image (2)" diff --git a/tests/xfs/113 b/tests/xfs/113 index 3ab3cf5e..3dc51381 100755 --- a/tests/xfs/113 +++ b/tests/xfs/113 @@ -79,8 +79,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/117 b/tests/xfs/117 index 15765a56..d3f4675f 100755 --- a/tests/xfs/117 +++ b/tests/xfs/117 @@ -88,7 +88,7 @@ fi echo "broken: ${broken}" echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/120 b/tests/xfs/120 index 9fcce9ee..1f594ebc 100755 --- a/tests/xfs/120 +++ b/tests/xfs/120 @@ -74,7 +74,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/123 b/tests/xfs/123 index a7fae5f6..ced453bd 100755 --- a/tests/xfs/123 +++ b/tests/xfs/123 @@ -69,7 +69,7 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/124 b/tests/xfs/124 index f4b24dd6..50faa66b 100755 --- a/tests/xfs/124 +++ b/tests/xfs/124 @@ -78,8 +78,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/125 b/tests/xfs/125 index 3bdf73c4..c9ee2cf3 100755 --- a/tests/xfs/125 +++ b/tests/xfs/125 @@ -78,8 +78,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/126 b/tests/xfs/126 index 3f069c16..0ca0670c 100755 --- a/tests/xfs/126 +++ b/tests/xfs/126 @@ -83,8 +83,8 @@ if _try_scratch_mount >> $seqres.full 2>&1; then fi echo "+ repair fs" -_scratch_xfs_repair >> $seqres.full 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/130 b/tests/xfs/130 index b4404c5d..7ff565c6 100755 --- a/tests/xfs/130 +++ b/tests/xfs/130 @@ -71,7 +71,7 @@ _scratch_unmount >> $seqres.full 2>&1 echo "+ repair fs" _disable_dmesg_check _repair_scratch_fs >> "$seqres.full" 2>&1 -_scratch_xfs_repair >> "$seqres.full" 2>&1 +_repair_scratch_fs >> "$seqres.full" 2>&1 echo "+ mount image (2)" _scratch_mount diff --git a/tests/xfs/235 b/tests/xfs/235 index 553a3bc8..fe3a2cd0 100755 --- a/tests/xfs/235 +++ b/tests/xfs/235 @@ -72,7 +72,7 @@ fi echo "+ repair fs" _disable_dmesg_check _repair_scratch_fs >> "$seqres.full" 2>&1 -_scratch_xfs_repair >> $seqres.full 2>&1 +_repair_scratch_fs >> $seqres.full 2>&1 echo "+ mount image (2)" _scratch_mount