netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: John Heffner <jheffner@psc.edu>
To: Stephen Hemminger <shemminger@linux-foundation.org>
Cc: David Miller <davem@davemloft.net>,
	ian.mcdonald@jandi.co.nz, rick.jones2@hp.com,
	netdev@vger.kernel.org
Subject: Re: NCR, was [PATCH] make _minimum_ TCP retransmission timeout configurable
Date: Wed, 29 Aug 2007 18:58:12 -0400	[thread overview]
Message-ID: <46D5FA04.1060600@psc.edu> (raw)
In-Reply-To: <20070829155106.43cf69eb@freepuppy.rosehill.hemminger.net>

Stephen Hemminger wrote:
> On Wed, 29 Aug 2007 15:28:12 -0700 (PDT)
> David Miller <davem@davemloft.net> wrote:
>> And reading NCR some more, we already have something similar in the
>> form of Alexey's reordering detection, in fact it handles exactly the
>> case NCR supposedly deals with.  We do not trigger loss recovery
>> strictly on the 3rd duplicate ACK, and we've known about and dealt
>> with the reordering issue explicitly for years.
>>
> 
> Yeah, it looked like another case of BSD RFC writers reinventing
> Linux algorithms, but it is worth getting the behaviour standardized
> and more widely reviewed.

I don't believe this was the case.  NCR is substantially different, and 
came out of work at Texas A&M.  The original (only) implementation was 
in Linux IIRC.  Its goal was to do better.  Their papers say it does. 
It might be worth looking at.

In my own experience with reordering, Alexey's code had some 
hard-to-track-down bugs (look at all the work Ilpo's been doing), and 
the relative simplicity of NCR may be one of the reasons it does well in 
tests.

   -John

  reply	other threads:[~2007-08-29 22:58 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-29 20:52 [PATCH] make _minimum_ TCP retransmission timeout configurable Rick Jones
2007-08-29 21:13 ` Eric Dumazet
2007-08-29 22:11   ` Rick Jones
2007-08-29 21:32 ` Ian McDonald
2007-08-29 21:46   ` David Miller
2007-08-29 22:10     ` Ian McDonald
2007-08-29 22:23       ` David Miller
2007-08-29 22:13     ` Stephen Hemminger
2007-08-29 22:28       ` David Miller
2007-08-29 22:51         ` Stephen Hemminger
2007-08-29 22:58           ` John Heffner [this message]
2007-08-29 22:59             ` NCR, was " David Miller
2007-08-29 22:32       ` Rick Jones
2007-08-29 22:29     ` Rick Jones
2007-08-29 22:35       ` David Miller
2007-08-29 22:48         ` John Heffner
2007-08-29 22:52           ` John Heffner
2007-08-29 22:53         ` Edgar E. Iglesias
2007-08-29 23:06         ` Rick Jones
2007-08-29 23:15           ` David Miller
2007-08-29 23:31             ` Rick Jones
2007-08-30  5:22               ` Krishna Kumar2
2007-08-30 17:10                 ` Rick Jones
2007-08-29 23:44             ` John Heffner
2007-09-05 19:04             ` Ilpo Järvinen
2007-09-06 20:39               ` David Miller
2007-08-29 22:09   ` Rick Jones
2007-08-29 22:20     ` David Miller
2007-08-29 22:33       ` Ian McDonald
2007-08-29 22:37         ` David Miller

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=46D5FA04.1060600@psc.edu \
    --to=jheffner@psc.edu \
    --cc=davem@davemloft.net \
    --cc=ian.mcdonald@jandi.co.nz \
    --cc=netdev@vger.kernel.org \
    --cc=rick.jones2@hp.com \
    --cc=shemminger@linux-foundation.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).