aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Tuexen <tuexen@FreeBSD.org>2025-01-04 20:08:43 +0000
committerMichael Tuexen <tuexen@FreeBSD.org>2025-01-04 20:08:43 +0000
commit4bce1a19fcfac0c9f22c11278daa510546ccd3f2 (patch)
treef17300cef72c8981c484c85e134aed6980406405
parentb662ca1d6cd82044c6cb79075e1830b97594bef3 (diff)
TCP BBR: remove code which is not needed
rc_bbr_substate is a 3-bit unsigned int, so it can't be larger than or equal to 8. The wrap around already happens. No functional change intended. Reviewed by: rrs CID: 1523795 MFC after: 1 week Sponsored by: Netflix, Inc. Differential Revision: https://reviews.freebsd.org/D48320
-rw-r--r--sys/netinet/tcp_stacks/bbr.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c
index ed7e07861ebd..535766a0a1b0 100644
--- a/sys/netinet/tcp_stacks/bbr.c
+++ b/sys/netinet/tcp_stacks/bbr.c
@@ -10313,10 +10313,6 @@ bbr_substate_change(struct tcp_bbr *bbr, uint32_t cts, int32_t line, int dolog)
bbr->r_ctl.bbr_smallest_srtt_state2 = bbr->r_ctl.bbr_smallest_srtt_this_state;
}
bbr->rc_bbr_substate++;
- if (bbr->rc_bbr_substate >= BBR_SUBSTATE_COUNT) {
- /* Cycle back to first state-> gain */
- bbr->rc_bbr_substate = 0;
- }
if (bbr_state_val(bbr) == BBR_SUB_GAIN) {
/*
* We enter the gain(5/4) cycle (possibly less if