> > * If we're going to include the ip_*register_hook() API, why do > > we omit ipobs_init(), ipobs_fini() and ipobs_hook()? > > Because those aren't an API, but functions internal to the ip module.
They're still an API even if they're internal. Personally, I don't see much difference between ipnet.c calling ipobs_register_hook() and tcp.c calling ipobs_hook(); having tcp.c compiled into ip but ipnet.c depend on ip seems an implementation matter, not an architectural one. > > * Why are we classifying the net_* and NE_* stuff as Committed? > > That is because the rest of the netinfo API is Committed. I find that a weak justification. Just because part of an API is Committed doesn't mean all parts of it need to be. -- meem
