]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
tcp: do not use cached RTT for RTT estimation
authorYuchung Cheng <ycheng@google.com>
Fri, 30 Aug 2013 15:35:53 +0000 (08:35 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 30 Aug 2013 19:14:38 +0000 (15:14 -0400)
commit1b7fdd2ab5852717a4fc7d79847759c67065d7e9
tree6ddfcff2fb0595cb28b72a72b7c0809395c3082a
parent08f89b981b0e6b46058dae2957241d4099129521
tcp: do not use cached RTT for RTT estimation

RTT cached in the TCP metrics are valuable for the initial timeout
because SYN RTT usually does not account for serialization delays
on low BW path.

However using it to seed the RTT estimator maybe disruptive because
other components (e.g., pacing) require the smooth RTT to be obtained
from actual connection.

The solution is to use the higher cached RTT to set the first RTO
conservatively like tcp_rtt_estimator(), but avoid seeding the other
RTT estimator variables such as srtt.  It is also a good idea to
keep RTO conservative to obtain the first RTT sample, and the
performance is insured by TCP loss probe if SYN RTT is available.

To keep the seeding formula consistent across SYN RTT and cached RTT,
the rttvar is twice the cached RTT instead of cached RTTVAR value. The
reason is because cached variation may be too small (near min RTO)
which defeats the purpose of being conservative on first RTO. However
the metrics still keep the RTT variations as they might be useful for
user applications (through ip).

Signed-off-by: Yuchung Cheng <ycheng@google.com>
Signed-off-by: Neal Cardwell <ncardwell@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Tested-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_metrics.c