Commit 5415e3c3 authored by Pengcheng Yang's avatar Pengcheng Yang Committed by David S. Miller

tcp: fix stretch ACK bugs in Scalable

Change Scalable to properly handle stretch ACKs in additive
increase mode by passing in the count of ACKed packets to
tcp_cong_avoid_ai().

In addition, because we are now precisely accounting for
stretch ACKs, including delayed ACKs, we can now change
TCP_SCALABLE_AI_CNT to 100.
Signed-off-by: default avatarPengcheng Yang <yangpc@wangsu.com>
Acked-by: default avatarNeal Cardwell <ncardwell@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent be0d935e
...@@ -10,10 +10,9 @@ ...@@ -10,10 +10,9 @@
#include <net/tcp.h> #include <net/tcp.h>
/* These factors derived from the recommended values in the aer: /* These factors derived from the recommended values in the aer:
* .01 and and 7/8. We use 50 instead of 100 to account for * .01 and and 7/8.
* delayed ack.
*/ */
#define TCP_SCALABLE_AI_CNT 50U #define TCP_SCALABLE_AI_CNT 100U
#define TCP_SCALABLE_MD_SCALE 3 #define TCP_SCALABLE_MD_SCALE 3
static void tcp_scalable_cong_avoid(struct sock *sk, u32 ack, u32 acked) static void tcp_scalable_cong_avoid(struct sock *sk, u32 ack, u32 acked)
...@@ -23,11 +22,13 @@ static void tcp_scalable_cong_avoid(struct sock *sk, u32 ack, u32 acked) ...@@ -23,11 +22,13 @@ static void tcp_scalable_cong_avoid(struct sock *sk, u32 ack, u32 acked)
if (!tcp_is_cwnd_limited(sk)) if (!tcp_is_cwnd_limited(sk))
return; return;
if (tcp_in_slow_start(tp)) if (tcp_in_slow_start(tp)) {
tcp_slow_start(tp, acked); acked = tcp_slow_start(tp, acked);
else if (!acked)
return;
}
tcp_cong_avoid_ai(tp, min(tp->snd_cwnd, TCP_SCALABLE_AI_CNT), tcp_cong_avoid_ai(tp, min(tp->snd_cwnd, TCP_SCALABLE_AI_CNT),
1); acked);
} }
static u32 tcp_scalable_ssthresh(struct sock *sk) static u32 tcp_scalable_ssthresh(struct sock *sk)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment