]> git.baikalelectronics.ru Git - kernel.git/commit
ipv4: fix DO and PROBE pmtu mode regarding local fragmentation with UFO/CORK
authorHannes Frederic Sowa <hannes@stressinduktion.org>
Sun, 27 Oct 2013 16:29:11 +0000 (17:29 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Oct 2013 04:15:22 +0000 (00:15 -0400)
commit3f3f3ae4b0b6cd4119f04392bb02cd7bef189a3c
tree388feed8fdf3b531da0d365a06d36ae6fcd5bdab
parent8f6fb073da23bfc802ef12ba3bdaf3e28b037d69
ipv4: fix DO and PROBE pmtu mode regarding local fragmentation with UFO/CORK

UFO as well as UDP_CORK do not respect IP_PMTUDISC_DO and
IP_PMTUDISC_PROBE well enough.

UFO enabled packet delivery just appends all frags to the cork and hands
it over to the network card. So we just deliver non-DF udp fragments
(DF-flag may get overwritten by hardware or virtual UFO enabled
interface).

UDP_CORK does enqueue the data until the cork is disengaged. At this
point it sets the correct IP_DF and local_df flags and hands it over to
ip_fragment which in this case will generate an icmp error which gets
appended to the error socket queue. This is not reflected in the syscall
error (of course, if UFO is enabled this also won't happen).

Improve this by checking the pmtudisc flags before appending data to the
socket and if we still can fit all data in one packet when IP_PMTUDISC_DO
or IP_PMTUDISC_PROBE is set, only then proceed.

We use (mtu-fragheaderlen) to check for the maximum length because we
ensure not to generate a fragment and non-fragmented data does not need
to have its length aligned on 64 bit boundaries. Also the passed in
ip_options are already aligned correctly.

Maybe, we can relax some other checks around ip_fragment. This needs
more research.

Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_output.c