svn commit: r347080 - stable/12/sys/netinet

[Michael Tuexen]

Author: tuexen
Date: Sat May  4 09:01:56 2019
New Revision: 347080
URL: https://svnweb.freebsd.org/changeset/base/347080

Log:
  MFC r343401:
  Update a comment to reflect the current reality.
  SYN-cache entries live for abaut 12 seconds, not 45, when default
  setting are used.
  
  MFC r343402:
  Kill a trailing whitespace character...
  
  MFC r343403:
  Get the arithmetic right...
  
  Sponsored by:		Netflix, Inc.

Modified:
  stable/12/sys/netinet/tcp_syncache.c
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sys/netinet/tcp_syncache.c
==============================================================================
--- stable/12/sys/netinet/tcp_syncache.c	Sat May  4 02:45:24 2019	(r347079)
+++ stable/12/sys/netinet/tcp_syncache.c	Sat May  4 09:01:56 2019	(r347080)
@@ -154,7 +154,12 @@ static int	 syncookie_cmp(struct in_conninfo *inc, str
 
 /*
  * Transmit the SYN,ACK fewer times than TCP_MAXRXTSHIFT specifies.
- * 3 retransmits corresponds to a timeout of 3 * (1 + 2 + 4 + 8) == 45 seconds,
+ * 3 retransmits corresponds to a timeout with default values of
+ * TCPTV_RTOBASE * (                 1 +
+ *                  tcp_syn_backoff[1] +
+ *                  tcp_syn_backoff[2] +
+ *                  tcp_syn_backoff[3]) + 3 * tcp_rexmit_slop,
+ * 3000 ms * (1 + 1 + 1 + 1) +  3 * 200 ms = 12600 ms,
  * the odds are that the user has given up attempting to connect by then.
  */
 #define SYNCACHE_MAXREXMTS		3