From: Frederic Lecaille <flecaille@haproxy.com>
Date: Fri, 13 Dec 2024 07:28:56 +0000 (+0100)
Subject: BUG/MINOR: quic: fix the wrong tracked recovery start time value
X-Git-Tag: v3.2-dev2~81
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e61b4189078f97d036952371c7fc9e5ceefdfff0;p=thirdparty%2Fhaproxy.git

BUG/MINOR: quic: fix the wrong tracked recovery start time value

bbr_congestion_event() role is to track the start time of recovery periods.
This was done using <ts> passed as parameter. But this parameter is the
time the newest lost packet has been sent.
The timestamp value to store in ->recovery_start_ts is <now_ms>.

Must be backported to 3.1.
---

diff --git a/src/quic_cc_bbr.c b/src/quic_cc_bbr.c
index f2008edf6a..6179c9f24b 100644
--- a/src/quic_cc_bbr.c
+++ b/src/quic_cc_bbr.c
@@ -1446,7 +1446,7 @@ static void bbr_congestion_event(struct quic_cc *cc, uint32_t ts)
 		tick_isset(bbr->recovery_start_ts) || in_recovery_period(p, ts))
 		return;
 
-	bbr->recovery_start_ts = ts;
+	bbr->recovery_start_ts = now_ms;
 }
 
 /* Callback to return the delivery rate sample struct from <cc> */