From e61b4189078f97d036952371c7fc9e5ceefdfff0 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. --- 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 f2008edf6..6179c9f24 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 */ -- 2.47.3