diff options
author | Richard Scheffenegger <rscheff@FreeBSD.org> | 2021-02-19 12:52:06 +0000 |
---|---|---|
committer | Richard Scheffenegger <rscheff@FreeBSD.org> | 2021-02-26 07:39:55 +0000 |
commit | c3aa4ba5dfc084e40e4151b25a0d5f8d24a036ba (patch) | |
tree | af514fa85e4e56fbeb30c44d1bfa5c97805cc064 | |
parent | cdba1277dd26c8f7e54a40d86d44748f2d1a8c29 (diff) | |
download | src-c3aa4ba5dfc084e40e4151b25a0d5f8d24a036ba.tar.gz src-c3aa4ba5dfc084e40e4151b25a0d5f8d24a036ba.zip |
Ensure cwnd doesn't shrink to zero with PRR
Under some circumstances, PRR may end up with a fully
collapsed cwnd when finalizing the loss recovery.
Reviewed By: #transport, kbowling
Reported by: Liang Tian
MFC after: 5 days
Sponsored by: NetApp, Inc.
Differential Revision: https://reviews.freebsd.org/D28780
(cherry picked from commit 853fd7a2e39802e46bd3d6476529796ac22412d9)
-rw-r--r-- | sys/netinet/tcp_input.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c index 459b78cd444a..08d000611407 100644 --- a/sys/netinet/tcp_input.c +++ b/sys/netinet/tcp_input.c @@ -3972,8 +3972,8 @@ tcp_prr_partialack(struct tcpcb *tp, struct tcphdr *th) * If there is going to be a SACK retransmission, adjust snd_cwnd * accordingly. */ - tp->snd_cwnd = tp->snd_nxt - tp->snd_recover + - tp->sackhint.sack_bytes_rexmit + (snd_cnt * maxseg); + tp->snd_cwnd = max(maxseg, (int64_t)tp->snd_nxt - tp->snd_recover + + tp->sackhint.sack_bytes_rexmit + (snd_cnt * maxseg)); tp->t_flags |= TF_ACKNOW; (void) tcp_output(tp); } |