Re: [PATCH 1/2] [TCP]: Add two new spurious RTO responses to FRTO
From: David Miller <davem@davemloft.net>
Date: 2007-02-27 18:10:28
From: David Miller <davem@davemloft.net>
Date: 2007-02-27 18:10:28
From: "Ilpo_Järvinen" <redacted> Date: Tue, 27 Feb 2007 17:50:06 +0200 (EET)
New sysctl tcp_frto_response is added to select amongst these responses: - Rate halving based; reuses CA_CWR state (default) - Very conservative; used to be the only one available (=1) - Undo cwr; undoes ssthresh and cwnd reductions (=2) The response with rate halving requires a new parameter to tcp_enter_cwr because FRTO has already reduced ssthresh and doing a second reduction there has to be prevented. In addition, to keep things nice on 80 cols screen, a local variable was added. Signed-off-by: Ilpo Järvinen <redacted>
Applied to tcp-2.6, thanks!
I tidied up the coding style of the switch statement slightly.
So that it reads like this:
switch (sysctl_tcp_frto_response) {
case 2:
tcp_undo_spur_to_response(sk);
break;
case 1:
tcp_conservative_spur_to_response(tp);
break;
default:
tcp_ratehalving_spur_to_response(sk);
break;
};
Thanks again.