From owner-freebsd-rc@FreeBSD.ORG Sat Dec 10 18:21:02 2005 Return-Path: X-Original-To: freebsd-rc@FreeBSD.org Delivered-To: freebsd-rc@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 3627016A420; Sat, 10 Dec 2005 18:21:02 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id A907D43D5F; Sat, 10 Dec 2005 18:21:01 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id jBAIL0ow000864; Sat, 10 Dec 2005 10:21:00 -0800 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id jBAIL08g000863; Sat, 10 Dec 2005 10:21:00 -0800 Date: Sat, 10 Dec 2005 10:21:00 -0800 From: Brooks Davis To: Doug Barton Message-ID: <20051210182100.GA12946@odin.ac.hmc.edu> 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> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="GvXjxJ+pjyke8COw" Content-Disposition: inline In-Reply-To: <439A9857.9090003@FreeBSD.org> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new X-Spam-Status: No, hits=0.0 required=8.0 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on odin.ac.hmc.edu Cc: freebsd-rc@FreeBSD.org Subject: Re: For Review: Latest update to rcorder patch X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 10 Dec 2005 18:21:02 -0000 --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--