populate: reformat external ext[34] journal devices when restoring a cached image
The fs population code has the ability to save cached metadumps of
filesystems to save time when running fstests. The cached images should
be unmounted cleanly, so we never save the contents of external journal
devices.
Unfortunately, the cache restore code fails to reset the external
journal when restoring a clean image, so we ignore cached images because
the journal doesn't match the filesystem. This makes test runtimes
longer than they need to be.
Solve this by reformatting the external journal to match the filesystem.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Zorro Lang <zlang@redhat.com>
diff --git a/common/populate b/common/populate
index 9b71bec..681b36a 100644
--- a/common/populate
+++ b/common/populate
@@ -16,6 +16,9 @@
_require_command "$XFS_DB_PROG" "xfs_db"
_require_command "$WIPEFS_PROG" "wipefs"
;;
+ ext*)
+ _require_command "$DUMPE2FS_PROG" "dumpe2fs"
+ ;;
esac
}
@@ -871,9 +874,20 @@
return $res
;;
"ext2"|"ext3"|"ext4")
- # ext4 cannot e2image external logs, so we cannot restore
- test -n "${SCRATCH_LOGDEV}" && return 1
- e2image -r "${metadump}" "${SCRATCH_DEV}" && return 0
+ e2image -r "${metadump}" "${SCRATCH_DEV}"
+ ret=$?
+ test $ret -ne 0 && return $ret
+
+ # ext4 cannot e2image external logs, so we have to reformat
+ # the scratch device to match the restored fs
+ if [ -n "${SCRATCH_LOGDEV}" ]; then
+ local fsuuid="$($DUMPE2FS_PROG -h "${SCRATCH_DEV}" 2>/dev/null | \
+ grep 'Journal UUID:' | \
+ sed -e 's/Journal UUID:[[:space:]]*//g')"
+ $MKFS_EXT4_PROG -O journal_dev "${SCRATCH_LOGDEV}" \
+ -F -U "${fsuuid}"
+ fi
+ return 0
;;
esac
return 1
