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 f2008edf6a049bd0d5e0fcd189d3285e01be5813..6179c9f24b44a944882b79e48124ec839f80ff94 100644 (file)
--- 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> */