Re: RCng/NTP Catch-22

[-rc added to the CC list.]

On Thu, Oct 21, 2004 at 06:21:45AM -0700, Kevin Oberman wrote:
> > Date: Thu, 21 Oct 2004 00:35:35 +0200
> > From: Pawel Worach <pawel.worach_at_telia.com>
> > 
> > Kevin Oberman wrote:
> > >>Date: Thu, 21 Oct 2004 00:07:08 +0200
> > >>From: Pawel Worach <pawel.worach_at_telia.com>
> > > Pawel,
> > > 
> > > Thanks, but you missed the point. /etc/rc.d/devfs is always executed
> > > AFTER ntpd. So putting the links into devfs.conf does not help. It's too
> > > late.
> > 
> > Sorry, missed the patch I'm using.
> > 
> > --- /usr/src/etc/rc.d/NETWORKING        Fri Oct  8 01:49:33 2004
> > +++ NETWORKING  Thu Oct 21 00:30:39 2004
> > _at__at_ -6,7 +6,7 _at__at_
> > 
> >   # PROVIDE: NETWORKING NETWORK
> >   # REQUIRE: dhclient netif routing network_ipv6 isdnd ppp-user
> > -# REQUIRE: routed mrouted route6d mroute6d
> > +# REQUIRE: routed mrouted route6d mroute6d devfs
> > 
> >   #      This is a dummy dependency, for services which require networking
> >   #      to be operational before starting.
> > --- /usr/src/etc/rc.d/devfs     Fri Oct  8 01:49:33 2004
> > +++ devfs       Thu Oct 21 00:31:52 2004
> > _at__at_ -4,7 +4,6 _at__at_
> >   #
> > 
> >   # PROVIDE: devfs
> > -# REQUIRE: LOGIN
> >   # BEFORE:  securelevel
> >   # KEYWORD: nojail
> 
> Thanks. This does the trick, but allows devfs to start at any time
> before securelevel. I'm not sure if this might bite me somewhere down
> the line. Why does devfs require LOGIN? I assume three was a reason, but
> I have no idea if it was a good one or just a convenient choice.
> 
> In any case, this does get things started, but a solution really needs
> to be incorporated into the distribution system. People installing and
> maintaining systems should not have to patch startup files to make ntpd
> work.

The requiement of LOGIN is bogus.  Things that configure important
features like /dev must come before users might try to use them.
The current setup is extra nonsensical because devd is already
running when rc.d/devfs starts beating on things in /dev.

I'll commit the following patch soon if there are no objections.

-- Brooks

Index: devfs
===================================================================
RCS file: /home/ncvs/src/etc/rc.d/devfs,v
retrieving revision 1.9
diff -u -p -r1.9 devfs
--- devfs	7 Oct 2004 13:55:25 -0000	1.9
+++ devfs	21 Oct 2004 15:12:52 -0000
_at__at_ -4,8 +4,7 _at__at_
 #
 
 # PROVIDE: devfs
-# REQUIRE: LOGIN
-# BEFORE:  securelevel
+# BEFORE:  NETWORKING securelevel
 # KEYWORD: nojail
 
 . /etc/rc.subr

-- 
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