X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Fnetwork%2Fnetworkd-link.c;h=d41fe80f208e62f170bc2fe79799716729a1b3d6;hb=0617ffabe86a6d366252477eafbe59a888b149d4;hp=d0a3dccd9bc5a75eaa19013a2bf083c1ddb2e9af;hpb=8cd11a0f0f4ca05199e1166f6a07472b296f7455;p=elogind.git diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c index d0a3dccd9..d41fe80f2 100644 --- a/src/network/networkd-link.c +++ b/src/network/networkd-link.c @@ -28,7 +28,6 @@ int link_new(Manager *manager, struct udev_device *device, Link **ret) { _cleanup_link_free_ Link *link = NULL; - uint64_t ifindex; const char *mac; int r; @@ -39,8 +38,8 @@ int link_new(Manager *manager, struct udev_device *device, Link **ret) { if (!link) return -ENOMEM; - ifindex = udev_device_get_ifindex(device); - if (ifindex <= 0) + link->ifindex = udev_device_get_ifindex(device); + if (link->ifindex <= 0) return -EINVAL; mac = udev_device_get_sysattr_value(device, "address"); @@ -48,10 +47,10 @@ int link_new(Manager *manager, struct udev_device *device, Link **ret) { return -EINVAL; memcpy(&link->mac.ether_addr_octet[0], ether_aton(mac), ETH_ALEN); - link->ifindex = ifindex; link->manager = manager; + link->state = _LINK_STATE_INVALID; - r = hashmap_put(manager->links, &ifindex, link); + r = hashmap_put(manager->links, &link->ifindex, link); if (r < 0) return r; @@ -65,9 +64,9 @@ void link_free(Link *link) { if (!link) return; - network_free(link->network); + assert(link->manager); - hashmap_remove(link->manager->links, link); + hashmap_remove(link->manager->links, &link->ifindex); free(link); } @@ -103,23 +102,194 @@ int link_add(Manager *m, struct udev_device *device) { return 0; } -int link_up(Manager *manager, Link *link) { +static int link_enter_configured(Link *link) { + log_info("Link configured successfully."); + + link->state = LINK_STATE_CONFIGURED; + + return 0; +} + +static int link_enter_failed(Link *link) { + log_warning("Could not configure link."); + + link->state = LINK_STATE_FAILED; + + return 0; +} + +static bool link_is_up(Link *link) { + return link->flags & IFF_UP; +} + +static int link_enter_routes_set(Link *link) { + log_info("Routes set for link %u", (unsigned)link->ifindex); + + if (link_is_up(link)) + return link_enter_configured(link); + + link->state = LINK_STATE_ROUTES_SET; + + return 0; +} + +static int route_handler(sd_rtnl *rtnl, sd_rtnl_message *m, void *userdata) { + Link *link = userdata; + int r; + + assert(link->rtnl_messages > 0); + assert(link->state == LINK_STATE_SET_ROUTES || link->state == LINK_STATE_FAILED); + + link->rtnl_messages --; + + if (link->state == LINK_STATE_FAILED) + return 1; + + r = sd_rtnl_message_get_errno(m); + if (r < 0 && r != -EEXIST) { + log_warning("Could not set route on interface %u: %s", + (unsigned)link->ifindex, strerror(-r)); + return link_enter_failed(link); + } + + if (link->rtnl_messages == 0) + return link_enter_routes_set(link); + + return 1; +} + +static int link_enter_set_routes(Link *link) { + Route *route; + int r; + + assert(link); + assert(link->network); + assert(link->rtnl_messages == 0); + assert(link->state == LINK_STATE_ADDRESSES_SET); + + link->state = LINK_STATE_SET_ROUTES; + + if (!link->network->routes) + return link_enter_routes_set(link); + + LIST_FOREACH(routes, route, link->network->routes) { + r = route_configure(route, link, &route_handler); + if (r < 0) + link_enter_failed(link); + } + + return 0; +} + +static int link_enter_addresses_set(Link *link) { + log_info("Addresses set for link %u", (unsigned)link->ifindex); + + link->state = LINK_STATE_ADDRESSES_SET; + + return link_enter_set_routes(link); +} + +static int address_handler(sd_rtnl *rtnl, sd_rtnl_message *m, void *userdata) { + Link *link = userdata; + int r; + + assert(link->rtnl_messages > 0); + assert(link->state == LINK_STATE_SET_ADDRESSES || link->state == LINK_STATE_FAILED); + + link->rtnl_messages --; + + if (link->state == LINK_STATE_FAILED) + return 1; + + r = sd_rtnl_message_get_errno(m); + if (r < 0 && r != -EEXIST) { + log_warning("Could not set address on interface %u: %s", + (unsigned)link->ifindex, strerror(-r)); + link_enter_failed(link); + } + + if (link->rtnl_messages == 0) + link_enter_addresses_set(link); + + return 1; +} + +static int link_enter_set_addresses(Link *link) { + Address *address; + int r; + + assert(link); + assert(link->network); + assert(link->rtnl_messages == 0); + + if (!link->network->addresses) + return link_enter_addresses_set(link); + + link->state = LINK_STATE_SET_ADDRESSES; + + LIST_FOREACH(addresses, address, link->network->addresses) { + r = address_configure(address, link, &address_handler); + if (r < 0) + link_enter_failed(link); + } + + return 0; +} + +static int link_handler(sd_rtnl *rtnl, sd_rtnl_message *m, void *userdata) { + Link *link = userdata; + int r; + + r = sd_rtnl_message_get_errno(m); + if (r < 0) { + log_warning("Could not bring up interface %u: %s", + (unsigned)link->ifindex, strerror(-r)); + return link_enter_failed(link); + } + + link->flags |= IFF_UP; + + log_info("Link is UP."); + + if (link->state == LINK_STATE_ROUTES_SET) + return link_enter_configured(link); + + return 1; +} + +static int link_up(Link *link) { _cleanup_sd_rtnl_message_unref_ sd_rtnl_message *req = NULL; int r; + assert(link); + assert(link->manager); + assert(link->manager->rtnl); + r = sd_rtnl_message_link_new(RTM_NEWLINK, link->ifindex, 0, IFF_UP, &req); if (r < 0) { log_error("Could not allocate RTM_NEWLINK message"); return r; } - r = sd_rtnl_call(manager->rtnl, req, 0, NULL); + r = sd_rtnl_call_async(link->manager->rtnl, req, link_handler, link, 0, NULL); if (r < 0) { - log_error("Could not UP link: %s", strerror(-r)); + log_error("Could not send rtnetlink message: %s", strerror(-r)); return r; } - log_info("Link is UP"); + return 0; +} + +int link_configure(Link *link) { + int r; + + r = link_up(link); + if (r < 0) + return link_enter_failed(link); + + r = link_enter_set_addresses(link); + if (r < 0) + return link_enter_failed(link); return 0; }