Date: Tue, 09 Apr 2019 16:06:36 +0100 From: Chris Rees <crees@bayofrum.net> To: "O. Hartmann" <ohartmann@walstatt.org>, Chris Rees <crees@FreeBSD.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r346017 - in head: libexec/rc libexec/rc/rc.d share/man/man5 Message-ID: <749EBFB4-94B8-4FD9-8774-FE6FB18FFA07@bayofrum.net> In-Reply-To: <20190409151340.6f33b077@freyja> References: <201904071831.x37IVj4r096803@repo.freebsd.org> <20190409151340.6f33b077@freyja>
next in thread | previous in thread | raw e-mail | index | archive | help
Hey, On 9 April 2019 14:13:54 BST, "O. Hartmann" <ohartmann@walstatt.org> wrote: >On Sun, 7 Apr 2019 18:31:45 +0000 (UTC) >Chris Rees <crees@FreeBSD.org> wrote: > >> Author: crees (doc,ports committer) >> Date: Sun Apr 7 18:31:45 2019 >> New Revision: 346017 >> URL: https://svnweb.freebsd.org/changeset/base/346017 >> >> Log: >> Remove now unnecessary kldstat check before attempting to load >modules. >> >> Since r233109, kldload has the -n option, which silently ignores >options >> that are already loaded. >> >>=20=20 >https://lists.freebsd.org/pipermail/freebsd-rc/2018-December/003899.html >> >> Note that this script no longer reports if the module is already >loaded, >> but it could be argued this wasn't particularly useful information. >> >> PR: docs/234248 >> Reviewed by: bcr (docs), kib, rgrimes (visual) >> Approved by: jilles >> Differential Revision: https://reviews.freebsd.org/D18670 >> >> Modified: >> head/libexec/rc/rc.d/abi >> head/libexec/rc/rc.d/bthidd >> head/libexec/rc/rc.d/cfumass >> head/libexec/rc/rc.d/kld >> head/libexec/rc/rc.d/mdconfig >> head/libexec/rc/rc.d/mdconfig2 >> head/libexec/rc/rc.d/mountcritremote >> head/libexec/rc/rc.d/syscons >> head/libexec/rc/rc.subr >> head/share/man/man5/rc.conf.5 >> >> Modified: head/libexec/rc/rc.d/abi >> >=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D {Snip} >It seems this patch breaks ZFS if ZFS is built-in kernel! > >Having ZFS as kernel built-in leaves the system not mounting ANY(!) ZFS >volume >at boot time, rendering servers unusable. >Mounting ZFS filesystems with "zfs mount -a" bring back the ZFS >filesystems as >expected. > Are you sure it's this causing that? Please try reverting it. Chris=20 --=20 Sent from my Android device with K-9 Mail. Please excuse my brevity. --=20 This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?749EBFB4-94B8-4FD9-8774-FE6FB18FFA07>