Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Jan 2005 14:33:58 -0700
From:      Scott Long <scottl@freebsd.org>
To:        "Simon L. Nielsen" <simon@freebsd.org>
Cc:        cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/netgraph ng_base.c src/sys/sys kernel.h
Message-ID:  <41E2F4C6.1010000@freebsd.org>
In-Reply-To: <20050110205654.GF777@zaphod.nitro.dk>
References:  <200501061745.j06Hj344034786@repoman.freebsd.org> <20050110184139.GE19624@funkthat.com> <20050110202344.GC777@zaphod.nitro.dk> <41E2E561.9000504@freebsd.org> <20050110205654.GF777@zaphod.nitro.dk>

index | next in thread | previous in thread | raw e-mail

Simon L. Nielsen wrote:

> On 2005.01.10 13:28:17 -0700, Scott Long wrote:
> 
>>Simon L. Nielsen wrote:
>>
>>
>>>On 2005.01.10 10:41:39 -0800, John-Mark Gurney wrote:
>>>
>>>
>>>>So, to everone, MODULE_DEPEND only marks which modules are required
>>>>for this module to work, it does not infulence startup ordering at
>>>>all...
>>>
>>>So, should I commit this patch?
>>>
> 
> [...]
> 
>>>@@ -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.

Scott


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?41E2F4C6.1010000>