diff options
| author | Peter Lei <peterlei@netflix.com> | 2025-07-21 14:01:43 +0000 |
|---|---|---|
| committer | Michael Tuexen <tuexen@FreeBSD.org> | 2025-07-21 14:01:43 +0000 |
| commit | de2936aa36754d1b63475f41a1d1429ae6fdfad8 (patch) | |
| tree | da7ac0da489e8ff9750a8225482042079a150265 | |
| parent | 68dc9c6b8ef86caef3962cc9177bb21d8f38b0ed (diff) | |
tcp rack: fix clearing of app limited periods
Use the correct variable in the correct way. The app limited period
is cleared when gp_seq is greater than or equal to
cleared_app_ack_seq.
Reviewed by: rrs, tuexen, Nick Banks
MFC after: 1 week
Sponsored by: Netflix, Inc.
Differential Revision: https://reviews.freebsd.org/D51441
| -rw-r--r-- | sys/netinet/tcp_stacks/rack.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/netinet/tcp_stacks/rack.c b/sys/netinet/tcp_stacks/rack.c index 5280f18dc983..834e1347a152 100644 --- a/sys/netinet/tcp_stacks/rack.c +++ b/sys/netinet/tcp_stacks/rack.c @@ -17987,7 +17987,7 @@ start_set: tp->gput_ack = tp->gput_seq + rack_get_measure_window(tp, rack); rack->r_ctl.rc_gp_cumack_ts = 0; if ((rack->r_ctl.cleared_app_ack == 1) && - (SEQ_GEQ(rack->r_ctl.cleared_app_ack, tp->gput_seq))) { + (SEQ_GEQ(tp->gput_seq, rack->r_ctl.cleared_app_ack_seq))) { /* * We just cleared an application limited period * so the next seq out needs to skip the first |