]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: quic: mark BBR as stable
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 30 Jan 2025 13:57:27 +0000 (14:57 +0100)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 30 Jan 2025 16:20:41 +0000 (17:20 +0100)
Pacing has recently been moved out of experimental status and is
activated by default. This is a mandatory requirement for BBR.
Furthermore, BBR is now considered stable. As such, removes its
experimental status with this commit.

doc/configuration.txt
src/cfgparse-quic.c

index 8a0af0f9403099a660150ee923141ccfa6f74665..ac888590f1357b22adb27fc4245e564fdfce3dec 100644 (file)
@@ -17305,9 +17305,7 @@ quic-cc-algo { cubic | newreno | bbr | nocc }[(<args,...>)]
   improve throughput. It can be turned off via "tune.quic.disable-tx-pacing"
   global keyword. In most cases, pacing should remain activated, especially
   when using BBR as it relies on it to work as expected. Using BBR without
-  pacing may cause slowdowns or high loss rates during transfers. Also note
-  that haproxy's BBR implementation is also considered as experimental and
-  cannot be enabled without "expose-experimental-directives".
+  pacing may cause slowdowns or high loss rates during transfers.
 
   Default value: cubic
 
index aa90aa06b280589e41f3e706b21354ffb333342b..81de9559991c413bc62918b5ba842306c8488198 100644 (file)
@@ -106,12 +106,6 @@ static int bind_parse_quic_cc_algo(char **args, int cur_arg, struct proxy *px,
                arg += strlen(QUIC_CC_CUBIC_STR);
        }
        else if (isteq(algo_ist, ist(QUIC_CC_BBR_STR))) {
-               if (!experimental_directives_allowed) {
-                       ha_alert("'%s' algo is experimental, must be allowed via a global "
-                                "'expose-experimental-directives'\n", arg);
-                       goto fail;
-               }
-
                /* bbr */
                algo = QUIC_CC_BBR_STR;
                *cc_algo = quic_cc_algo_bbr;