Re: cvs commit: src/sys/netgraph/netgraph ng_base.c

[Hiten Pandya <hmp@xxxxxxxxxxxxx>]

Matthew Dillon wrote:
> :y0netan1    2005/09/03 23:58:24 PDT
> :
> :DragonFly src repository
> :
> :  Modified files:
> :    sys/netgraph/netgraph ng_base.c 
> :  Log:
> :  Avoid panic by not using linker_* API in the early stage of
> :  boot process, during which namei_zone is not initialized yet
> :  but linker_* API relies on it indirectly by calling nlookup_init().
> :  
> :  Revision  Changes    Path
> :  1.18      +18 -4     src/sys/netgraph/netgraph/ng_base.c
> :
> :
> :http://www.dragonflybsd.org/cvsweb/src/sys/netgraph/netgraph/ng_base.c.diff?r1=1.17&r2=1.18&f=u
>
>     Maybe the in-kernel loading for NG should be removed entirely in favor of 
>     more sophisticated userland 'auto' loading ?  I haven't researched it, but
>     NG's module loading is pretty high on my 'why is it trying to do it at all'
>     list :-)

It is not really netgraph's fault.  The fault lies with the stupid design 
of the linker and module API.  It needs to be a lot more flexible than it 
is right now.

I will do something about this once we finish the BUF/BIO work.  I have a 
few ideas for this part of the kernel.

				-Hiten