netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hagen Paul Pfeifer <hagen@jauu.net>
To: Jerry Chu <hkchu@google.com>
Cc: Arnd Hannemann <hannemann@nets.rwth-aachen.de>,
	Eric Dumazet <eric.dumazet@gmail.com>,
	<ilpo.jarvinen@helsinki.fi>, <davem@davemloft.net>,
	<netdev@vger.kernel.org>
Subject: Re: [PATCH] TCP_FAILFAST: a new socket option to timeout/abort a connection quicker
Date: Wed, 25 Aug 2010 10:21:39 +0200	[thread overview]
Message-ID: <cb32d2a8676d5447e04882302228c4b6@localhost> (raw)
In-Reply-To: <AANLkTikAGKObYnczsVoKnUOewJBp2J2036yOD7TncA0F@mail.gmail.com>


On Tue, 24 Aug 2010 15:13:44 -0700, Jerry Chu <hkchu@google.com> wrote:

> On Tue, Aug 24, 2010 at 9:28 AM, Hagen Paul Pfeifer <hagen@jauu.net>

wrote:



> So it looks like the user timeout can be used in either senario

(shortening

> or lengthening) and in both cases is a lower bound, i.e., the connection

> should abort at or shortly after the specified user timeout.

>

> In this case does it make sense to combine the two? Will your TCP_UTO

> patch be ready anytime soon?

> 

> Again an alternative is to allow configuring tcp_retries2 and

TCP_RTO_MAX

> directly. I'm open to suggestion but we'd like to get something in

sooner.



Hello Chu! My Idea: you provide functionality to modify the user timeout.

The interface should be generic enough to allow small as well as large - up

to 22 days - values. This interface should be sufficient for you and later

for me. Afterwards I provide an patch which apply on your groundwork. My

patch handle TCP UTO specific functionality like TCP option protocol

handling functionality, socket option, permissions, lower- and upper

bounds, ...



Did you check interactions with other TCP timers like keep-alive timer? 



Today in the evening I will focus on TCP Quick ACK modifications. After

that I am in the Alps for vacation for 5 days. Later on I will work on the

patch (the patch is in a good state, modification and testing should

consume only 2 evenings - hopefully ;-).



Cherry, is this ok for you?



Hagen

  reply	other threads:[~2010-08-25  8:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-24  6:20 [PATCH] TCP_FAILFAST: a new socket option to timeout/abort a connection quicker H.K. Jerry Chu
2010-08-24  6:44 ` Eric Dumazet
2010-08-24  8:04   ` Arnd Hannemann
2010-08-24  9:10     ` Hagen Paul Pfeifer
2010-08-24 14:58       ` Arnd Hannemann
2010-08-24 16:28         ` Hagen Paul Pfeifer
2010-08-24 22:13           ` Jerry Chu
2010-08-25  8:21             ` Hagen Paul Pfeifer [this message]
2010-08-25 20:20               ` Jerry Chu
2010-08-25 22:59                 ` Hagen Paul Pfeifer
2010-08-26  1:49                   ` Jerry Chu
2010-08-26  6:01                     ` Lars Eggert
2010-08-26  7:12                       ` Arnd Hannemann
2010-08-26  7:42                         ` Hagen Paul Pfeifer
2010-08-26  7:27                       ` Hagen Paul Pfeifer
2010-08-24 21:56     ` Jerry Chu
2010-08-24 20:47   ` Jerry Chu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cb32d2a8676d5447e04882302228c4b6@localhost \
    --to=hagen@jauu.net \
    --cc=davem@davemloft.net \
    --cc=eric.dumazet@gmail.com \
    --cc=hannemann@nets.rwth-aachen.de \
    --cc=hkchu@google.com \
    --cc=ilpo.jarvinen@helsinki.fi \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).