X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fnetifd.git;a=blobdiff_plain;f=interface.c;h=3820050eff038616e454250b47efecc30fe30700;hp=576d2a3a42848e7b8b8ca3ef9ddd408131ca489b;hb=2152229c45f470a14dee33507b331ac38055e062;hpb=fd50535d7d4060d3cb80f1c317a087507ecfdf2b

diff --git a/interface.c b/interface.c
index 576d2a3..3820050 100644
--- a/interface.c
+++ b/interface.c
@@ -31,7 +31,7 @@ const struct config_param_list interface_attr_list = {
 };
 
 static void
-clear_interface_errors(struct interface *iface)
+interface_clear_errors(struct interface *iface)
 {
 	struct interface_error *error, *tmp;
 
@@ -78,7 +78,7 @@ void interface_add_error(struct interface *iface, const char *subsystem,
 static void
 interface_event(struct interface *iface, enum interface_event ev)
 {
-	/* TODO */
+	interface_queue_event(iface, ev);
 }
 
 static void
@@ -119,7 +119,7 @@ __interface_set_up(struct interface *iface)
 static void
 __interface_set_down(struct interface *iface, bool force)
 {
-	clear_interface_errors(iface);
+	interface_clear_errors(iface);
 
 	if (iface->state == IFS_DOWN ||
 		iface->state == IFS_TEARDOWN)
@@ -157,6 +157,7 @@ interface_set_available(struct interface *iface, bool new_state)
 	if (iface->available == new_state)
 		return;
 
+	D(INTERFACE, "Interface '%s', available=%d\n", iface->name, new_state);
 	iface->available = new_state;
 
 	if (new_state) {
@@ -167,6 +168,66 @@ interface_set_available(struct interface *iface, bool new_state)
 }
 
 static void
+interface_claim_device(struct interface *iface)
+{
+	struct device *dev;
+
+	if (iface->proto_handler &&
+		!(iface->proto_handler->flags & PROTO_FLAG_NODEV)) {
+		dev = device_get(iface->ifname, true);
+		if (dev)
+			device_add_user(&iface->main_dev, dev);
+	}
+}
+
+
+static void
+interface_cleanup(struct interface *iface)
+{
+	interface_clear_errors(iface);
+	if (iface->main_dev.dev)
+		device_remove_user(&iface->main_dev);
+	interface_set_proto_state(iface, NULL);
+}
+
+static void
+interface_do_free(struct interface *iface)
+{
+	interface_cleanup(iface);
+	free(iface->config);
+	netifd_ubus_remove_interface(iface);
+	avl_delete(&interfaces.avl, &iface->node.avl);
+	free(iface);
+}
+
+static void
+interface_do_reload(struct interface *iface)
+{
+	interface_cleanup(iface);
+
+	interface_claim_device(iface);
+	proto_init_interface(iface, iface->config);
+
+	if (iface->autostart && !config_init)
+		interface_set_up(iface);
+}
+
+static void
+interface_handle_config_change(struct interface *iface)
+{
+	switch(iface->config_state) {
+	case IFC_NORMAL:
+		break;
+	case IFC_RELOAD:
+		interface_do_reload(iface);
+		break;
+	case IFC_REMOVE:
+		interface_do_free(iface);
+		break;
+	}
+}
+
+static void
 interface_proto_cb(struct interface_proto_state *state, enum interface_proto_event ev)
 {
 	struct interface *iface = state->iface;
@@ -184,6 +245,7 @@ interface_proto_cb(struct interface_proto_state *state, enum interface_proto_eve
 			return;
 
 		mark_interface_down(iface);
+		interface_handle_config_change(iface);
 		break;
 	case IFPEV_LINK_LOST:
 		if (iface->state != IFS_UP)
@@ -198,7 +260,6 @@ interface_proto_cb(struct interface_proto_state *state, enum interface_proto_eve
 void interface_set_proto_state(struct interface *iface, struct interface_proto_state *state)
 {
 	if (iface->proto) {
-		interface_proto_event(iface->proto, PROTO_CMD_TEARDOWN, true);
 		iface->proto->free(iface->proto);
 		iface->proto = NULL;
 	}
@@ -217,11 +278,11 @@ interface_init(struct interface *iface, const char *name,
 {
 	struct blob_attr *tb[IFACE_ATTR_MAX];
 	struct blob_attr *cur;
-	struct device *dev;
 	const char *proto_name = NULL;
 
 	strncpy(iface->name, name, sizeof(iface->name) - 1);
 	INIT_LIST_HEAD(&iface->errors);
+	INIT_LIST_HEAD(&iface->hotplug_list);
 
 	iface->main_dev.cb = interface_cb;
 	iface->l3_dev = &iface->main_dev;
@@ -234,23 +295,25 @@ interface_init(struct interface *iface, const char *name,
 
 	proto_attach_interface(iface, proto_name);
 
-	if (iface->proto_handler &&
-	    !(iface->proto_handler->flags & PROTO_FLAG_NODEV) &&
-	    (cur = tb[IFACE_ATTR_IFNAME])) {
-		dev = device_get(blobmsg_data(cur), true);
-		if (dev)
-			device_add_user(&iface->main_dev, dev);
-	}
-
 	if ((cur = tb[IFACE_ATTR_AUTO]))
 		iface->autostart = blobmsg_get_bool(cur);
 	else
 		iface->autostart = true;
+	iface->config_autostart = iface->autostart;
 }
 
 void
 interface_add(struct interface *iface, struct blob_attr *config)
 {
+	struct blob_attr *tb[IFACE_ATTR_MAX];
+	struct blob_attr *cur;
+
+	blobmsg_parse(iface_attrs, IFACE_ATTR_MAX, tb,
+		      blob_data(config), blob_len(config));
+
+	if ((cur = tb[IFACE_ATTR_IFNAME]))
+		iface->ifname = blobmsg_data(cur);
+
 	iface->config = config;
 	vlist_add(&interfaces, &iface->node);
 }
@@ -326,21 +389,73 @@ interface_start_pending(void)
 }
 
 static void
+set_config_state(struct interface *iface, enum interface_config_state s)
+{
+	iface->config_state = s;
+	if (iface->state == IFS_DOWN)
+		interface_handle_config_change(iface);
+	else
+		__interface_set_down(iface, false);
+}
+
+static void
+interface_change_config(struct interface *if_old, struct interface *if_new)
+{
+	struct blob_attr *old_config = if_old->config;
+	const char *old_ifname = if_old->ifname;
+	const struct proto_handler *proto = if_old->proto_handler;
+
+	interface_clear_errors(if_old);
+	if_old->config = if_new->config;
+	if (!if_old->config_autostart && if_new->config_autostart)
+		if_old->autostart = true;
+
+	if_old->config_autostart = if_new->config_autostart;
+	if_old->ifname = if_new->ifname;
+	if_old->proto_handler = if_new->proto_handler;
+
+	if (strcmp(old_ifname, if_new->ifname) != 0 ||
+		proto != if_new->proto_handler) {
+		D(INTERFACE, "Reload interface '%s' because of ifname/proto change\n",
+		  if_old->name);
+		goto reload;
+	}
+
+	if (!proto->config_params)
+		D(INTERFACE, "No config parameters for interface '%s'\n",
+		  if_old->name);
+	else if (!config_check_equal(old_config, if_new->config,
+				proto->config_params)) {
+		D(INTERFACE, "Reload interface '%s because of config changes\n",
+		  if_old->name);
+		goto reload;
+	}
+
+	goto out;
+
+reload:
+	set_config_state(if_old, IFC_RELOAD);
+out:
+	free(old_config);
+	free(if_new);
+}
+
+static void
 interface_update(struct vlist_tree *tree, struct vlist_node *node_new,
 		 struct vlist_node *node_old)
 {
 	struct interface *if_old = container_of(node_old, struct interface, node);
 	struct interface *if_new = container_of(node_new, struct interface, node);
 
-	if (node_old) {
-		free(if_old->config);
-		netifd_ubus_remove_interface(if_old);
-		if (if_old->proto->free)
-			if_old->proto->free(if_old->proto);
-		free(if_old);
-	}
-
-	if (node_new) {
+	if (node_old && node_new) {
+		D(INTERFACE, "Update interface '%s'\n", if_new->name);
+		interface_change_config(if_old, if_new);
+	} else if (node_old) {
+		D(INTERFACE, "Remove interface '%s'\n", if_old->name);
+		set_config_state(if_old, IFC_REMOVE);
+	} else if (node_new) {
+		D(INTERFACE, "Create interface '%s'\n", if_new->name);
+		interface_claim_device(if_new);
 		proto_init_interface(if_new, if_new->config);
 		interface_ip_init(if_new);
 		netifd_ubus_add_interface(if_new);
@@ -353,4 +468,6 @@ interface_init_list(void)
 {
 	vlist_init(&interfaces, avl_strcmp, interface_update,
 		   struct interface, node, name);
+	interfaces.keep_old = true;
+	interfaces.no_delete = true;
 }