tcp: RFC6298 supersedes RFC2988bis
authorEric Dumazet <eric.dumazet@gmail.com>
Thu, 12 Apr 2012 19:48:40 +0000 (19:48 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 14 Apr 2012 19:24:26 +0000 (15:24 -0400)
Updates some comments to track RFC6298

Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: H.K. Jerry Chu <hkchu@google.com>
Cc: Tom Herbert <therbert@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/tcp.h
net/ipv4/inet_connection_sock.c
net/ipv4/tcp_input.c

index f75a04d752cb394d02586f4d4a8632a63d01cd13..057f0168462e15cdb68f5149a6cdf9111164631b 100644 (file)
@@ -123,7 +123,7 @@ extern void tcp_time_wait(struct sock *sk, int state, int timeo);
 #endif
 #define TCP_RTO_MAX    ((unsigned)(120*HZ))
 #define TCP_RTO_MIN    ((unsigned)(HZ/5))
-#define TCP_TIMEOUT_INIT ((unsigned)(1*HZ))    /* RFC2988bis initial RTO value */
+#define TCP_TIMEOUT_INIT ((unsigned)(1*HZ))    /* RFC6298 2.1 initial RTO value        */
 #define TCP_TIMEOUT_FALLBACK ((unsigned)(3*HZ))        /* RFC 1122 initial RTO value, now
                                                 * used as a fallback RTO for the
                                                 * initial data transmission if no
index 19d66cefd7d34beae89c617a10f195f2f097770f..c12396f2785fdf0a002eea58a78ebb89ce3c1b7b 100644 (file)
@@ -514,7 +514,7 @@ void inet_csk_reqsk_queue_prune(struct sock *parent,
 
        /* Normally all the openreqs are young and become mature
         * (i.e. converted to established socket) for first timeout.
-        * If synack was not acknowledged for 3 seconds, it means
+        * If synack was not acknowledged for 1 second, it means
         * one of the following things: synack was lost, ack was lost,
         * rtt is high or nobody planned to ack (i.e. synflood).
         * When server is a bit loaded, queue is populated with old
index 9944c1d9a2180b8fab1b431b1a2792d7ccbc3b87..dc1e0be30b776e1da8da0acdbbece295a0b369f8 100644 (file)
@@ -936,7 +936,7 @@ static void tcp_init_metrics(struct sock *sk)
        tcp_set_rto(sk);
 reset:
        if (tp->srtt == 0) {
-               /* RFC2988bis: We've failed to get a valid RTT sample from
+               /* RFC6298: 5.7 We've failed to get a valid RTT sample from
                 * 3WHS. This is most likely due to retransmission,
                 * including spurious one. Reset the RTO back to 3secs
                 * from the more aggressive 1sec to avoid more spurious
@@ -946,7 +946,7 @@ reset:
                inet_csk(sk)->icsk_rto = TCP_TIMEOUT_FALLBACK;
        }
        /* Cut cwnd down to 1 per RFC5681 if SYN or SYN-ACK has been
-        * retransmitted. In light of RFC2988bis' more aggressive 1sec
+        * retransmitted. In light of RFC6298 more aggressive 1sec
         * initRTO, we only reset cwnd when more than 1 SYN/SYN-ACK
         * retransmission has occurred.
         */