preserve interface main hotplug device on reload
authorFelix Fietkau <nbd@openwrt.org>
Sat, 3 Mar 2012 19:55:03 +0000 (20:55 +0100)
committerFelix Fietkau <nbd@openwrt.org>
Sat, 3 Mar 2012 19:55:03 +0000 (20:55 +0100)
device.c
interface.c

index 489c6601de8dfe076436a29615fe2ca7d70a5354..8f0e823373e75e27eeb53f656a122f393fa0d34a 100644 (file)
--- a/device.c
+++ b/device.c
@@ -465,6 +465,9 @@ device_set_disabled(struct device *dev, bool value)
 
 void device_add_user(struct device_user *dep, struct device *dev)
 {
+       if (dep->dev)
+               device_remove_user(dep);
+
        dep->dev = dev;
        list_add_tail(&dep->list, &dev->users);
        if (dep->cb && dev->present) {
index 2404878a3e03177a1ec02941c2b67738cab22fe3..6e3feebfa28dff2ea2d3687ba74aec877cc362aa 100644 (file)
@@ -195,7 +195,7 @@ interface_claim_device(struct interface *iface)
 
 
 static void
-interface_cleanup(struct interface *iface)
+interface_cleanup(struct interface *iface, bool reload)
 {
        struct interface_user *dep, *tmp;
 
@@ -205,7 +205,8 @@ interface_cleanup(struct interface *iface)
        interface_ip_flush(&iface->config_ip);
        interface_flush_state(iface);
        interface_clear_errors(iface);
-       if (iface->main_dev.dev)
+       if (iface->main_dev.dev &&
+           (!reload || !iface->main_dev.hotplug))
                device_remove_user(&iface->main_dev);
        iface->l3_dev = &iface->main_dev;
        interface_set_proto_state(iface, NULL);
@@ -214,7 +215,7 @@ interface_cleanup(struct interface *iface)
 static void
 interface_do_free(struct interface *iface)
 {
-       interface_cleanup(iface);
+       interface_cleanup(iface, false);
        free(iface->config);
        netifd_ubus_remove_interface(iface);
        avl_delete(&interfaces.avl, &iface->node.avl);
@@ -224,7 +225,7 @@ interface_do_free(struct interface *iface)
 static void
 interface_do_reload(struct interface *iface)
 {
-       interface_cleanup(iface);
+       interface_cleanup(iface, true);
        proto_init_interface(iface, iface->config);
        interface_claim_device(iface);
 }