Date: Sat, 10 Dec 2005 10:21:00 -0800 From: Brooks Davis <brooks@one-eyed-alien.net> To: Doug Barton <dougb@FreeBSD.org> Cc: freebsd-rc@FreeBSD.org Subject: Re: For Review: Latest update to rcorder patch Message-ID: <20051210182100.GA12946@odin.ac.hmc.edu> In-Reply-To: <439A9857.9090003@FreeBSD.org> References: <4398AB6A.3000209@FreeBSD.org> <20051208222013.GB5759@odin.ac.hmc.edu> <43996043.8020609@FreeBSD.org> <20051209193830.GA15097@odin.ac.hmc.edu> <439A9857.9090003@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--GvXjxJ+pjyke8COw
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Sat, Dec 10, 2005 at 12:56:55AM -0800, Doug Barton wrote:
> Brooks Davis wrote:
>=20
> >Instead of talking about diskless systems I'd talk about having remote
> >file systems in ${local_startup}=20
>=20
> Ok, how's this:
>=20
> +The early stage should contain all the services needed to
> +get the disks (local or remote) mounted so that the late
> +stage can include scripts contained in the directories
> +listed in the
> +.Va local_startup
> +variable (see below).
> +Thus, the two likely candidates for this value are
> +mountcritlocal for the typical system,
> +and mountcritremote if the system needs remote file
> +systems mounted to get access to the
> +.Va local_startup
> +directories.
> +For example when
> +.Pa /usr/local
> +is NFS mounted.
That looks fine.
> >>>- diskless.8 should probably be updated to mentation that
> >>> early_late_divider must be set to mountcritremote for the documented
> >>> configuration to work.
> >>I can do that as part of the change if you want, or you can do it, just=
=20
> >>let
> >>me know.
> >
> >Please go ahead.=20
>=20
> Ok, how is this:
>=20
> +.Pp
> +One difference that you should pay particular attention to is
> +the value of
> +.Va local_startup
> +in
> +.Pa /etc/defaults/rc.conf .
> +A typical value for a
> +.Nm
> +boot is
> +.Va mountcritremote ,
> +however your needs may be different.
> +.Pp
> The scripts provide four
That looks OK as well.
> Suggestions for improvements are welcome, it's been a long day.
>=20
> >OK. I might make rcconf.sh into a no-op that provides rcconf in the
> >first commit and remove it for real in a day or so to make it easier to
> >debug any inadvertent issues that result from increasing the number of
> >scripts with no dependencies.
>=20
> Yeah, not only is this a factor, but I also failed to take into account=
=20
> another critical element. rcorder needs at least one script without any=
=20
> REQUIRE lines to know where to start. Right now that's rcconf.sh by=20
> design, but I think we need to replace that, rather than just delete=20
> rcconf.sh. This is one of the few times when I think a dummy dependency=
=20
> would be useful. I propose that I add a dummy dependency called START, an=
d=20
> change everything that currently depends on rcconf to depend on START=20
> instead. Sound good?
I'm not sure you actually need a dummy. NetBSD just lets early things
have no REQUIRE lines (for instance cdd only contains "PROVIDE:
disks"). There's no requirement that the tree have a single top.
> BTW, I noticed that there are several scripts that have no REQUIRE or=20
> nostart. Should those be updated? They are: ccd, gbde, mixer, and=20
> securelevel.
mixer is broken and should REQUIRE on cleanvar. The ccd script probably
doesn't depend on anything but the obsolete rcconf.sh (which it
currently gets from initrandom's BEFORE: disks), but gbde may depend on
initrandom (I haven't read the code). I might just let initrandom take
the top slot. securelevel is just plain wrong in the current setup. If
it's going to do any good, it needs to come much earlier (NetBSD has it
BEFORE: DAEMON which makes sense if it's supposed to secure your system
from bad guys.) Right now it's in the noise at the end of the list due
to a whole lot of IMO rather bogus BEFORE: securelevel entries.
-- Brooks
--=20
Any statement of the form "X is the one, true Y" is FALSE.
PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4
--GvXjxJ+pjyke8COw
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
iD8DBQFDmxyLXY6L6fI4GtQRAndsAKDRL2MkANE4ZuNWm0qk6XwXoHKtrgCggx0G
6aJ0BXFfsM2hMV4TG/9q8IM=
=s8j/
-----END PGP SIGNATURE-----
--GvXjxJ+pjyke8COw--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20051210182100.GA12946>