* [PATCH ipsec] vti6: Change minimum MTU to IPV4_MIN_MTU, vti6 can carry IPv4 too
@ 2018-04-26 17:39 Stefano Brivio
2018-04-30 5:59 ` Steffen Klassert
0 siblings, 1 reply; 2+ messages in thread
From: Stefano Brivio @ 2018-04-26 17:39 UTC (permalink / raw)
To: Steffen Klassert
Cc: Xin Long, Alexey Kodanev, Jarod Wilson, Sabrina Dubroca, netdev
A vti6 interface can carry IPv4 as well, so it makes no sense to
enforce a minimum MTU of IPV6_MIN_MTU.
If the user sets an MTU below IPV6_MIN_MTU, IPv6 will be
disabled on the interface, courtesy of addrconf_notify().
Reported-by: Xin Long <lucien.xin@gmail.com>
Fixes: b96f9afee4eb ("ipv4/6: use core net MTU range checking")
Fixes: c6741fbed6dc ("vti6: Properly adjust vti6 MTU from MTU of lower device")
Fixes: 53c81e95df17 ("ip6_vti: adjust vti mtu according to mtu of lower device")
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
---
net/ipv6/ip6_vti.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/ipv6/ip6_vti.c b/net/ipv6/ip6_vti.c
index c214ffec02f0..ca957dd93a29 100644
--- a/net/ipv6/ip6_vti.c
+++ b/net/ipv6/ip6_vti.c
@@ -669,7 +669,7 @@ static void vti6_link_config(struct ip6_tnl *t, bool keep_mtu)
else
mtu = ETH_DATA_LEN - LL_MAX_HEADER - sizeof(struct ipv6hdr);
- dev->mtu = max_t(int, mtu, IPV6_MIN_MTU);
+ dev->mtu = max_t(int, mtu, IPV4_MIN_MTU);
}
/**
@@ -881,7 +881,7 @@ static void vti6_dev_setup(struct net_device *dev)
dev->priv_destructor = vti6_dev_free;
dev->type = ARPHRD_TUNNEL6;
- dev->min_mtu = IPV6_MIN_MTU;
+ dev->min_mtu = IPV4_MIN_MTU;
dev->max_mtu = IP_MAX_MTU - sizeof(struct ipv6hdr);
dev->flags |= IFF_NOARP;
dev->addr_len = sizeof(struct in6_addr);
--
2.15.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH ipsec] vti6: Change minimum MTU to IPV4_MIN_MTU, vti6 can carry IPv4 too
2018-04-26 17:39 [PATCH ipsec] vti6: Change minimum MTU to IPV4_MIN_MTU, vti6 can carry IPv4 too Stefano Brivio
@ 2018-04-30 5:59 ` Steffen Klassert
0 siblings, 0 replies; 2+ messages in thread
From: Steffen Klassert @ 2018-04-30 5:59 UTC (permalink / raw)
To: Stefano Brivio
Cc: Xin Long, Alexey Kodanev, Jarod Wilson, Sabrina Dubroca, netdev
On Thu, Apr 26, 2018 at 07:39:09PM +0200, Stefano Brivio wrote:
> A vti6 interface can carry IPv4 as well, so it makes no sense to
> enforce a minimum MTU of IPV6_MIN_MTU.
>
> If the user sets an MTU below IPV6_MIN_MTU, IPv6 will be
> disabled on the interface, courtesy of addrconf_notify().
>
> Reported-by: Xin Long <lucien.xin@gmail.com>
> Fixes: b96f9afee4eb ("ipv4/6: use core net MTU range checking")
> Fixes: c6741fbed6dc ("vti6: Properly adjust vti6 MTU from MTU of lower device")
> Fixes: 53c81e95df17 ("ip6_vti: adjust vti mtu according to mtu of lower device")
> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Applied, thanks Stefano!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-04-30 5:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-26 17:39 [PATCH ipsec] vti6: Change minimum MTU to IPV4_MIN_MTU, vti6 can carry IPv4 too Stefano Brivio
2018-04-30 5:59 ` Steffen Klassert
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.