• Maciej Żenczykowski's avatar
    xfrm: do not set IPv4 DF flag when encapsulating IPv6 frames <= 1280 bytes. · 6821ad87
    Maciej Żenczykowski authored
    One may want to have DF set on large packets to support discovering
    path mtu and limiting the size of generated packets (hence not
    setting the XFRM_STATE_NOPMTUDISC tunnel flag), while still
    supporting networks that are incapable of carrying even minimal
    sized IPv6 frames (post encapsulation).
    
    Having IPv4 Don't Frag bit set on encapsulated IPv6 frames that
    are not larger than the minimum IPv6 mtu of 1280 isn't useful,
    because the resulting ICMP Fragmentation Required error isn't
    actionable (even assuming you receive it) because IPv6 will not
    drop it's path mtu below 1280 anyway.  While the IPv4 stack
    could prefrag the packets post encap, this requires the ICMP
    error to be successfully delivered and causes a loss of the
    original IPv6 frame (thus requiring a retransmit and latency
    hit).  Luckily with IPv4 if we simply don't set the DF flag,
    we'll just make further fragmenting the packets some other
    router's problems.
    
    We'll still learn the correct IPv4 path mtu through encapsulation
    of larger IPv6 frames.
    
    I'm still not convinced this patch is entirely sufficient to make
    everything happy... but I don't see how it could possibly
    make things worse.
    
    See also recent:
      4ff2980b 'xfrm: fix tunnel model fragmentation behavior'
    and friends
    
    Cc: Lorenzo Colitti <lorenzo@google.com>
    Cc: Eric Dumazet <edumazet@google.com>
    Cc: Lina Wang <lina.wang@mediatek.com>
    Cc: Steffen Klassert <steffen.klassert@secunet.com>
    Signed-off-by: default avatarMaciej Zenczykowski <maze@google.com>
    Signed-off-by: default avatarSteffen Klassert <steffen.klassert@secunet.com>
    6821ad87
xfrm_output.c 21.2 KB