Deprecate --gfe2_reloadable_flag_quic_bbr2_flip_bbq2.
Startblock:
after 2020-10-27 10:30 in Google/US-CAM
PiperOrigin-RevId: 339253198
Change-Id: I22d11b5b612a3b40300d0297e8b1530620258a29
diff --git a/quic/core/congestion_control/bbr2_misc.h b/quic/core/congestion_control/bbr2_misc.h
index 0872076..2287960 100644
--- a/quic/core/congestion_control/bbr2_misc.h
+++ b/quic/core/congestion_control/bbr2_misc.h
@@ -76,8 +76,7 @@
*/
// The gain for both CWND and PacingRate at startup.
- float startup_cwnd_gain =
- GetQuicReloadableFlag(quic_bbr2_flip_bbq2) ? 2.0 : 2.885;
+ float startup_cwnd_gain = 2.0;
// TODO(wub): Maybe change to the newly derived value of 2.773 (4 * ln(2)).
float startup_pacing_gain = 2.885;
@@ -99,8 +98,7 @@
/*
* DRAIN parameters.
*/
- float drain_cwnd_gain =
- GetQuicReloadableFlag(quic_bbr2_flip_bbq2) ? 2.0 : 2.885;
+ float drain_cwnd_gain = 2.0;
float drain_pacing_gain = 1.0 / 2.885;
/*
diff --git a/quic/core/congestion_control/bbr2_sender.cc b/quic/core/congestion_control/bbr2_sender.cc
index 6761202..a2e23bf 100644
--- a/quic/core/congestion_control/bbr2_sender.cc
+++ b/quic/core/congestion_control/bbr2_sender.cc
@@ -148,14 +148,8 @@
void Bbr2Sender::ApplyConnectionOptions(
const QuicTagVector& connection_options) {
if (ContainsQuicTag(connection_options, kBBQ2)) {
- if (GetQuicReloadableFlag(quic_bbr2_flip_bbq2)) {
- params_.startup_cwnd_gain = 2.885;
- params_.drain_cwnd_gain = 2.885;
- } else {
- // 2 is the lower, derived gain for CWND.
- params_.startup_cwnd_gain = 2;
- params_.drain_cwnd_gain = 2;
- }
+ params_.startup_cwnd_gain = 2.885;
+ params_.drain_cwnd_gain = 2.885;
}
if (GetQuicReloadableFlag(quic_bbr2_no_exit_startup_on_loss_with_bw_growth) &&
ContainsQuicTag(connection_options, kB2NE)) {
diff --git a/quic/core/congestion_control/bbr2_simulator_test.cc b/quic/core/congestion_control/bbr2_simulator_test.cc
index c769bff..022dba8 100644
--- a/quic/core/congestion_control/bbr2_simulator_test.cc
+++ b/quic/core/congestion_control/bbr2_simulator_test.cc
@@ -623,17 +623,9 @@
EXPECT_APPROX_EQ(sender_->BandwidthEstimate() * (1 / 2.885f),
sender_->PacingRate(0), 0.01f);
- if (GetQuicReloadableFlag(quic_bbr2_flip_bbq2)) {
- // BBR uses CWND gain of 2 during STARTUP, hence it will fill the buffer
- // with approximately 1 BDP. Here, we use 0.95 to give some margin for
- // error.
- EXPECT_GE(queue->bytes_queued(), 0.95 * params.BDP());
- } else {
- // BBR uses CWND gain of 2.88 during STARTUP, hence it will fill the buffer
- // with approximately 1.88 BDPs. Here, we use 1.5 to give some margin for
- // error.
- EXPECT_GE(queue->bytes_queued(), 1.5 * params.BDP());
- }
+ // BBR uses CWND gain of 2 during STARTUP, hence it will fill the buffer with
+ // approximately 1 BDP. Here, we use 0.95 to give some margin for error.
+ EXPECT_GE(queue->bytes_queued(), 0.95 * params.BDP());
// Observe increased RTT due to bufferbloat.
const QuicTime::Delta queueing_delay =