]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
network: drop misleading debugging logs about MTU
authorYu Watanabe <watanabe.yu+github@gmail.com>
Mon, 14 Jun 2021 06:43:43 +0000 (15:43 +0900)
committerLuca Boccassi <luca.boccassi@gmail.com>
Mon, 14 Jun 2021 13:19:02 +0000 (14:19 +0100)
This fixes the following spurious logs on enumerating links:
```
wlan0: Saved original MTU 1500 (min: 256, max: 2304)
wlan0: MTU is changed: 0 → 1500 (min: 256, max: 2304)
```

src/network/networkd-link.c

index 911fb9b302c2581d1b99f3df261aa5a101ddc1d1..27ad2c03a22024b92b48b2fdff3e19ac8b28b293 100644 (file)
@@ -2038,6 +2038,8 @@ static int link_update_mtu(Link *link, sd_netlink_message *message) {
                 return 0;
         if (r < 0)
                 return log_link_debug_errno(link, r, "rtnl: failed to read MTU in RTM_NEWLINK message: %m");
+        if (mtu == 0)
+                return 0;
 
         r = sd_netlink_message_read_u32(message, IFLA_MIN_MTU, &min_mtu);
         if (r < 0 && r != -ENODATA)
@@ -2047,9 +2049,6 @@ static int link_update_mtu(Link *link, sd_netlink_message *message) {
         if (r < 0 && r != -ENODATA)
                 return log_link_debug_errno(link, r, "rtnl: failed to read maximum MTU in RTM_NEWLINK message: %m");
 
-        if (mtu == 0)
-                return 0;
-
         if (max_mtu == 0)
                 max_mtu = UINT32_MAX;
 
@@ -2065,8 +2064,9 @@ static int link_update_mtu(Link *link, sd_netlink_message *message) {
         if (link->mtu == mtu)
                 return 0;
 
-        log_link_debug(link, "MTU is changed: %"PRIu32" → %"PRIu32" (min: %"PRIu32", max: %"PRIu32")",
-                       link->mtu, mtu, link->min_mtu, link->max_mtu);
+        if (link->mtu != 0)
+                log_link_debug(link, "MTU is changed: %"PRIu32" → %"PRIu32" (min: %"PRIu32", max: %"PRIu32")",
+                               link->mtu, mtu, link->min_mtu, link->max_mtu);
 
         link->mtu = mtu;