networkd: link - ignore missing MAC address

Currently DHCP/IPv4LL only works on ethernet devices, but no reason not to otherwise
manage them.

diff --git a/TODO b/TODO
index 66bf6f0545d537327e9562a8cf33141ab04ff8a0..375d129c5ceb24451d5c03e92332674a38bb8d63 100644 (file)
--- a/TODO
+++ b/TODO
@@ -615,13 +615,11 @@ Features:
    - add reduced [Link] support to .network files
    - add Scope= parsing option for [Network]
    - properly handle routerless dhcp leases
-   - set lifetime on the address acquired from dhcp
-   - patch kernel to support module alias for tunnel device (ipip/sit/gre),
-     then remove remove libkmod dependency and CAP_SYS_MODULE
    - add veth netdev support (c.f. http://shorewall.net/bridge-Shorewall-perl.html#veth)
    - add tun/tap netdev support
    - add more attribute support for SIT tunnel
    - make metric of routes configurable
+   - work with non-ethernet devices
 
 * networkd-wait-online:
    - make operstates to wait for configurable?

diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c
index 8674eee9487a8949ed05d1deb25e63fcddd9d1ad..ce9fbb5ed8d5241c75678970c65c7a5b67b42c2a 100644 (file)
--- a/src/network/networkd-link.c
+++ b/src/network/networkd-link.c
@@ -78,7 +78,7 @@ static int link_new(Manager *manager, sd_rtnl_message *message, Link **ret) {
 
         r = sd_rtnl_message_read_ether_addr(message, IFLA_ADDRESS, &link->mac);
         if (r < 0)
-                return r;
+                log_debug_link(link, "MAC address not found for new device, continuing without");
 
         r = asprintf(&link->state_file, "/run/systemd/netif/links/%"PRIu64,
                      link->ifindex);

diff --git a/src/network/networkd-manager.c b/src/network/networkd-manager.c
index c93d598c6bbc4a664d0f72c2f9c5ed16d4319f58..c1d50128a5d978cb13dd269d6251ee331730215b 100644 (file)
--- a/src/network/networkd-manager.c
+++ b/src/network/networkd-manager.c
@@ -292,7 +292,8 @@ static int manager_rtnl_process_link(sd_rtnl *rtnl, sd_rtnl_message *message, vo
                         /* link is new, so add it */
                         r = link_add(m, message, &link);
                         if (r < 0) {
-                                log_debug("could not add new link");
+                                log_debug("could not add new link: %s",
+                                           strerror(-r));
                                 return 0;
                         }
                 }