MINOR: quic: Stop hardcoding a scale shifting value (CUBIC_BETA_SCALE_FACTOR_SHIFT)
authorFrederic Lecaille <flecaille@haproxy.com>
Thu, 25 Jan 2024 06:55:33 +0000 (07:55 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 31 Jan 2024 14:44:56 +0000 (15:44 +0100)
Very minor modification to replace a statement with an hardcoded value by a macro.

Should be backported as far as 2.6 to ease any further modification to come.

(cherry picked from commit 96385f40b508a395fcd802a9dff13e968c12433d)
Signed-off-by: Willy Tarreau <w@1wt.eu>

src/quic_cc_cubic.c

index 791987c..0ed4a32 100644 (file)
@@ -138,7 +138,7 @@ static inline void quic_cubic_update(struct quic_cc *cc, uint32_t acked)
                goto leave;
        }
 
-       delta = path->mtu * ((CUBIC_C * diff * diff * diff) >> (10 + 3 * TIME_SCALE_FACTOR_SHIFT));
+       delta = path->mtu * ((CUBIC_C * diff * diff * diff) >> (CUBIC_BETA_SCALE_SHIFT + 3 * TIME_SCALE_FACTOR_SHIFT));
        if (t < c->K)
                target = c->origin_point - delta;
        else