CoDel AQM discussions
 help / color / mirror / Atom feed
From: Rick Jones <rick.jones2@hp.com>
To: Jim Gettys <jg@freedesktop.org>
Cc: Eric Dumazet <edumazet@google.com>,
	Dave Taht <dave.taht@bufferbloat.net>,
	codel@lists.bufferbloat.net
Subject: Re: [Codel] [PATCH 1/2] codel: Controlled Delay AQM
Date: Mon, 07 May 2012 20:32:55 -0700	[thread overview]
Message-ID: <4FA893E7.2010603@hp.com> (raw)
In-Reply-To: <4FA88FA9.7030804@freedesktop.org>

On 05/07/2012 08:14 PM, Jim Gettys wrote:
> There are three cases here:
>
>   1.   you initiate a TCP session;
>   2.   you receive a TCP session;
>   3.   you mark a packet in transit through a system (you are routing).
>
> Case 2 is known safe, and is deploying rapidly in the Internet.  Linux
> defaults to 2): if you talk to it asking for ECN, it responds.
>
> The problem is that some networks screw up the ecn bits.  And worse yet,
> there at least used to be some hardware out in the great internet that
> would go belly up if it saw marked packets.
>
> So it's 1 and 3 that we might get in trouble about.  There is some way
> to turn on 1( in Linux already; using ECN is negotiated as part of the open.
>
> But we better not do 3) (mark for ECN rather than drop) without knowing
> if it "safe" to do so.  Steve Bauer and his collaborators have more
> understanding than anyone yet on this list.

Then get the folks at queue.acm.org on the ball and get the paper 
published already :) :) :)

But I'm still confused.  If Case 2 is "safe" is it safe only because 
no-one initiates ECN making Case 2 a noop?  IE, it is not known to be 
safe to set tcp_ecn=1?  That being the case, I'd still think that codel 
going "if I see ECT and would drop, I'll mark" is OK because it will 
only see ECT if the client set tcp_ecn=1, and presumably then, the owner 
of the client will not have set tcp_ecn=1 unless they know it is "safe" 
to do so.  Particularly with Eric's recent patch to upstream to only 
accept an ECN request in a SYN if the ECN bits in the IP header are clear.

rick

  reply	other threads:[~2012-05-08  3:32 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-05 19:32 Dave Taht
2012-05-05 19:32 ` [Codel] [PATCH 2/2] codel: RED is dead Dave Taht
2012-05-05 19:47 ` [Codel] [PATCH 1/2] codel: Controlled Delay AQM Dave Taht
2012-05-05 20:05   ` Eric Dumazet
2012-05-07 16:43 ` Rick Jones
2012-05-07 17:10   ` Eric Dumazet
2012-05-07 17:18     ` Dave Taht
2012-05-07 17:27     ` Rick Jones
2012-05-07 17:34       ` Eric Dumazet
2012-05-07 17:52         ` Dave Taht
2012-05-07 18:01           ` Eric Dumazet
2012-05-07 18:09             ` Eric Dumazet
2012-05-07 18:15               ` Eric Dumazet
2012-05-07 18:32           ` Jim Gettys
2012-05-07 18:44             ` Dave Taht
2012-05-07 18:50               ` Jim Gettys
2012-05-07 19:16                 ` Eric Dumazet
2012-05-07 19:36                   ` Jim Gettys
2012-05-07 20:03                     ` Eric Dumazet
2012-05-07 20:55                       ` dave taht
2012-05-08  1:15                       ` Jim Gettys
2012-05-08  1:20                         ` Andrew McGregor
2012-05-08  1:45                           ` Jim Gettys
2012-05-08  2:16                             ` Rick Jones
2012-05-08  3:14                               ` Jim Gettys
2012-05-08  3:32                                 ` Rick Jones [this message]
2012-05-08  3:40                                   ` Kathleen Nichols
2012-05-08  4:14                                   ` Jim Gettys
2012-05-08  5:10                                     ` Eric Dumazet
2012-05-08 10:25                                       ` Jim Gettys
2012-05-08 13:57                                         ` Kathleen Nichols
2012-05-08 14:04                                           ` Jim Gettys

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

  List information: https://lists.bufferbloat.net/postorius/lists/codel.lists.bufferbloat.net/

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

  git send-email \
    --in-reply-to=4FA893E7.2010603@hp.com \
    --to=rick.jones2@hp.com \
    --cc=codel@lists.bufferbloat.net \
    --cc=dave.taht@bufferbloat.net \
    --cc=edumazet@google.com \
    --cc=jg@freedesktop.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