From 933c70a0a4e4fac47d18e0348ae97ee3d48dc139 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Wed, 15 May 2019 16:02:06 +0900 Subject: [PATCH] network: honor MTUBytes= setting Closes #12552. --- src/network/networkd-dhcp4.c | 4 ++-- src/network/networkd-link.c | 6 +++--- src/network/networkd-link.h | 2 +- src/network/networkd-network.c | 30 +----------------------------- src/network/networkd-network.h | 1 - 5 files changed, 7 insertions(+), 36 deletions(-) diff --git a/src/network/networkd-dhcp4.c b/src/network/networkd-dhcp4.c index 81c694822c1..7907988a9bb 100644 --- a/src/network/networkd-dhcp4.c +++ b/src/network/networkd-dhcp4.c @@ -261,7 +261,7 @@ static int dhcp_lease_lost(Link *link) { r = sd_dhcp_lease_get_mtu(link->dhcp_lease, &mtu); if (r >= 0 && link->original_mtu != mtu) { - r = link_set_mtu(link, link->original_mtu, true); + r = link_set_mtu(link, link->original_mtu); if (r < 0) { log_link_warning(link, "DHCP error: could not reset MTU"); @@ -454,7 +454,7 @@ static int dhcp_lease_acquired(sd_dhcp_client *client, Link *link) { r = sd_dhcp_lease_get_mtu(lease, &mtu); if (r >= 0) { - r = link_set_mtu(link, mtu, true); + r = link_set_mtu(link, mtu); if (r < 0) log_link_error_errno(link, r, "Failed to set MTU to %" PRIu16 ": %m", mtu); } diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c index cd1b0325e8e..6c9a056d782 100644 --- a/src/network/networkd-link.c +++ b/src/network/networkd-link.c @@ -1383,7 +1383,7 @@ static int set_mtu_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *link) return 1; } -int link_set_mtu(Link *link, uint32_t mtu, bool force) { +int link_set_mtu(Link *link, uint32_t mtu) { _cleanup_(sd_netlink_message_unrefp) sd_netlink_message *req = NULL; int r; @@ -1394,7 +1394,7 @@ int link_set_mtu(Link *link, uint32_t mtu, bool force) { if (mtu == 0 || link->setting_mtu) return 0; - if (force ? link->mtu == mtu : link->mtu >= mtu) + if (link->mtu == mtu) return 0; log_link_debug(link, "Setting MTU: %" PRIu32, mtu); @@ -2634,7 +2634,7 @@ static int link_configure(Link *link) { return r; } - r = link_set_mtu(link, link->network->mtu, link->network->mtu_is_set); + r = link_set_mtu(link, link->network->mtu); if (r < 0) return r; diff --git a/src/network/networkd-link.h b/src/network/networkd-link.h index e1f4b23a5ac..cc4cd070755 100644 --- a/src/network/networkd-link.h +++ b/src/network/networkd-link.h @@ -158,7 +158,7 @@ bool link_has_carrier(Link *link); int link_ipv6ll_gained(Link *link, const struct in6_addr *address); -int link_set_mtu(Link *link, uint32_t mtu, bool force); +int link_set_mtu(Link *link, uint32_t mtu); int ipv4ll_configure(Link *link); bool link_ipv4ll_enabled(Link *link); diff --git a/src/network/networkd-network.c b/src/network/networkd-network.c index 56a45379c16..81d2b974edf 100644 --- a/src/network/networkd-network.c +++ b/src/network/networkd-network.c @@ -143,24 +143,6 @@ static int network_resolve_stacked_netdevs(Network *network) { return 0; } -static uint32_t network_get_stacked_netdevs_mtu(Network *network) { - uint32_t mtu = 0; - NetDev *dev; - Iterator i; - - HASHMAP_FOREACH(dev, network->stacked_netdevs, i) - if (dev->kind == NETDEV_KIND_VLAN && dev->mtu > 0) - /* See vlan_dev_change_mtu() in kernel. - * Note that the additional 4bytes may not be necessary for all devices. */ - mtu = MAX(mtu, dev->mtu + 4); - - else if (dev->kind == NETDEV_KIND_MACVLAN && dev->mtu > mtu) - /* See macvlan_change_mtu() in kernel. */ - mtu = dev->mtu; - - return mtu; -} - int network_verify(Network *network) { Address *address, *address_next; Route *route, *route_next; @@ -169,7 +151,6 @@ int network_verify(Network *network) { AddressLabel *label, *label_next; Prefix *prefix, *prefix_next; RoutingPolicyRule *rule, *rule_next; - uint32_t mtu; assert(network); assert(network->filename); @@ -252,16 +233,7 @@ int network_verify(Network *network) { if (network->ip_masquerade) network->ip_forward |= ADDRESS_FAMILY_IPV4; - network->mtu_is_set = network->mtu > 0; - mtu = network_get_stacked_netdevs_mtu(network); - if (network->mtu < mtu) { - if (network->mtu_is_set) - log_notice("%s: Bumping MTUBytes= from %"PRIu32" to %"PRIu32" because of stacked device", - network->filename, network->mtu, mtu); - network->mtu = mtu; - } - - if (network->mtu_is_set && network->dhcp_use_mtu) { + if (network->mtu > 0 && network->dhcp_use_mtu) { log_warning("%s: MTUBytes= in [Link] section and UseMTU= in [DHCP] section are set. " "Disabling UseMTU=.", network->filename); network->dhcp_use_mtu = false; diff --git a/src/network/networkd-network.h b/src/network/networkd-network.h index d00062fe599..702e864eafa 100644 --- a/src/network/networkd-network.h +++ b/src/network/networkd-network.h @@ -218,7 +218,6 @@ struct Network { struct ether_addr *mac; uint32_t mtu; - bool mtu_is_set; /* Indicate MTUBytes= is specified. */ int arp; int multicast; int allmulticast; -- 2.39.2