Development issues regarding the cerowrt test router project
 help / color / mirror / Atom feed
From: Jonathan Morton <chromatix99@gmail.com>
To: Sebastian Moeller <moeller0@gmx.de>
Cc: "Klatsky, Carl" <carl_klatsky@cable.comcast.com>,
	Simon Barber <simon@superduper.net>,
	cake@lists.bufferbloat.net, cerowrt-devel@lists.bufferbloat.net,
	bloat <bloat@lists.bufferbloat.net>
Subject: Re: [Cerowrt-devel] [Bloat] heisenbug: dslreports 16 flow test vs cablemodems
Date: Mon, 18 May 2015 20:17:12 +0300	[thread overview]
Message-ID: <25D8D287-B5E9-49A9-B47E-4C7122884E1C@gmail.com> (raw)
In-Reply-To: <8E2F7EDC-3356-451A-8EFA-023A1EED4F0F@gmx.de>


> On 18 May, 2015, at 20:03, Sebastian Moeller <moeller0@gmx.de> wrote:
> 
>> Adding Diffserv and recommending that LEDBAT applications use the
>> “background” traffic class (CS1 DSCP) solves this problem more
>> elegantly.  The share of bandwidth used by BitTorrent (say) is then
>> independent of the number of flows it uses, and it also makes sense to
>> configure FQ for ideal flow isolation rather than for mitigation.
> 
> I wonder, for this to work well wouldn't we need to allow/honor at least CS1 marks on ingress? I remember there was some discussion about mislabeled traffic on ingress (Comcast I believe), do you see an easy way around that issue?

I don’t know much about the characteristics of this mislabelling.  Presumably though, Comcast is using DSCP remarking in an attempt to manage internal congestion.  If latency-sensitive and/or inelastic traffic is getting marked CS1, that would be a real problem, and Comcast would need leaning on to fix it.  It’s slightly less serious if general best-effort traffic gets CS1 markings.

One solution would be to re-mark the traffic at the CPE on ingress, using local knowledge of what traffic is important and which ports are associated with BitTorrent.  Unfortunately, the ingress qdisc runs before iptables, making that more difficult.  I think it would be necessary to do re-marking using an ingress action before passing it to the qdisc.  Either that, or a pseudo-qdisc which just does the re-marking before handing the packet up the stack.

I’m not sure whether it’s possible to attach two ingress actions to the same interface, though.  If not, the re-marking action module would also need to incorporate act_mirred functionality, or a minimal subset thereof.

 - Jonathan Morton


  reply	other threads:[~2015-05-18 17:17 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-14 19:13 [Cerowrt-devel] " Dave Taht
2015-05-15  2:48 ` Greg White
2015-05-15  4:44   ` Aaron Wood
2015-05-15  8:18     ` [Cerowrt-devel] [Bloat] " Eggert, Lars
2015-05-15  8:55       ` Sebastian Moeller
2015-05-15 11:10         ` [Cerowrt-devel] [Cake] " Alan Jenkins
2015-05-15 11:27       ` [Cerowrt-devel] " Bill Ver Steeg (versteb)
2015-05-15 12:19         ` Jonathan Morton
2015-05-15 12:44         ` Eggert, Lars
2015-05-15 13:09           ` Bill Ver Steeg (versteb)
2015-05-15 13:35             ` Jim Gettys
2015-05-15 14:36               ` Simon Barber
2015-05-18  3:30                 ` dpreed
2015-05-18  5:06                   ` Simon Barber
2015-05-18  9:06                     ` Bill Ver Steeg (versteb)
2015-05-18 11:42                     ` Eggert, Lars
2015-05-18 11:57                       ` luca.muscariello
2015-05-18 12:30                       ` Simon Barber
2015-05-18 15:03                         ` Jonathan Morton
2015-05-18 15:09                         ` dpreed
2015-05-18 15:32                           ` Simon Barber
2015-05-18 17:21                             ` [Cerowrt-devel] [Cake] " Dave Taht
2015-05-18 15:40                           ` [Cerowrt-devel] " Jonathan Morton
2015-05-18 17:03                             ` Sebastian Moeller
2015-05-18 17:17                               ` Jonathan Morton [this message]
2015-05-18 18:14                                 ` Sebastian Moeller
2015-05-18 18:37                                   ` Dave Taht
2015-05-19 16:25                           ` [Cerowrt-devel] [Cake] " Sebastian Moeller
2015-05-15 16:59               ` [Cerowrt-devel] " Dave Taht
2015-05-15 17:47   ` [Cerowrt-devel] " 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/cerowrt-devel.lists.bufferbloat.net/

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

  git send-email \
    --in-reply-to=25D8D287-B5E9-49A9-B47E-4C7122884E1C@gmail.com \
    --to=chromatix99@gmail.com \
    --cc=bloat@lists.bufferbloat.net \
    --cc=cake@lists.bufferbloat.net \
    --cc=carl_klatsky@cable.comcast.com \
    --cc=cerowrt-devel@lists.bufferbloat.net \
    --cc=moeller0@gmx.de \
    --cc=simon@superduper.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