git: 6ab740bc90c1 - stable/14 - TCP BBR: remove code which is not needed
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 05 Feb 2025 20:51:13 UTC
The branch stable/14 has been updated by tuexen:
URL: https://cgit.FreeBSD.org/src/commit/?id=6ab740bc90c188e757124533e6c4b67e40becb67
commit 6ab740bc90c188e757124533e6c4b67e40becb67
Author: Michael Tuexen <tuexen@FreeBSD.org>
AuthorDate: 2025-01-04 20:08:43 +0000
Commit: Michael Tuexen <tuexen@FreeBSD.org>
CommitDate: 2025-02-05 09:50:55 +0000
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
Sponsored by: Netflix, Inc.
Differential Revision: https://reviews.freebsd.org/D48320
(cherry picked from commit 4bce1a19fcfac0c9f22c11278daa510546ccd3f2)
---
sys/netinet/tcp_stacks/bbr.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c
index 144b3f25e9c7..6c6bc5d1532f 100644
--- a/sys/netinet/tcp_stacks/bbr.c
+++ b/sys/netinet/tcp_stacks/bbr.c
@@ -10302,10 +10302,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