Date: Mon, 10 Jan 2005 14:12:00 -0800 From: John-Mark Gurney <gurney_j@resnet.uoregon.edu> To: Scott Long <scottl@freebsd.org> Cc: "Simon L. Nielsen" <simon@freebsd.org> Subject: Re: cvs commit: src/sys/netgraph ng_base.c src/sys/sys kernel.h Message-ID: <20050110221200.GF19624@funkthat.com> In-Reply-To: <41E2F4C6.1010000@freebsd.org> References: <200501061745.j06Hj344034786@repoman.freebsd.org> <20050110184139.GE19624@funkthat.com> <20050110202344.GC777@zaphod.nitro.dk> <41E2E561.9000504@freebsd.org> <20050110205654.GF777@zaphod.nitro.dk> <41E2F4C6.1010000@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Scott Long wrote this message on Mon, Jan 10, 2005 at 14:33 -0700:
> Simon L. Nielsen wrote:
>
> >On 2005.01.10 13:28:17 -0700, Scott Long wrote:
> >
> >>Simon L. Nielsen wrote:
> >>
> >>
> >>>@@ -69,3 +69,7 @@
> >>>.An -nosplit
> >>>This manual page was written by
> >>>.An Alexander Langer Aq alex@FreeBSD.org .
> >>>+.Sh BUGS
> >>>+.Nm
> >>>+only marks which modules are required for the module to work, it does
> >>>+not influence start-up ordering.
> >>
> >>It also is used by the loader and by kldload to load the dependent
> >>modules. So it's more than just an informational tag, it actually does
> >>get used by the system.
> >
> >
> >I don't think the above says it's only informational, but I'm sure the
> >wording could be improved. Perhpaps somebody has a suggestion for
> >better describing the issue?
> >
>
> Well, I guess my point is that it's important to document exactly what
> MODULE_DEPEND does do, and well as what it doesn't do. It's not really
> a bug that it doesn't change SI_ORDER order, it's just a not a feature
> that it was ever intended to support. A second paragraph under
> DESCRIPTION that talks about this would probably be most appropriate.
> Something like:
>
> .Nm
> provides hints to the kernel
> .Xr loader 8
> and to
> .Xr kldload 8
> to ensure that the named dependency is loaded prior to the existing
> module. It does not change or dictate the order in which modules are
> initialized at runtime.
This looks good, though it doesn't hint to kldload, since it will be the
kernel linker than does dependancy loading (look at
linker_load_dependencies in kern_linker.c)...
it definately should NOT be in a BUGS section as it was never intended
to influence startup order...
--
John-Mark Gurney Voice: +1 415 225 5579
"All that I will do, has been done, All that I have, has not."
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050110221200.GF19624>