BUG/MINOR: quic: Remaining useless statements in cubic slow start callback

When entering a recovery period, the algo state is set by quic_enter_recovery().
And that's it!. These two lines should have been removed with this commit:

     BUG/MINOR: quic: Wrong use of now_ms timestamps (cubic algo)

Take the opportunity of this patch to add a missing TRACE_LEAVE() call in
quic_cc_cubic_ca_cb().

Must be backported to 2.7 and 2.6.
This commit is contained in:
Frédéric Lécaille 2023-03-31 20:18:44 +02:00
parent d481ed3fa0
commit 23b8eef05b

View File

@ -211,8 +211,6 @@ static void quic_cc_cubic_ss_cb(struct quic_cc *cc, struct quic_cc_event *ev)
case QUIC_CC_EVT_LOSS:
quic_enter_recovery(cc);
/* Exit to congestion avoidance. */
cc->algo->state = QUIC_CC_ST_CA;
break;
case QUIC_CC_EVT_ECN_CE:
@ -244,6 +242,7 @@ static void quic_cc_cubic_ca_cb(struct quic_cc *cc, struct quic_cc_event *ev)
out:
TRACE_PROTO("CC cubic", QUIC_EV_CONN_CC, cc->qc, NULL, cc);
TRACE_LEAVE(QUIC_EV_CONN_CC, cc->qc);
}
/* Recovery period callback */