MINOR: quic: transform pacing settings into a global option
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 30 Jan 2025 13:50:19 +0000 (14:50 +0100)
committerAurelien DARRAGON <adarragon@haproxy.com>
Wed, 16 Apr 2025 08:29:46 +0000 (10:29 +0200)
commit02b4fa4f11d4aad3d174998958a235b4854325d9
tree42db833d5624ba322005e486171809f6b78e69ed
parenta005c1b4e41d4302083ff15ae9c9b40c7579ab8a
MINOR: quic: transform pacing settings into a global option

Pacing support was previously activated on each bind line individually,
via an optional argument of quic-cc-algo keyword. Remove this optional
argument and introduce a global setting to enable/disable pacing. Pacing
activation is still flagged as experimental.

One important change is that previously BBR usage automatically
activated pacing support. This is not the case anymore, so users should
now always explicitely activate pacing if BBR is selected. A new warning
message will be displayed if this is not the case.

Another consequence of this change is that now pacing_inter callback is
always defined for every quic_cc_algo types. As such, QUIC MUX uses
global.tune.options to determine if pacing is required.

This should be backported up to 3.1, after a period of observation.

(cherry picked from commit 0c8b54b2d1cbf56ab696c376e145cdd2e944aadd)
Signed-off-by: Aurelien DARRAGON <adarragon@haproxy.com>
doc/configuration.txt
include/haproxy/global-t.h
include/haproxy/quic_cc-t.h
src/cfgparse-quic.c
src/cfgparse.c
src/haproxy.c
src/mux_quic.c
src/quic_cc_cubic.c
src/quic_cc_newreno.c
src/quic_cc_nocc.c