From 9aac5923811e0240526f2ef58d6e352bff195740 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Wed, 23 May 2012 22:13:43 +0200 Subject: [PATCH] add interface alias support --- config.c | 35 ++++++++++++---- config/network | 7 ++++ interface.c | 110 ++++++++++++++++++++++++++++++++++++++++++------- interface.h | 4 ++ 4 files changed, 132 insertions(+), 24 deletions(-) diff --git a/config.c b/config.c index 5687b4c..c5a2d2c 100644 --- a/config.c +++ b/config.c @@ -156,16 +156,17 @@ config_parse_bridge_interface(struct uci_section *s) } static void -config_parse_interface(struct uci_section *s) +config_parse_interface(struct uci_section *s, bool alias) { struct interface *iface; - const char *type; + const char *type = NULL; struct blob_attr *config; struct device *dev; blob_buf_init(&b, 0); - type = uci_lookup_option_string(uci_ctx, s, "type"); + if (!alias) + type = uci_lookup_option_string(uci_ctx, s, "type"); if (type && !strcmp(type, "bridge")) if (config_parse_bridge_interface(s)) return; @@ -181,13 +182,17 @@ config_parse_interface(struct uci_section *s) uci_to_blob(&b, s, iface->proto_handler->config_params); config = malloc(blob_pad_len(b.head)); - if (!config) { - free(iface); - return; - } + if (!config) + goto error; memcpy(config, b.head, blob_pad_len(b.head)); - interface_add(iface, config); + + if (alias) { + if (!interface_add_alias(iface, config)) + goto error_free_config; + } else { + interface_add(iface, config); + } /* * need to look up the interface name again, in case of config update, @@ -207,6 +212,11 @@ config_parse_interface(struct uci_section *s) return; device_set_config(dev, dev->type, b.head); + return; +error_free_config: + free(config); +error: + free(iface); } static void @@ -378,7 +388,14 @@ config_init_interfaces(void) struct uci_section *s = uci_to_section(e); if (!strcmp(s->type, "interface")) - config_parse_interface(s); + config_parse_interface(s, false); + } + + uci_foreach_element(&uci_network->sections, e) { + struct uci_section *s = uci_to_section(e); + + if (!strcmp(s->type, "alias")) + config_parse_interface(s, true); } } diff --git a/config/network b/config/network index 0fa7af4..3e26967 100644 --- a/config/network +++ b/config/network @@ -42,6 +42,13 @@ config interface wan option username foo option password bar +config alias wan1 + option proto static + option interface wan + option ipaddr 192.168.99.1 + option netmask 255.255.255.0 + option gateway 192.168.99.3 + config interface wlan option proto static option ipaddr 192.168.2.1 diff --git a/interface.c b/interface.c index 8f5fa98..3215f49 100644 --- a/interface.c +++ b/interface.c @@ -36,6 +36,7 @@ enum { IFACE_ATTR_DNS, IFACE_ATTR_DNS_SEARCH, IFACE_ATTR_METRIC, + IFACE_ATTR_INTERFACE, IFACE_ATTR_MAX }; @@ -48,6 +49,7 @@ static const struct blobmsg_policy iface_attrs[IFACE_ATTR_MAX] = { [IFACE_ATTR_METRIC] = { .name = "metric", .type = BLOBMSG_TYPE_INT32 }, [IFACE_ATTR_DNS] = { .name = "dns", .type = BLOBMSG_TYPE_ARRAY }, [IFACE_ATTR_DNS_SEARCH] = { .name = "dns_search", .type = BLOBMSG_TYPE_ARRAY }, + [IFACE_ATTR_INTERFACE] = { .name = "interface", .type = BLOBMSG_TYPE_STRING }, }; static const union config_param_info iface_attr_info[IFACE_ATTR_MAX] = { @@ -166,13 +168,15 @@ interface_flush_state(struct interface *iface) static void mark_interface_down(struct interface *iface) { - if (iface->state == IFS_UP) + enum interface_state state = iface->state; + + iface->state = IFS_DOWN; + if (state == IFS_UP) interface_event(iface, IFEV_DOWN); interface_ip_set_enabled(&iface->config_ip, false); interface_ip_flush(&iface->proto_ip); interface_flush_state(iface); system_flush_routes(); - iface->state = IFS_DOWN; } void @@ -250,17 +254,53 @@ interface_remove_user(struct interface_user *dep) dep->iface = NULL; } +static void +interface_alias_cb(struct interface_user *dep, struct interface *iface, enum interface_event ev) +{ + struct interface *alias = container_of(dep, struct interface, parent_iface); + struct device *dev = iface->l3_dev.dev; + + switch (ev) { + case IFEV_UP: + if (!dev) + return; + + interface_set_main_dev(alias, dev); + interface_set_available(alias, true); + break; + case IFEV_DOWN: + interface_set_available(alias, false); + interface_set_main_dev(alias, NULL); + break; + case IFEV_FREE: + interface_remove_user(dep); + break; + case IFEV_RELOAD: + break; + } +} + static void interface_claim_device(struct interface *iface) { - struct device *dev; + struct interface *parent; + struct device *dev = NULL; + + if (iface->parent_iface.iface) + interface_remove_user(&iface->parent_iface); - if (iface->ifname && + if (iface->parent_ifname) { + parent = vlist_find(&interfaces, iface->parent_ifname, parent, node); + iface->parent_iface.cb = interface_alias_cb; + interface_add_user(&iface->parent_iface, parent); + } else if (iface->ifname && !(iface->proto_handler->flags & PROTO_FLAG_NODEV)) { dev = device_get(iface->ifname, true); - if (dev) - interface_set_main_dev(iface, dev); } + + if (dev) + interface_set_main_dev(iface, dev); + if (iface->proto_handler->flags & PROTO_FLAG_INIT_AVAILABLE) interface_set_available(iface, true); } @@ -271,14 +311,17 @@ interface_cleanup(struct interface *iface, bool reload) { struct interface_user *dep, *tmp; + if (iface->parent_iface.iface) + interface_remove_user(&iface->parent_iface); + list_for_each_entry_safe(dep, tmp, &iface->users, list) interface_remove_user(dep); interface_ip_flush(&iface->config_ip); interface_flush_state(iface); interface_clear_errors(iface); - if (iface->main_dev.dev && - (!reload || !iface->main_dev.hotplug)) + + if (iface->main_dev.dev && !reload) interface_set_main_dev(iface, NULL); interface_set_proto_state(iface, NULL); } @@ -419,8 +462,7 @@ interface_init(struct interface *iface, const char *name, iface->config_autostart = iface->autostart; } -void -interface_add(struct interface *iface, struct blob_attr *config) +static bool __interface_add(struct interface *iface, struct blob_attr *config, bool alias) { struct blob_attr *tb[IFACE_ATTR_MAX]; struct blob_attr *cur; @@ -428,11 +470,36 @@ interface_add(struct interface *iface, struct blob_attr *config) 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); + if (alias) { + if ((cur = tb[IFACE_ATTR_INTERFACE])) + iface->parent_ifname = blobmsg_data(cur); + + if (!iface->parent_ifname) + return false; + } else { + if ((cur = tb[IFACE_ATTR_IFNAME])) + iface->ifname = blobmsg_data(cur); + } + iface->config = config; vlist_add(&interfaces, &iface->node, iface->name); + return true; +} + +void +interface_add(struct interface *iface, struct blob_attr *config) +{ + __interface_add(iface, config, false); +} + +bool +interface_add_alias(struct interface *iface, struct blob_attr *config) +{ + if (iface->proto_handler->flags & PROTO_FLAG_NODEV) + return false; + + return __interface_add(iface, config, true); } void @@ -607,6 +674,7 @@ 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 char *old_parent_ifname = if_old->parent_ifname; const struct proto_handler *proto = if_old->proto_handler; interface_clear_errors(if_old); @@ -616,11 +684,21 @@ interface_change_config(struct interface *if_old, struct interface *if_new) if_old->config_autostart = if_new->config_autostart; if_old->ifname = if_new->ifname; + if_old->parent_ifname = if_new->parent_ifname; if_old->proto_handler = if_new->proto_handler; - if ((!!old_ifname != !!if_new->ifname) || - (old_ifname && strcmp(old_ifname, if_new->ifname) != 0) || - proto != if_new->proto_handler) { +#define FIELD_CHANGED_STR(field) \ + ((!!if_old->field != !!old_ ## field) || \ + (old_ ## field && \ + strcmp(old_ ## field, if_old->field) != 0)) + + if (FIELD_CHANGED_STR(parent_ifname)) { + if (if_old->parent_iface.iface) + interface_remove_user(&if_old->parent_iface); + goto reload; + } + + if (FIELD_CHANGED_STR(ifname) || proto != if_new->proto_handler) { D(INTERFACE, "Reload interface '%s' because of ifname/proto change\n", if_old->name); goto reload; @@ -660,6 +738,8 @@ interface_change_config(struct interface *if_old, struct interface *if_new) reload: set_config_state(if_old, IFC_RELOAD); out: + if_new->config = NULL; + interface_cleanup(if_new, false); free(old_config); free(if_new); } diff --git a/interface.h b/interface.h index 8c7b347..6724bb9 100644 --- a/interface.h +++ b/interface.h @@ -93,6 +93,9 @@ struct interface { struct list_head users; + const char *parent_ifname; + struct interface_user parent_iface; + /* main interface that the interface is bound to */ struct device_user main_dev; @@ -128,6 +131,7 @@ void interface_init(struct interface *iface, const char *name, struct blob_attr *config); void interface_add(struct interface *iface, struct blob_attr *config); +bool interface_add_alias(struct interface *iface, struct blob_attr *config); void interface_set_proto_state(struct interface *iface, struct interface_proto_state *state); -- 2.25.1