From a6947c96fc4ed7d998644db395216326e2a9cab8 Mon Sep 17 00:00:00 2001 From: Frederic Lecaille Date: Fri, 13 Dec 2024 08:28:56 +0100 Subject: [PATCH] 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 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 . Must be backported to 3.1. (cherry picked from commit e61b4189078f97d036952371c7fc9e5ceefdfff0) Signed-off-by: Christopher Faulet --- src/quic_cc_bbr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/quic_cc_bbr.c b/src/quic_cc_bbr.c index f2008ed..6179c9f 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 */ -- 1.7.10.4