network: drop duplicated link_up_can()
authorYu Watanabe <watanabe.yu+github@gmail.com>
Thu, 29 Apr 2021 21:57:15 +0000 (06:57 +0900)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Sat, 15 May 2021 18:11:23 +0000 (20:11 +0200)
(cherry picked from commit f282ce20aaa767f5395b065f2be587cdef3e5491)
(cherry picked from commit 179a92c9d8b2df688764095ce1d242de20ee72fa)
(cherry picked from commit d1e06f70368ac4b08c2804f3ecfa7cb3ed874fac)

src/network/networkd-can.c
src/network/networkd-link.c
src/network/networkd-link.h

index e5504f77381c9d4405466bbd8217ae29e9eea27e..c8a23d5b70379a9643905eabdc74ee814592f236 100644 (file)
@@ -52,48 +52,6 @@ int config_parse_can_bitrate(
         return 0;
 }
 
-static int link_up_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *link) {
-        int r;
-
-        assert(link);
-
-        if (IN_SET(link->state, LINK_STATE_FAILED, LINK_STATE_LINGER))
-                return 1;
-
-        r = sd_netlink_message_get_errno(m);
-        if (r < 0)
-                /* we warn but don't fail the link, as it may be brought up later */
-                log_link_message_warning_errno(link, m, r, "Could not bring up interface");
-
-        return 1;
-}
-
-static int link_up_can(Link *link) {
-        _cleanup_(sd_netlink_message_unrefp) sd_netlink_message *req = NULL;
-        int r;
-
-        assert(link);
-
-        log_link_debug(link, "Bringing CAN link up");
-
-        r = sd_rtnl_message_new_link(link->manager->rtnl, &req, RTM_SETLINK, link->ifindex);
-        if (r < 0)
-                return log_link_error_errno(link, r, "Could not allocate RTM_SETLINK message: %m");
-
-        r = sd_rtnl_message_link_set_flags(req, IFF_UP, IFF_UP);
-        if (r < 0)
-                return log_link_error_errno(link, r, "Could not set link flags: %m");
-
-        r = netlink_call_async(link->manager->rtnl, NULL, req, link_up_handler,
-                               link_netlink_destroy_callback, link);
-        if (r < 0)
-                return log_link_error_errno(link, r, "Could not send rtnetlink message: %m");
-
-        link_ref(link);
-
-        return 0;
-}
-
 static int link_set_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *link) {
         int r;
 
@@ -252,7 +210,7 @@ static int link_set_can(Link *link) {
         link_ref(link);
 
         if (!(link->flags & IFF_UP))
-                return link_up_can(link);
+                return link_up(link);
 
         return 0;
 }
@@ -304,7 +262,7 @@ int link_configure_can(Link *link) {
         }
 
         if (!(link->flags & IFF_UP)) {
-                r = link_up_can(link);
+                r = link_up(link);
                 if (r < 0) {
                         link_enter_failed(link);
                         return r;
index c767c96142310cb5a9e14b32026952a30e837a82..1108d922515a39350da4f07d905dfe0ead5c6a43 100644 (file)
@@ -1895,7 +1895,7 @@ static int link_up_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *link)
         return 1;
 }
 
-static int link_up(Link *link) {
+int link_up(Link *link) {
         _cleanup_(sd_netlink_message_unrefp) sd_netlink_message *req = NULL;
         int r;
 
index a0c5661149fee4d1e40d17b8831bbdd2a1fe5e8e..3a02e895acf5e82c97381dcbeb867a9e3e2f7109 100644 (file)
@@ -205,6 +205,7 @@ int link_get(Manager *m, int ifindex, Link **ret);
 int link_add(Manager *manager, sd_netlink_message *message, Link **ret);
 void link_drop(Link *link);
 
+int link_up(Link *link);
 int link_down(Link *link, link_netlink_message_handler_t callback);
 
 void link_enter_failed(Link *link);