aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMenglong Dong <imagedong@tencent.com>2023-08-11 10:55:28 +0800
committerDavid S. Miller <davem@davemloft.net>2023-08-13 12:21:37 +0100
commit800a666141de75baebb59b347b9c9e43d963e4da (patch)
tree0037b3cec942b685fd922f1ff1451b67e932c231
parente2142825c120d4317abf7160a0fc34b3de532586 (diff)
downloadnf-next-800a666141de75baebb59b347b9c9e43d963e4da.tar.gz
net: tcp: allow zero-window ACK update the window
Fow now, an ACK can update the window in following case, according to the tcp_may_update_window(): 1. the ACK acknowledged new data 2. the ACK has new data 3. the ACK expand the window and the seq of it is valid Now, we allow the ACK update the window if the window is 0, and the seq/ack of it is valid. This is for the case that the receiver replies an zero-window ACK when it is under memory stress and can't queue the new data. Signed-off-by: Menglong Dong <imagedong@tencent.com> Reviewed-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/ipv4/tcp_input.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c
index 2ac05948341014..d34d52fdfdb1e0 100644
--- a/net/ipv4/tcp_input.c
+++ b/net/ipv4/tcp_input.c
@@ -3525,7 +3525,7 @@ static inline bool tcp_may_update_window(const struct tcp_sock *tp,
{
return after(ack, tp->snd_una) ||
after(ack_seq, tp->snd_wl1) ||
- (ack_seq == tp->snd_wl1 && nwin > tp->snd_wnd);
+ (ack_seq == tp->snd_wl1 && (nwin > tp->snd_wnd || !nwin));
}
/* If we update tp->snd_una, also update tp->bytes_acked */