svn commit: r364817 - head/libexec/rc/rc.d
Andriy Gapon
avg at FreeBSD.org
Thu Aug 27 08:54:07 UTC 2020
On 26/08/2020 16:13, Cy Schubert wrote:
> Author: cy
> Date: Wed Aug 26 13:13:57 2020
> New Revision: 364817
> URL: https://svnweb.freebsd.org/changeset/base/364817
>
> Log:
> As of r364746 (OpenZFS import) existing ZPOOLs are not imported
> prior to zvol and mountcritlocal resulting in ZVOLs (swap and
I probably missed some discussion, so I am curious why that is.
> virtual machine UFS filesystems) being unavailable, leading to
> boot failures.
> We move the zpool import from zfs to a new zpool script, with the
> -N option to avoid mounting datasets while making the ZPOOL's
> datasets available for "legacy" mount (mountpoint=legacy) and ZVOLs
> available for subsequent use for swap (in the zvol rc sript) or
> for UFS or other filesystems in fstab(5), mounted by mountcritlocal.
>
> Reviewed by: freqlabs (previous version)
> Differential Revision: https://reviews.freebsd.org/D26185
>
> Added:
> head/libexec/rc/rc.d/zpool (contents, props changed)
[snip]
> +zpool_start()
> +{
> + local cachefile
> +
> + for cachefile in /boot/zfs/zpool.cache /etc/zfs/zpool.cache; do
> + if [ -r $cachefile ]; then
> + zpool import -c $cachefile -a -N
I would add a break here, so that pools are imported either from one cache file
or the other but not both. It makes sense to have two cache file definitions
for migration, but I think that it does not make sense to split pools between
the cache files.
> + fi
> + done
> +}
--
Andriy Gapon
More information about the svn-src-all
mailing list