tcp_bbr: fix pacing_gain to always be unity when using lt_bw
authorNeal Cardwell <ncardwell@google.com>
Wed, 31 Jan 2018 20:43:05 +0000 (15:43 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 1 Feb 2018 14:43:38 +0000 (09:43 -0500)
commit3aff3b4b986e51bcf4ab249e5d48d39596e0df6a
tree578224f45d5d80e61ccfd2fa008bc65145b16c8e
parent2e85283dabc22f4715b136e8a7426bd9bef4ce69
tcp_bbr: fix pacing_gain to always be unity when using lt_bw

This commit fixes the pacing_gain to remain at BBR_UNIT (1.0) when
using lt_bw and returning from the PROBE_RTT state to PROBE_BW.

Previously, when using lt_bw, upon exiting PROBE_RTT and entering
PROBE_BW the bbr_reset_probe_bw_mode() code could sometimes randomly
end up with a cycle_idx of 0 and hence have bbr_advance_cycle_phase()
set a pacing gain above 1.0. In such cases this would result in a
pacing rate that is 1.25x higher than intended, potentially resulting
in a high loss rate for a little while until we stop using the lt_bw a
bit later.

This commit is a stable candidate for kernels back as far as 4.9.

Fixes: 0f8782ea1497 ("tcp_bbr: add BBR congestion control")
Signed-off-by: Neal Cardwell <ncardwell@google.com>
Signed-off-by: Yuchung Cheng <ycheng@google.com>
Signed-off-by: Soheil Hassas Yeganeh <soheil@google.com>
Reported-by: Beyers Cronje <bcronje@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_bbr.c