diff options
| author |   Neal Cardwell <ncardwell@google.com> | 2015-03-10 17:17:04 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-03-11 16:51:51 -0400 |
| commit | d578e18ce93f5d33a7120fd57c453e22a4c0fc37 (patch) | |
| tree | c5a801604470af1670387f46c4d613ef11260ca0 /net/core | |
| parent | tcp: fix tcp_cong_avoid_ai() credit accumulation bug with decreases in w (diff) | |
| download | linux-dev-d578e18ce93f5d33a7120fd57c453e22a4c0fc37.tar.xz linux-dev-d578e18ce93f5d33a7120fd57c453e22a4c0fc37.zip | |
tcp: restore 1.5x per RTT limit to CUBIC cwnd growth in congestion avoidance
Commit 814d488c6126 ("tcp: fix the timid additive increase on stretch
ACKs") fixed a bug where tcp_cong_avoid_ai() would either credit a
connection with an increase of snd_cwnd_cnt, or increase snd_cwnd, but
not both, resulting in cwnd increasing by 1 packet on at most every
alternate invocation of tcp_cong_avoid_ai().
Although the commit correctly implemented the CUBIC algorithm, which
can increase cwnd by as much as 1 packet per 1 packet ACKed (2x per
RTT), in practice that could be too aggressive: in tests on network
paths with small buffers, YouTube server retransmission rates nearly
doubled.
This commit restores CUBIC to a maximum cwnd growth rate of 1 packet
per 2 packets ACKed (1.5x per RTT). In YouTube tests this restored
retransmit rates to low levels.
Testing: This patch has been tested in datacenter netperf transfers
and live youtube.com and google.com servers.
Fixes: 9cd981dcf174 ("tcp: fix stretch ACK bugs in CUBIC")
Signed-off-by: Neal Cardwell <ncardwell@google.com>
Signed-off-by: Yuchung Cheng <ycheng@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
0 files changed, 0 insertions, 0 deletions