We try to get all events for all net_devices to be able to add special sysfs folders for the batman-adv configuration. This also includes such events like NETDEV_POST_INIT which has no valid kobject according to v2.6.32-rc3-13-g7ffbe3f. This would create an oops in that situation.
We also drop NETDEV_UNREGISTER as event which may register a new batman_if because it would be destroyed after its initialisation.
Signed-off-by: Sven Eckelmann sven.eckelmann@gmx.de Cc: stable stable@kernel.org --- batman-adv/hard-interface.c | 19 +++++++++++++------ 1 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/batman-adv/hard-interface.c b/batman-adv/hard-interface.c index 1a829be..09678a6 100644 --- a/batman-adv/hard-interface.c +++ b/batman-adv/hard-interface.c @@ -457,15 +457,24 @@ static int hard_if_event(struct notifier_block *this, struct batman_if *batman_if = get_batman_if_by_netdev(net_dev); struct bat_priv *bat_priv;
- if (!batman_if) - batman_if = hardif_add_interface(net_dev); + if (!batman_if) { + switch (event) { + case NETDEV_REGISTER: + case NETDEV_UP: + case NETDEV_GOING_DOWN: + case NETDEV_DOWN: + case NETDEV_CHANGENAME: + case NETDEV_CHANGEADDR: + batman_if = hardif_add_interface(net_dev); + default: + break; + } + }
if (!batman_if) goto out;
switch (event) { - case NETDEV_REGISTER: - break; case NETDEV_UP: hardif_activate_interface(batman_if); break; @@ -476,8 +485,6 @@ static int hard_if_event(struct notifier_block *this, case NETDEV_UNREGISTER: hardif_remove_interface(batman_if); break; - case NETDEV_CHANGENAME: - break; case NETDEV_CHANGEADDR: if (batman_if->if_status == IF_NOT_IN_USE) goto out;
This patch will be replaced by a new one in a larger patch queue. It is not "wrong", but can be done a little bit different.
thanks, Sven
b.a.t.m.a.n@lists.open-mesh.org