rc scripts change for review

[Rick Macklem]

> On Sun, Apr 24, 2011 at 08:05:04AM -0400, Rick Macklem wrote:
> > One possible explanation here (and it may be that I haven't set the
> > modules/sysctl stuff up correctly?) is that the "nfsd" module
> > depends
> > on the module "nfscommon" (which is loaded when nfsd is loaded,
> > because
> > of a MODULE_DEPEND() in "nfsd"). The SYSCTL_NODE() is in
> > "nfscommon".
> > I only have a SYSCTL_DECL() in "nfsd". (ie. maybe kldload doesn't
> > wait
> > for "nfscommon" to be loaded.)
> >
> > I just tried:
> > load_kld nfscommon
> > load_kld nfsd
> > - and this works without needing "nfssrv"
> >
> > I'm not sure if the above implies a bug in either my server code or
> > the module loading/sysctl registering stuff, but I'm confortable
> > with
> > doing the 2 load_kld's.
> 
> Dependencies are loaded synchronously too.
Oops, I know what my bug was/is now.

Since the client was linked into the kernel "vfs.newnfs" was there, so
the code snippet that did "load_kld nfsd" when it wasn't there didn't
happen.

Once I made "load_kld nfsd" unconditional, it works and doesn't need
the "load_kld nfscommon" first.

I think this is now resolved.

Sorry about the noise and thanks for the help, rick