Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread David Miller
From: Angelo P. Castellani [EMAIL PROTECTED] Date: Mon, 19 Feb 2007 11:35:09 +0100 From: Angelo P. Castellani [EMAIL PROTECTED] RFC3742: limited slow start See http://www.ietf.org/rfc/rfc3742.txt Signed-off-by: Angelo P. Castellani [EMAIL PROTECTED] --- To allow code reutilization

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread David Miller
From: John Heffner [EMAIL PROTECTED] Date: Mon, 19 Feb 2007 16:00:06 -0500 I'd prefer to make it apply automatically across all congestion controls that do slow-start, and also make the max_ssthresh parameter controllable via sysctl. This patch (attached) should implement this. Note the

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread David Miller
From: David Miller [EMAIL PROTECTED] Date: Thu, 22 Feb 2007 00:27:04 -0800 (PST) I'll apply this, but could you please also when making suggestions like this provide the patch necessary to kill the function added for YeaH and the call site in the YeaH algorithm? Here is how I'm resolving

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread John Heffner
My patch is meant as a replacement for YeAH patch 2/2, not meant to back it out. You do still need the second hunk below. Sorry 'bout that. If you're going to apply YeAH patch 2/2 first, you will also need to remove the declaration of tcp_limited_slow_start() in include/net/tcp.h. Thanks,

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread David Miller
From: John Heffner [EMAIL PROTECTED] Date: Thu, 22 Feb 2007 09:25:07 -0500 My patch is meant as a replacement for YeAH patch 2/2, not meant to back it out. You do still need the second hunk below. Sorry 'bout that. If you're going to apply YeAH patch 2/2 first, you will also need to

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-22 Thread Angelo P. Castellani
Ok, I've missed this bit in the submitting patches documentation.. however sorry for that. 2007/2/22, David Miller [EMAIL PROTECTED]: Please never submit patches like this, submit the infrastructure FIRST, then submit the stuff that uses it. When a sequence of patches is applied, in sequence,

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-20 Thread Angelo P. Castellani
John Heffner ha scritto: Sorry for the confusion. The patch I attached to my message was compile-tested only. Well I've read your reply by night and I haven't seen that you attached a patch. Sorry for that. Kind regards, Angelo - To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-19 Thread Angelo P. Castellani
Forgot the patch.. Angelo P. Castellani ha scritto: From: Angelo P. Castellani [EMAIL PROTECTED] RFC3742: limited slow start See http://www.ietf.org/rfc/rfc3742.txt Signed-off-by: Angelo P. Castellani [EMAIL PROTECTED] --- To allow code reutilization I've added the limited slow start

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-19 Thread John Heffner
I'd prefer to make it apply automatically across all congestion controls that do slow-start, and also make the max_ssthresh parameter controllable via sysctl. This patch (attached) should implement this. Note the default value for sysctl_tcp_max_ssthresh = 0, which disables limited

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-19 Thread Angelo P. Castellani
John Heffner ha scritto: Note the patch is compile-tested only! I can do some real testing if you'd like to apply this Dave. The date you read on the patch is due to the fact I've splitted this patchset into 2 diff files. This isn't compile-tested only, I've used this piece of code for about

Re: [PATCH 2/2][TCP] YeAH-TCP: limited slow start exported function

2007-02-19 Thread John Heffner
Angelo P. Castellani wrote: John Heffner ha scritto: Note the patch is compile-tested only! I can do some real testing if you'd like to apply this Dave. The date you read on the patch is due to the fact I've splitted this patchset into 2 diff files. This isn't compile-tested only, I've used