Re: [PATCH] tcp: add setsockopt to disable slow start after idle

From: Cristian KLEIN
Date: Sun Apr 11 2010 - 14:45:26 EST


On 11/04/2010 00:47, David Miller wrote:
From: Cristian KLEIN<cristiklein@xxxxxxxxx>
Date: Sat, 10 Apr 2010 14:09:03 +0200

Could you please explain me why it is dangerous? To me it seems that
it's just like allowing applications to disable NAGLE or to choose a
congestion control algorithm.

Because you can cause undue congestion to other people on the network
because you are believing path information that has been outdated and
has not been validated by sending data for a certain amount of time.

I consider your argument an important concern, but I'm not quite convinced this patch is so bad.

An application which does not need this behaviour will continue to slow start after idle by default.

Without this patch, an application which needs this behaviour (i.e. not to slow start after idle) is forced to implement its own UDP-based protocol with all the congestion control, retransmission etc. Undue congestion might still occur.


If you don't agree with the above two points, would you consider accepting a patch with an allow_user_fast_start_after_idle sysctl?

Cristi.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/