netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] tcp FRTO: in-order-only "TCP proxy" fragility workaround
@ 2008-08-25 16:45 Thomas Jarosch
  2008-08-26 12:03 ` Ilpo Järvinen
  0 siblings, 1 reply; 107+ messages in thread
From: Thomas Jarosch @ 2008-08-25 16:45 UTC (permalink / raw)
  To: Netdev; +Cc: netfilter-devel

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

Forward mail. Upgrading to KDE 4.1.0/kdepim4 from KDE 3.5.9
enabled HTML emails by default and I didn't notice it before.

----------  Forwarded Message  ----------

Subject: Re: [PATCH] tcp FRTO: in-order-only "TCP proxy" fragility workaround
Date: Monday, 25. August 2008
From: Thomas Jarosch <thomas.jarosch@intra2net.com>
To: "Ilpo Järvinen" <ilpo.jarvinen@helsinki.fi>

On Friday, 22. August 2008 23:18:44 Ilpo Järvinen wrote:
> In the meantime, can you check the attached patches. Besides the kernel
> patch, you need to build your own patched iproute2 as well to configure
> the features (ip tool among them is enough in case the build of some other
> part of the toolset fails like it did for me). I somewhat tested them, and
> the result seemed to be what I'd expect (I just forced RTOs with some
> netem heavy dropping and quickly glanced over the resulting packet
> patterns near RTO).

Your patches work fine.

I've noticed two small things:
1. Maybe it's a good idea to add a note above the tcp_use_frto() change
to explain that the value is negated. Took me a while to figure out
why there is no "!" in there :-)
2. Maybe rename the "features" option in iproute2 to "disable_features".
Then it would be more intuitive what it does.


btw: If you apply something to the iproute2 git tree,
I got a compiler error while testing the patch:

In file included from lnstat.c:40:
lnstat.h:28: error: field 'last_read' has incomplete type
lnstat.h:29: error: field 'interval' has incomplete type

Attached small patch fixes the issue.

Cheers,
Thomas


[-- Attachment #2: iproute2-fix-include-for-timeval.patch --]
[-- Type: text/x-patch, Size: 552 bytes --]

Fix this compile error:

In file included from lnstat.c:40:
lnstat.h:28: error: field 'last_read' has incomplete type
lnstat.h:29: error: field 'interval' has incomplete type

Signed-off-by: Thomas Jarosch <thomas.jarosch@intra2net.com>

diff -u -r iproute2-2.6.25/misc/lnstat.h iproute2.timeval/misc/lnstat.h
--- iproute2-2.6.25/misc/lnstat.h	Thu Apr 17 19:12:54 2008
+++ iproute2.timeval/misc/lnstat.h	Mon Aug 25 17:48:33 2008
@@ -2,6 +2,7 @@
 #define _LNSTAT_H
 
 #include <limits.h>
+#include <sys/time.h>
 
 #define LNSTAT_VERSION "0.02 041002"
 

^ permalink raw reply	[flat|nested] 107+ messages in thread

end of thread, other threads:[~2008-09-22 16:13 UTC | newest]

Thread overview: 107+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <47EA0DAB.7080205@securenet.de>
     [not found] ` <Pine.LNX.4.64.0807042251500.4142@blackhole.kfki.hu>
     [not found]   ` <200807071118.32988.thomas.jarosch@intra2net.com>
2008-07-07 13:18     ` TCP connection stalls under 2.6.24.7 Thomas Jarosch
2008-07-10 13:17       ` Jozsef Kadlecsik
2008-07-10 14:12         ` Thomas Jarosch
2008-07-10 21:21           ` Jozsef Kadlecsik
2008-07-11 14:33             ` Thomas Jarosch
2008-07-15 11:47               ` Thomas Jarosch
2008-07-15 16:10                 ` Thomas Jarosch
2008-07-15 18:30                   ` Dâniel Fraga
2008-07-31  4:47                     ` Dâniel Fraga
2008-07-31  7:39                       ` Ilpo Järvinen
2008-08-02 12:24                         ` Dâniel Fraga
2008-07-15 20:17                   ` Ilpo Järvinen
2008-07-16  8:07                     ` Thomas Jarosch
2008-07-16  9:03                     ` Thomas Jarosch
2008-07-17 13:55                       ` Ilpo Järvinen
2008-07-17 15:15                         ` Thomas Jarosch
2008-07-17 15:53                           ` Ilpo Järvinen
2008-07-18  9:14                             ` Thomas Jarosch
2008-07-18 13:55                               ` Ilpo Järvinen
2008-07-18 14:02                                 ` Thomas Jarosch
2008-07-19  7:35                                   ` Ilpo Järvinen
2008-07-25 10:00                                   ` Ilpo Järvinen
2008-07-25 13:00                                     ` Thomas Jarosch
2008-07-25 14:06                                       ` Ilpo Järvinen
2008-07-25 15:34                                         ` Thomas Jarosch
2008-07-31  7:39                                           ` Thomas Jarosch
2008-07-31 12:44                                             ` Dâniel Fraga
2008-07-31 13:47                                               ` Thomas Jarosch
2008-07-31 14:11                                                 ` Dâniel Fraga
2008-08-06 18:53                                                 ` Dâniel Fraga
2008-08-07  6:54                                                   ` Ilpo Järvinen
2008-08-07 11:50                                                     ` Denys Fedoryshchenko
2008-08-07 12:11                                                       ` Thomas Jarosch
2008-08-07 12:14                                                       ` Ilpo Järvinen
2008-08-07 12:23                                                         ` Denys Fedoryshchenko
2008-08-08  9:56                                                           ` Ilpo Järvinen
2008-08-08 10:32                                                             ` Denys Fedoryshchenko
2008-08-07 11:33                                                   ` [PATCH] tcp FRTO: in-order-only "TCP proxy" fragility workaround Ilpo Järvinen
2008-08-08  4:42                                                     ` Bill Fink
2008-08-08 10:32                                                       ` Ilpo Järvinen
2008-08-11 21:44                                                         ` David Miller
2008-08-12  7:46                                                           ` Thomas Jarosch
2008-08-12  8:18                                                             ` David Miller
2008-08-12 17:43                                                               ` Dâniel Fraga
2008-08-12 17:52                                                                 ` Ilpo Järvinen
2008-08-13 17:53                                                                   ` Dâniel Fraga
2008-08-13 18:34                                                                     ` Ilpo Järvinen
2008-08-15  4:34                                                                       ` Dâniel Fraga
2008-08-15  7:06                                                                         ` Ilpo Järvinen
2008-08-15 21:35                                                                           ` Dâniel Fraga
2008-08-15 22:06                                                                             ` Ilpo Järvinen
2008-08-15 23:57                                                                               ` Dâniel Fraga
2008-08-16  2:15                                                                               ` Dâniel Fraga
2008-08-16  7:10                                                                                 ` Ilpo Järvinen
2008-08-16 19:18                                                                                   ` Ilpo Järvinen
2008-08-17  0:36                                                                                     ` Dâniel Fraga
2008-08-19 10:38                                                                                       ` Ilpo Järvinen
2008-08-20  0:34                                                                                         ` Dâniel Fraga
2008-08-20  7:57                                                                                           ` Ilpo Järvinen
2008-08-20 12:37                                                                                           ` Ilpo Järvinen
2008-08-22 21:32                                                                                             ` Dâniel Fraga
2008-08-22 21:37                                                                                               ` David Miller
2008-08-23 14:14                                                                                                 ` Dâniel Fraga
2008-08-23 14:38                                                                                                   ` Ilpo Järvinen
2008-08-24 19:38                                                                                                     ` Dâniel Fraga
2008-08-26 14:10                                                                                                       ` Ilpo Järvinen
2008-08-26 14:32                                                                                                         ` Ilpo Järvinen
2008-08-26 17:18                                                                                                         ` Dâniel Fraga
2008-08-26 20:40                                                                                                           ` Ilpo Järvinen
2008-08-26 21:17                                                                                                             ` Dâniel Fraga
2008-08-27 10:22                                                                                                               ` Ilpo Järvinen
2008-08-27 19:51                                                                                                                 ` Dâniel Fraga
2008-08-27 20:32                                                                                                                   ` Ilpo Järvinen
2008-08-27 20:50                                                                                                                     ` Dâniel Fraga
2008-08-27 21:25                                                                                                                       ` Ilpo Järvinen
2008-08-27 21:42                                                                                                                         ` Dâniel Fraga
2008-08-27 22:24                                                                                                                           ` Dâniel Fraga
2008-08-28 21:49                                                                                                             ` Dâniel Fraga
2008-08-29 13:07                                                                                                               ` Ilpo Järvinen
2008-08-29 17:41                                                                                                                 ` Dâniel Fraga
2008-09-01  7:11                                                                                                                   ` Ilpo Järvinen
2008-08-30  6:56                                                                                                                 ` Dâniel Fraga
2008-09-01  7:11                                                                                                                   ` Ilpo Järvinen
2008-09-07  8:17                                                                                                                     ` Dâniel Fraga
2008-09-08 10:27                                                                                                                       ` Ilpo Järvinen
2008-09-08 20:20                                                                                                                         ` Dâniel Fraga
2008-09-11 13:44                                                                                                                           ` Ilpo Järvinen
2008-09-11 17:30                                                                                                                             ` Dâniel Fraga
2008-09-12 10:16                                                                                                                               ` Ilpo Järvinen
2008-09-13 23:31                                                                                                                                 ` Dâniel Fraga
2008-09-16 12:10                                                                                                                                   ` Ilpo Järvinen
2008-09-16 14:24                                                                                                                                     ` Dâniel Fraga
2008-09-17 10:23                                                                                                                                       ` Ilpo Järvinen
2008-09-18 20:35                                                                                                                                         ` Dâniel Fraga
2008-09-18 21:04                                                                                                                                           ` Ilpo Järvinen
2008-09-21  3:02                                                                                                                                             ` Dâniel Fraga
2008-09-22  4:23                                                                                                                                             ` Dâniel Fraga
2008-09-22 11:22                                                                                                                                               ` Ilpo Järvinen
2008-09-22 16:13                                                                                                                                                 ` Dâniel Fraga
2008-09-15 19:42                                                                                                                                 ` Dâniel Fraga
2008-09-11 18:12                                                                                                                             ` Dâniel Fraga
2008-08-15 21:59                                                                           ` Dâniel Fraga
2008-08-13  8:00                                                               ` Thomas Jarosch
2008-08-22 21:18                                                             ` Ilpo Järvinen
2008-08-11 21:41                                                       ` David Miller
2008-08-25 16:45 Thomas Jarosch
2008-08-26 12:03 ` Ilpo Järvinen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).