浏览代码

[TCP]: Don't enter to fast recovery while using FRTO

Because TCP is not in Loss state during FRTO recovery, fast
recovery could be triggered by accident. Non-SACK FRTO is more
robust than not yet included SACK-enhanced version (that can
receiver high number of duplicate ACKs with SACK blocks during
FRTO), at least with unidirectional transfers, but under
extraordinary patterns fast recovery can be incorrectly
triggered, e.g., Data loss+ACK losses => cumulative ACK with
enough SACK blocks to meet sacked_out >= dupthresh condition).

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Ilpo Järvinen 18 年之前
父节点
当前提交
52c63f1e86
共有 1 个文件被更改,包括 4 次插入0 次删除
  1. 4 0
      net/ipv4/tcp_input.c

+ 4 - 0
net/ipv4/tcp_input.c

@@ -1547,6 +1547,10 @@ static int tcp_time_to_recover(struct sock *sk, struct tcp_sock *tp)
 {
 {
 	__u32 packets_out;
 	__u32 packets_out;
 
 
+	/* Do not perform any recovery during FRTO algorithm */
+	if (tp->frto_counter)
+		return 0;
+
 	/* Trick#1: The loss is proven. */
 	/* Trick#1: The loss is proven. */
 	if (tp->lost_out)
 	if (tp->lost_out)
 		return 1;
 		return 1;