Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 04 Jan 2005 13:20:57 -0800
From:      Maksim Yevmenkin <maksim.yevmenkin@savvis.net>
To:        current@freebsd.org
Cc:        Julian Elischer <julian@elischer.org>
Subject:   netgraph(3) initialization order
Message-ID:  <41DB08B9.6090801@savvis.net>

next in thread | raw e-mail | index | archive | help
Dear Hackers,

would anyone object if i change SI_ORDER_MIDDLE in the 
/sys/netgraph/ng_base.c:2994 to say SI_ORDER_THIRD, i.e.

change

DECLARE_MODULE(netgraph, netgraph_mod, SI_SUB_DRIVERS, SI_ORDER_MIDDLE);

to

DECLARE_MODULE(netgraph, netgraph_mod, SI_SUB_DRIVERS, SI_ORDER_THIRD);

the reason for this change is that bluetooth device drivers depend on 
netgraph(4) and when both netgraph(4) and bluetooth device driver (such 
as ng_ubt(4)) compiled in the kernel you get a crash. basically 
ng_ubt(4) mod_load callback is called before netgraph(4) mod_load 
callback and ng_findtype() crashes on uninitialized mutex (DEVICE_MODULE 
macro passes SI_SUB_DRIVERS, SI_ORDER_THIRD to the DECLARE_MODULE).


option #2 would be to have DEVICE_MODULE_ORDERED macro which accepts two 
extra parameters.


and finally option #3 would be to duplicate entire content of the 
DEVICE_MODULE macro in all bluetooth device drivers and specify order in 
DECLARE_MODULE macro.


any thoughts?

thanks,
max



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