CoDel AQM discussions
 help / color / mirror / Atom feed
From: Dave Taht <dave.taht@gmail.com>
To: Jonathan Morton <chromatix99@gmail.com>
Cc: "codel@lists.bufferbloat.net" <codel@lists.bufferbloat.net>,
	"Dave Täht" <dave.taht@bufferbloat.net>
Subject: Re: [Codel] [RFCv2 PATCH] codel: add ecn_target for when to drop rather than mark ecn packets
Date: Mon, 25 Jun 2012 11:25:13 -0700	[thread overview]
Message-ID: <CAA93jw6s16EQbQSGFvBj9aJhgqW9FqLfL=b0hokNkum_TKfaVw@mail.gmail.com> (raw)
In-Reply-To: <5AD96B37-CDEE-44B6-83CC-449C27FCBA76@gmail.com>

On Mon, Jun 25, 2012 at 10:48 AM, Jonathan Morton <chromatix99@gmail.com> wrote:
> My impression is that ECN ignorant flows do exist, because of stupid routers rather than malice, and that Dave considers this a problem worth tackling in codel.

Malice will exist. It is easy to create a set of flows that will
overwhelm either codel or fq_codel, with ecn enabled.

fq_codel does respond well to udp flooding, I note, far better than pfifo_fast.

>
> However I would disagree that the problem needs tackling here, at least for fq_codel which already segregates non-responsive flows from others. The ignorant flows will experience higher latency if this link is a bottleneck for them, that is all.

the first version of fq_codel had a separate codel queue per flow,
which would behave as you say. The lastest version uses the global
drop probability, which makes more sense in the general case.

I continue to prototype with qfq+codel which has the first behavior,
presently. I like the idea of a more perfect fluid model, especially
at the bandwidths I operate at (2-30Mbit), with the cpu horsepower
available to make more better choices than fq_codel can.


> Plain codel might need a more robust defence, but I have a different suggestion for that. Determine whether a prolonged bout of marking is being caused by ECN ignorant flows, and if so disable ECN marking (just drop) until the ignorant flows have clearly gone away.

Well, an example algo for that would be helpful. SFB tried to do this.

I note that codel operates independently of RTT and I am thinking hard
that RTT related issues need to be explored overall.

> The key to knowledge is not to rely on others to teach you it.

I am an experimenter first and foremost. My inspirations are more from
edison than tesla. Queue behavior is rarely intuitive, and the only
way to build intuition, for me, at least, is to design an experiment
and observe what happens.

Kelly Johnson could "see air". Perhaps van and kathie can "see
packets". Me, on the other hand...

>
> On 25 Jun 2012, at 18:45, Eric Dumazet <eric.dumazet@gmail.com> wrote:
>
>> On Mon, 2012-06-25 at 08:30 -0700, Dave Taht wrote:
>>
>>> It was the way I was leaning until I observed me dropping ecn enabled
>>> mosh, ssh, and babel packets , which tend to be small, so I started
>>> thinking in terms of dropping ecn packets on a graduated packet size
>>> scale after exceeding target. ESPECIALLY in case of overload you want
>>> command and control packets to get through.
>>>
>>
>> Don't try to add in CoDel things that should be done at another layer.
>> There is no classification in CoDel : Its a FIFO.
>>
>> Only way is to prioritize control packets if you need to, and use
>> several queues.
>>
>> I dont understand... Are you saying that you feel the need to drop
>> packets instead of marking them in a 'good citizen world' (none of your
>> flow pretends to use ECN to avoid drops) ?
>>
>>
>>
>> _______________________________________________
>> Codel mailing list
>> Codel@lists.bufferbloat.net
>> https://lists.bufferbloat.net/listinfo/codel



-- 
Dave Täht
http://www.bufferbloat.net/projects/cerowrt/wiki - "3.3.8-6 is out
with fq_codel!"

  reply	other threads:[~2012-06-25 18:25 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-25  5:00 [Codel] [RFCv2 PATCH] iproute2: Add ecn_target option to codel and fq_codel Dave Täht
2012-06-25  5:00 ` [Codel] [RFCv2 PATCH] codel: add ecn_target for when to drop rather than mark ecn packets Dave Täht
2012-06-25  5:22   ` Eric Dumazet
2012-06-25 14:50     ` Dave Taht
2012-06-25 15:20       ` Eric Dumazet
2012-06-25 15:30         ` Dave Taht
2012-06-25 15:45           ` Eric Dumazet
2012-06-25 17:48             ` Jonathan Morton
2012-06-25 18:25               ` Dave Taht [this message]
2012-06-25 18:51                 ` Eric Dumazet
2012-06-25  5:24 ` [Codel] [RFCv2 PATCH] iproute2: Add ecn_target option to codel and fq_codel Eric Dumazet
2012-06-25  5:29 ` Eric Dumazet
2012-06-25 14:54   ` Dave Taht

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='CAA93jw6s16EQbQSGFvBj9aJhgqW9FqLfL=b0hokNkum_TKfaVw@mail.gmail.com' \
    --to=dave.taht@gmail.com \
    --cc=chromatix99@gmail.com \
    --cc=codel@lists.bufferbloat.net \
    --cc=dave.taht@bufferbloat.net \
    /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