Re: tcp_slow_start_after_idle

[Rick Jones <rick.jones2@hp.com>]

> +tcp_slow_start_after_idle - BOOLEAN
> +	If set, provide RFC2861 behavior and time out the congestion
> +	window after an idle period.  An idle period is defined at
> +	the current RTO.  If unset, the congestion window will not
> +	be timed out after an idle period.
> +	Default: 1

Did you mean "defined as" rather than "defined at?"

Also, does the congestion window "time out" or does it decay?

Perhaps:

tcp_slow_start_after_idle - BOOLEAN
      If set, provide RFC2861 behavior and decay the congestion
      window after the connection has been idle for the connection's
      current RTO.  If unset, the congestion window will not decay
      when the connection has been idle.
      Default: 1

> diff --git a/include/net/tcp.h b/include/net/tcp.h
> index de88c54..bfc71f9 100644
> --- a/include/net/tcp.h
> +++ b/include/net/tcp.h
> @@ -227,6 +227,7 @@ extern int sysctl_tcp_abc;
>  extern int sysctl_tcp_mtu_probing;
>  extern int sysctl_tcp_base_mss;
>  extern int sysctl_tcp_workaround_signed_windows;
> +extern int sysctl_tcp_slow_start_after_idle;

> diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c
> index 743016b..be6d929 100644
> --- a/net/ipv4/tcp_output.c
> +++ b/net/ipv4/tcp_output.c
> @@ -59,6 +59,9 @@ int sysctl_tcp_tso_win_divisor = 3;
>  int sysctl_tcp_mtu_probing = 0;
>  int sysctl_tcp_base_mss = 512;
>  
> +/* By default, RFC2861 behavior.  */
> +int sysctl_tcp_slow_start_after_idle = 1;
> +

Is this a candidate for "readmostly?"

rick jones