netns: Fix device renaming for sysfs
authorDaniel Lezcano <dlezcano@fr.ibm.com>
Sat, 3 May 2008 00:00:58 +0000 (17:00 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sat, 3 May 2008 00:00:58 +0000 (17:00 -0700)
commitaaf8cdc34ddba08122f02217d9d684e2f9f5d575
tree40746e08bc9e6b590c9e7d1f7b54597539737c22
parent705d209168681b4408d10fca7257de3343be573d
netns: Fix device renaming for sysfs

When a netdev is moved across namespaces with the
'dev_change_net_namespace' function, the 'device_rename' function is
used to fixup kobject and refresh the sysfs tree. The device_rename
function will call kobject_rename and this one will check if there is
an object with the same name and this is the case because we are
renaming the object with the same name.

The use of 'device_rename' seems for me wrong because we usually don't
rename it but just move it across namespaces. As we just want to do a
mini "netdev_[un]register", IMO the functions
'netdev_[un]register_kobject' should be used instead, like an usual
network device [un]registering.

This patch replace device_rename by netdev_unregister_kobject,
followed by netdev_register_kobject.

The netdev_register_kobject will call device_initialize and will raise
a warning indicating the device was already initialized. In order to
fix that, I split the device initialization into a separate function
and use it together with 'netdev_register_kobject' into
register_netdevice. So we can safely call 'netdev_register_kobject' in
'dev_change_net_namespace'.

This fix will allow to properly use the sysfs per namespace which is
coming from -mm tree.

Signed-off-by: Daniel Lezcano <dlezcano@fr.ibm.com>
Acked-by: Benjamin Thery <benjamin.thery@bull.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c
net/core/net-sysfs.c
net/core/net-sysfs.h