Eric W. Biederman wrote:
Mark Lord <[email protected]> writes:
Arjan van de Ven wrote:
On Sat, 01 Dec 2007 15:21:12 -0500
Mark Lord <[email protected]> wrote:
Eric W. Biederman wrote:
Stephen Hemminger <[email protected]> writes:
Sure. We keep the updated dev_get_by_XXXX that takes a network
namespace parameter.
..
And what should code be passing in when "# CONFIG_NET_NS is not set" ?
network drivers probably really really don't want to call
dev_get_by_XXX...
..
Fine. But all of them want to call sk_alloc(),
network drivers should be calling sk_alloc less then they should
call dev_get_by_XXXX. Only protocols call sk_alloc.
..
I think I saw some bridge code that calls it, too.
Regardless, it's EXPORT_SYMBOL(), but now unusable
due to the GPL-only symbol "init_net".
Meanwhile, you are avoiding answering the question:
And what should code be passing in when "# CONFIG_NET_NS is not set" ?
..
and many want to do register_netdev().
I haven't even touched register_netdev.
..
Bull-pucky. Somebody did:
@@ -3361,6 +3595,8 @@ int register_netdevice(struct net_device *dev)
/* When net_device's are persistent, this will be fatal. */
BUG_ON(dev->reg_state != NETREG_UNINITIALIZED);
+ BUG_ON(!dev->nd_net);
+ net = dev->nd_net;
spin_lock_init(&dev->queue_lock);
spin_lock_init(&dev->_xmit_lock);
..
That new BUG_ON() line complains if nd_net has not been initialized,
and the only thing I see drivers putting there is GPL-only "&init_net".
So what should they be using there ?
What are you having problems with?
..
Avoiding the question again there, too.
I personally am not having problems with anything.
But I noticed the API change, and would like somebody to fix it.
Any of the three ways (or a fourth, if you've got one) might do.
Cheers
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
