From owner-cvs-all@FreeBSD.ORG Mon Jan 10 20:28:20 2005 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8E3C216A4CE; Mon, 10 Jan 2005 20:28:20 +0000 (GMT) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id 057A543D1F; Mon, 10 Jan 2005 20:28:20 +0000 (GMT) (envelope-from scottl@freebsd.org) Received: from [192.168.254.12] (g4.samsco.home [192.168.254.12]) (authenticated bits=0) by pooker.samsco.org (8.12.11/8.12.10) with ESMTP id j0AKVkgb041138; Mon, 10 Jan 2005 13:31:46 -0700 (MST) (envelope-from scottl@freebsd.org) Message-ID: <41E2E561.9000504@freebsd.org> Date: Mon, 10 Jan 2005 13:28:17 -0700 From: Scott Long User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7) Gecko/20040514 X-Accept-Language: en-us, en MIME-Version: 1.0 To: "Simon L. Nielsen" References: <200501061745.j06Hj344034786@repoman.freebsd.org> <20050110184139.GE19624@funkthat.com> <20050110202344.GC777@zaphod.nitro.dk> In-Reply-To: <20050110202344.GC777@zaphod.nitro.dk> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, hits=0.0 required=3.8 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on pooker.samsco.org cc: cvs-src@freebsd.org cc: John-Mark Gurney cc: src-committers@freebsd.org cc: Maksim Yevmenkin cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/netgraph ng_base.c src/sys/sys kernel.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 10 Jan 2005 20:28:20 -0000 Simon L. Nielsen wrote: > On 2005.01.10 10:41:39 -0800, John-Mark Gurney wrote: > >>Maksim Yevmenkin wrote this message on Thu, Jan 06, 2005 at 17:45 +0000: >> >>> This does not address the bigger problem: MODULE_DEPEND >>> does not seem to work when modules are compiled in the >>> kernel, but it fixes the problem with Netgraph Bluetooth >>> device drivers reported by a few folks. >> >>The real problem is that MODULE_DEPEND only documents other kernel >>dependancies, but does not reorder SYSINIT's... Considering that >>SYSINIT's are always suppose to be ordered properly (otherwise, what >>is the point of SYSINIT's) when staticly linked, having SYSINIT's not >>properly ordered is the only bug... >> >>If we move to a more complex and module aware system of SYSINIT's, then >>it might happen, but this will be very difficult considering that when >>staticly linked, you have no way to seperate which SYSINIT's belong to >>which modules... >> >>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? > > > > ------------------------------------------------------------------------ > > Index: MODULE_DEPEND.9 > =================================================================== > RCS file: /home/ncvs/src/share/man/man9/MODULE_DEPEND.9,v > retrieving revision 1.6 > diff -u -d -r1.6 MODULE_DEPEND.9 > --- MODULE_DEPEND.9 7 Jan 2005 08:44:31 -0000 1.6 > +++ MODULE_DEPEND.9 10 Jan 2005 20:23:23 -0000 > @@ -28,7 +28,7 @@ > .\" > .\" $FreeBSD: src/share/man/man9/MODULE_DEPEND.9,v 1.6 2005/01/07 08:44:31 keramida Exp $ > .\" > -.Dd January 7, 2005 > +.Dd January 10, 2005 > .Dt MODULE_DEPEND 9 > .Os > .Sh NAME > @@ -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. Scott