Re: [PATCH] netns: fix net_alloc_generic()

Eric Dumazet [off-list ref] writes:

Le jeudi 26 janvier 2012 à 14:57 -0800, Eric W. Biederman a écrit :

quoted

It is not a chicken and egg problem.  It is a bug in caif.
caif is claiming to be a network device when it is acting as a subsytem.
That means it is being initialized too late.

Ah ok !

quoted

Untested but this should trivially fix the problem, and a bunch
of others of the same ilk.

Hmm, please refrain from using "trivially" or "trivial", you're not
fooling anyone.

All I meant is that the change was trivial.

Truth is this netns layer is horribly complex, since this CAIF bug
needed no more than four patch attempts and lastly your own work before
finding the root cause.

As for the complexity I don't know that it is noticeably worse than the
initialization complexity of the network stack in general.

I do think that it is non-obvious that serious initialization ordering
problems can be caused by such a small difference.  The non-locality
and of cause and effect, combined with unfamiliarity of the code seems
to be what hides problems like this.

Once you know that initialization ordering problems tend to registering
the wrong way.  Aka as a device instead of a subsys the solution to
problems like this tend to jump out at you.

Now the common plumbing in net/core/net_namespace.c does count as
complex.  The fact we missed such an obvious optimization opportunity
for so long seems to confirm that.

I am open for ideas on how to simply things.  

Eric