batman-adv: Avoid sysfs name collision for netns moves
authorSven Eckelmann <sven@narfation.org>
Mon, 13 Jun 2016 05:41:32 +0000 (07:41 +0200)
committerSimon Wunderlich <sw@simonwunderlich.de>
Tue, 9 Aug 2016 05:54:28 +0000 (07:54 +0200)
commitf4acb1086bc972f69e1ae14ec5b4473bc6c02dd5
tree12ce27ffd9d476342c13960d017e747d762188af
parent569c98504b2004dba207127bcceb8f3aeb9b8e7d
batman-adv: Avoid sysfs name collision for netns moves

The kobject_put is only removing the sysfs entry and corresponding entries
when its reference counter becomes zero. This tends to lead to collisions
when a device is moved between two different network namespaces because
some of the sysfs files have to be removed first and then added again to
the already moved sysfs entry.

    WARNING: CPU: 0 PID: 290 at lib/kobject.c:240 kobject_add_internal+0x5ec/0x8a0
    kobject_add_internal failed for batman_adv with -EEXIST, don't try to register things with the same name in the same directory.

But the caller of kobject_put can already remove the sysfs entry before it
does the kobject_put. This removal is done even when the reference counter
is not yet zero and thus avoids the problem.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
net/batman-adv/sysfs.c