* Re: XTP for 2.6.25
2008-04-25 8:23 ` Ilpo Järvinen
@ 2008-04-25 8:28 ` David Miller
2008-04-25 8:51 ` Shigeo N
2008-05-07 4:32 ` Shigeo N
2 siblings, 0 replies; 13+ messages in thread
From: David Miller @ 2008-04-25 8:28 UTC (permalink / raw)
To: ilpo.jarvinen; +Cc: shigeonx, andi, linux-kernel, netdev
From: "Ilpo_Järvinen" <ilpo.jarvinen@helsinki.fi>
Date: Fri, 25 Apr 2008 11:23:58 +0300 (EEST)
> Where these results with 2.6.25 as well? ...John Heffner just couple of
> days ago fixed reordering related problem in TCP (in case that netem
> parametrization causes reordering), the fix is not even in 2.6.24.y, I
> think...
Right, I didn't submit it there yet but I will make sure it
hits -stable in the next round.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: XTP for 2.6.25
2008-04-25 8:23 ` Ilpo Järvinen
2008-04-25 8:28 ` David Miller
@ 2008-04-25 8:51 ` Shigeo N
2008-04-25 8:55 ` David Miller
2008-05-07 4:32 ` Shigeo N
2 siblings, 1 reply; 13+ messages in thread
From: Shigeo N @ 2008-04-25 8:51 UTC (permalink / raw)
To: Ilpo Järvinen; +Cc: Andi Kleen, LKML, Netdev
One host(sender) is 2.6.25, but the other(receiver) is 2.6.18.
I will evaluate again after I upgrade it to 2.6.25.
Thanks
On Fri, Apr 25, 2008 at 5:23 PM, Ilpo Järvinen
<ilpo.jarvinen@helsinki.fi> wrote:
> On Fri, 25 Apr 2008, Shigeo N wrote:
>
> > case2) random delay
> >
> > In this case, add a fixed amount of delay + 10% random delay by
> > command, "tc change dev eth0 root netem delay 100ms 10ms 25%".
> >
> > Here is the result.
> >
> > fixed delay UDP/TCP/XTP throghput
> > ---------------------------------------
> > 10ms 88/80/92 Mbps
> > 30ms 88/51/91 Mbps
> > 50ms 88/41/89 Mbps
> > 70ms 88/28/80 Mbps
> > 100ms 88/19/66 Mbps
> >
> > TCP's perfomance looks very poor when delay is long and variable.
>
> Where these results with 2.6.25 as well? ...John Heffner just couple of
> days ago fixed reordering related problem in TCP (in case that netem
> parametrization causes reordering), the fix is not even in 2.6.24.y, I
> think...
>
> --
> i.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: XTP for 2.6.25
2008-04-25 8:51 ` Shigeo N
@ 2008-04-25 8:55 ` David Miller
2008-04-25 13:05 ` Ilpo Järvinen
0 siblings, 1 reply; 13+ messages in thread
From: David Miller @ 2008-04-25 8:55 UTC (permalink / raw)
To: shigeonx; +Cc: ilpo.jarvinen, andi, linux-kernel, netdev
From: "Shigeo N" <shigeonx@gmail.com>
Date: Fri, 25 Apr 2008 17:51:49 +0900
> One host(sender) is 2.6.25, but the other(receiver) is 2.6.18.
> I will evaluate again after I upgrade it to 2.6.25.
The bug fix isn't in 2.6.25 either.
Here is the patch in question, please add it to the kernel
during your tests.
>From 3c0390e1a6d8a5d0cdae57959f0be37a236b474d Mon Sep 17 00:00:00 2001
From: John Heffner <jheffner@napa.(none)>
Date: Fri, 25 Apr 2008 01:43:57 -0700
Subject: [PATCH] [TCP]: Increase the max_burst threshold from 3 to tp->reordering.
[ Upstream commit: dd9e0dda66ba38a2ddd1405ac279894260dc5c36 ]
This change is necessary to allow cwnd to grow during persistent
reordering. Cwnd moderation is applied when in the disorder state
and an ack that fills the hole comes in. If the hole was greater
than 3 packets, but less than tp->reordering, cwnd will shrink when
it should not have.
Signed-off-by: John Heffner <jheffner@napa.(none)>
Signed-off-by: David S. Miller <davem@davemloft.net>
---
include/net/tcp.h | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/include/net/tcp.h b/include/net/tcp.h
index 4fd3eb2..d69d12e 100644
--- a/include/net/tcp.h
+++ b/include/net/tcp.h
@@ -776,11 +776,14 @@ extern void tcp_enter_cwr(struct sock *sk, const int set_ssthresh);
extern __u32 tcp_init_cwnd(struct tcp_sock *tp, struct dst_entry *dst);
/* Slow start with delack produces 3 packets of burst, so that
- * it is safe "de facto".
+ * it is safe "de facto". This will be the default - same as
+ * the default reordering threshold - but if reordering increases,
+ * we must be able to allow cwnd to burst at least this much in order
+ * to not pull it back when holes are filled.
*/
static __inline__ __u32 tcp_max_burst(const struct tcp_sock *tp)
{
- return 3;
+ return tp->reordering;
}
/* Returns end sequence number of the receiver's advertised window */
--
1.5.5.1.57.g5909c
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: XTP for 2.6.25
2008-04-25 8:55 ` David Miller
@ 2008-04-25 13:05 ` Ilpo Järvinen
2008-04-25 20:12 ` David Miller
0 siblings, 1 reply; 13+ messages in thread
From: Ilpo Järvinen @ 2008-04-25 13:05 UTC (permalink / raw)
To: David Miller; +Cc: shigeonx, andi, LKML, Netdev
On Fri, 25 Apr 2008, David Miller wrote:
> From: "Shigeo N" <shigeonx@gmail.com>
> Date: Fri, 25 Apr 2008 17:51:49 +0900
>
> > One host(sender) is 2.6.25, but the other(receiver) is 2.6.18.
> > I will evaluate again after I upgrade it to 2.6.25.
>
> The bug fix isn't in 2.6.25 either.
Sure it is, or git is lying:
$ git-describe dd9e0dda66ba38a2ddd1405ac279894260dc5c36
v2.6.25-rc8-1190-gdd9e0dd
> [ Upstream commit: dd9e0dda66ba38a2ddd1405ac279894260dc5c36 ]
--
i.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: XTP for 2.6.25
2008-04-25 13:05 ` Ilpo Järvinen
@ 2008-04-25 20:12 ` David Miller
2008-04-25 21:06 ` Ilpo Järvinen
0 siblings, 1 reply; 13+ messages in thread
From: David Miller @ 2008-04-25 20:12 UTC (permalink / raw)
To: ilpo.jarvinen; +Cc: shigeonx, andi, linux-kernel, netdev
From: "Ilpo_Järvinen" <ilpo.jarvinen@helsinki.fi>
Date: Fri, 25 Apr 2008 16:05:51 +0300 (EEST)
> On Fri, 25 Apr 2008, David Miller wrote:
>
> > From: "Shigeo N" <shigeonx@gmail.com>
> > Date: Fri, 25 Apr 2008 17:51:49 +0900
> >
> > > One host(sender) is 2.6.25, but the other(receiver) is 2.6.18.
> > > I will evaluate again after I upgrade it to 2.6.25.
> >
> > The bug fix isn't in 2.6.25 either.
>
> Sure it is, or git is lying:
>
> $ git-describe dd9e0dda66ba38a2ddd1405ac279894260dc5c36
> v2.6.25-rc8-1190-gdd9e0dd
I just submitted the TCP reorder patch for 2.6.25-stable, it applied
on top of a tree for which I did:
git checkout -b linux25 v2.6.25
so something is up :-)
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: XTP for 2.6.25
2008-04-25 20:12 ` David Miller
@ 2008-04-25 21:06 ` Ilpo Järvinen
0 siblings, 0 replies; 13+ messages in thread
From: Ilpo Järvinen @ 2008-04-25 21:06 UTC (permalink / raw)
To: David Miller; +Cc: shigeonx, andi, LKML, Netdev
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1092 bytes --]
On Fri, 25 Apr 2008, David Miller wrote:
> From: "Ilpo_Järvinen" <ilpo.jarvinen@helsinki.fi>
> Date: Fri, 25 Apr 2008 16:05:51 +0300 (EEST)
>
> > On Fri, 25 Apr 2008, David Miller wrote:
> >
> > > From: "Shigeo N" <shigeonx@gmail.com>
> > > Date: Fri, 25 Apr 2008 17:51:49 +0900
> > >
> > > > One host(sender) is 2.6.25, but the other(receiver) is 2.6.18.
> > > > I will evaluate again after I upgrade it to 2.6.25.
> > >
> > > The bug fix isn't in 2.6.25 either.
> >
> > Sure it is, or git is lying:
> >
> > $ git-describe dd9e0dda66ba38a2ddd1405ac279894260dc5c36
> > v2.6.25-rc8-1190-gdd9e0dd
>
> I just submitted the TCP reorder patch for 2.6.25-stable, it applied
> on top of a tree for which I did:
>
> git checkout -b linux25 v2.6.25
>
> so something is up :-)
Of course I was wrong :-), misintepreted git-describe's result as it
cannot prove what I was trying to make it to prove (basically it was
reporting just the latest net-2.6 -> net-2.6.26 merge of yours, while
that doesn't tell anything about entry of a commit to Linus' tree).
...Sorry about the noise.
--
i.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: XTP for 2.6.25
2008-04-25 8:23 ` Ilpo Järvinen
2008-04-25 8:28 ` David Miller
2008-04-25 8:51 ` Shigeo N
@ 2008-05-07 4:32 ` Shigeo N
2 siblings, 0 replies; 13+ messages in thread
From: Shigeo N @ 2008-05-07 4:32 UTC (permalink / raw)
To: Ilpo Järvinen; +Cc: Andi Kleen, LKML, Netdev
I tested again on Linux 2.6.25 with TCP reordering patch.
Of course, TCP performance was improved.
Here is the result.
case2-2) random delay (with TCP reordering patch)
In this case, add a fixed amount of delay + 10% random delay by command,
"tc change dev eth0 root netem delay 100ms 10ms 25%".
fixed delay UDP/TCP/XTP throghput
---------------------------------------
10ms 88/85/92 Mbps
30ms 88/67/91 Mbps
50ms 88/58/89 Mbps
70ms 88/43/80 Mbps
100ms 88/26/66 Mbps
case2) random delay (without TCP reordering patch)
In this case, add a fixed amount of delay + 10% random delay by command,
"tc change dev eth0 root netem delay 100ms 10ms 25%".
fixed delay UDP/TCP/XTP throghput
---------------------------------------
10ms 88/80/92 Mbps
30ms 88/51/91 Mbps
50ms 88/41/89 Mbps
70ms 88/28/80 Mbps
100ms 88/19/66 Mbps
case1) fixed delay
In this case, just adds a fixed amount of delay to all packets going out by
command, "tc qdisc add dev eth0 root netem delay 100ms".
fixed delay UDP/TCP/XTP throghput
---------------------------------------
10ms 88/85/92 Mbps
30ms 88/70/91 Mbps
50ms 88/63/89 Mbps
70ms 88/52/84 Mbps
100ms 88/36/70 Mbps
Thanks,
On Fri, Apr 25, 2008 at 5:23 PM, Ilpo Järvinen
<ilpo.jarvinen@helsinki.fi> wrote:
> On Fri, 25 Apr 2008, Shigeo N wrote:
>
>> case2) random delay
>>
>> In this case, add a fixed amount of delay + 10% random delay by
>> command, "tc change dev eth0 root netem delay 100ms 10ms 25%".
>>
>> Here is the result.
>>
>> fixed delay UDP/TCP/XTP throghput
>> ---------------------------------------
>> 10ms 88/80/92 Mbps
>> 30ms 88/51/91 Mbps
>> 50ms 88/41/89 Mbps
>> 70ms 88/28/80 Mbps
>> 100ms 88/19/66 Mbps
>>
>> TCP's perfomance looks very poor when delay is long and variable.
>
> Where these results with 2.6.25 as well? ...John Heffner just couple of
> days ago fixed reordering related problem in TCP (in case that netem
> parametrization causes reordering), the fix is not even in 2.6.24.y, I
> think...
>
> --
> i.
>
^ permalink raw reply [flat|nested] 13+ messages in thread