From c967e246835e84e136dfd5d14d457b6625d4a297 Mon Sep 17 00:00:00 2001 From: omni <79493359+omnivagant@users.noreply.github.com> Date: Fri, 15 Apr 2022 21:26:44 +0000 Subject: [PATCH] init.d/zfs-mount: Don't fsck or mount/umount fstab entries This is better handled by existing OS toolset. Reviewed-by: Brian Behlendorf Reviewed-by: Damian Szuberski Signed-off-by: omni Issue #7374 Closes #12780 --- etc/init.d/zfs-mount.in | 82 ----------------------------------------- 1 file changed, 82 deletions(-) diff --git a/etc/init.d/zfs-mount.in b/etc/init.d/zfs-mount.in index d196d1555a45..fd2678c1a290 100755 --- a/etc/init.d/zfs-mount.in +++ b/etc/init.d/zfs-mount.in @@ -71,50 +71,6 @@ do_mount() zfs_action "Mounting ZFS filesystem(s)" \ "$ZFS" mount "-a$verbose$overlay" "$MOUNT_EXTRA_OPTIONS" - # Require each volume/filesystem to have 'noauto' and no fsck - # option. This shouldn't really be necessary, as long as one - # can get zfs-import to run sufficiently early on in the boot - # process - before local mounts. This is just here in case/if - # this isn't possible. - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_begin_msg "Mounting volumes and filesystems registered in fstab" - - read_mtab "^/dev/(zd|zvol)" - read_fstab "^/dev/(zd|zvol)" - i=0; var="FSTAB_0" - while [ -n "$(eval echo "\$$var")" ] - do - mntpt=$(eval echo "\$$var") - dev=$(eval echo "\$FSTAB_dev_$i") - if ! in_mtab "$mntpt" && ! is_mounted "$mntpt" && [ -e "$dev" ] - then - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_progress_msg "$mntpt " - fsck "$dev" && mount "$mntpt" - fi - - i=$((i + 1)) - var=$(eval echo "FSTAB_$i") - done - - read_mtab "[[:space:]]zfs[[:space:]]" - read_fstab "[[:space:]]zfs[[:space:]]" - i=0; var=$(eval echo "FSTAB_$i") - while [ -n "$(eval echo "\$$var")" ] - do - mntpt=$(eval echo "\$$var") - if ! in_mtab "$mntpt" && ! is_mounted "$mntpt" - then - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_progress_msg "$mntpt " - mount "$mntpt" - fi - - i=$((i + 1)) - var=$(eval echo "FSTAB_$i") - done - check_boolean "$VERBOSE_MOUNT" && zfs_log_end_msg 0 - return 0 } @@ -129,44 +85,6 @@ do_unmount() # this isn't possible. zfs_action "Unmounting ZFS filesystems" "$ZFS" unmount -a - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_begin_msg "Unmounting volumes and filesystems registered in fstab" - - read_mtab "^/dev/(zd|zvol)" - read_fstab "^/dev/(zd|zvol)" - i=0; var="FSTAB_0" - while [ -n "$(eval echo "\$$var")" ] - do - mntpt=$(eval echo "\$$var") - dev=$(eval echo "\$FSTAB_dev_$i") - if in_mtab "$mntpt" - then - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_progress_msg "$mntpt " - umount "$mntpt" - fi - - i=$((i + 1)) - var=$(eval echo "FSTAB_$i") - done - - read_mtab "[[:space:]]zfs[[:space:]]" - read_fstab "[[:space:]]zfs[[:space:]]" - i=0; var="FSTAB_0" - while [ -n "$(eval echo "\$$var")" ] - do - mntpt=$(eval echo "\$$var") - if in_mtab "$mntpt"; then - check_boolean "$VERBOSE_MOUNT" && \ - zfs_log_progress_msg "$mntpt " - umount "$mntpt" - fi - - i=$((i + 1)) - var=$(eval echo "FSTAB_$i") - done - check_boolean "$VERBOSE_MOUNT" && zfs_log_end_msg 0 - return 0 }