Cake - FQ_codel the next generation
 help / color / mirror / Atom feed
From: moeller0 <moeller0@gmx.de>
To: Jonathan Morton <chromatix99@gmail.com>
Cc: "Toke Høiland-Jørgensen" <toke@toke.dk>, cake@lists.bufferbloat.net
Subject: Re: [Cake] cake/tc - removal of atm/ptm/ethernet specific overhead keywords
Date: Thu, 2 Jun 2016 20:53:00 +0200	[thread overview]
Message-ID: <0EDA2CBF-E6FF-449F-ACDF-909526CEE140@gmx.de> (raw)
In-Reply-To: <D88BC3DB-AE48-4892-B357-C197AFFBE3C8@gmail.com>

Dear Jonathan,


> On Jun 2, 2016, at 19:40 , Jonathan Morton <chromatix99@gmail.com> wrote:
> 
> 
>> On 2 Jun, 2016, at 18:42, moeller0 <moeller0@gmx.de> wrote:
>> 
>> I stopped believing this is “simple” a long time ago; I also failed to come up with a method to measure the actual overhead on VDSL links…
> 
> If it’s impossible to measure, then by definition it doesn’t have a significant performance impact,

	That is unfortunately not really true… it really is just that, hard to measure. And that might be more a statement about my methodological/creative gaps than anything else.

> and thus it doesn’t matter if it is sloppily estimated.  In that context an overestimate of overhead is safe.

	Safe, only if over-estimated; I would prefer if the user would specify the bandwidth “sacrifice” once when he sets the shapers brutto rate ;)

> 
> This is really a user-interface problem.  I think it would be very helpful for LuCI to get a basic, easy “conservative overhead compensation” button that simply turns on the “conservative” keyword for both ingress and egress.  That’s the bare minimum.

	That will, as currently implemented, drag in the ATM 48/53 cell compensation for all users, eating roughly 9% bandwidth, not sure we would do our users a favor with such a default policy. 


> 
> We can discuss details of improving granularity of that setting at greater leisure, preferably in the context of a proper redesign of the SQN configuration.
> 
> And let’s be absolutely clear: I will NOT drop the shortcut keywords from tc.

	So far I have tried to raise a number of in-detail issues (in other mails and postings) that seem to indicate that our assumptions about per-packet-overheads are not as robust and complete as they should be if we want to use them to justify changing the defaults.
	So, if you do not want to drop the keywords then please make sure they are well-documented and consistent. Currently it is not self-evident which numerical overhead corresponds to which keyword and that mapping should be crystal clear. And I would like to add that “conservative”-keyword needs special care in documentation as it is the only keyword that compounds per-packet-overhead and specific framing (ATM in this case, we simply ignore PTM’s 64/65 encoding (and rightly so)), this should be obvious from the documentation (man page and the output of “tc qdisc add cake help”) and in my opinion the keyword itself. I would propose as first half-measure “conservative_adsl” or “conservative_atm” to signify the compound nature.
	

Best Regards
 	Sebastian

> 
> - Jonathan Morton
> 


  reply	other threads:[~2016-06-02 18:53 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-02  9:37 Kevin Darbyshire-Bryant
2016-06-02 14:22 ` Jonathan Morton
2016-06-02 14:27   ` Toke Høiland-Jørgensen
2016-06-02 14:49     ` Jonathan Morton
2016-06-02 15:42       ` moeller0
2016-06-02 17:40         ` Jonathan Morton
2016-06-02 18:53           ` moeller0 [this message]
2016-06-02 18:55             ` Jonathan Morton
2016-06-02 19:17               ` moeller0
2016-06-02 14:59     ` moeller0
2016-06-02 15:10       ` Jonathan Morton
2016-06-02 15:33         ` moeller0
2016-06-02 14:51   ` moeller0

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/cake.lists.bufferbloat.net/

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

  git send-email \
    --in-reply-to=0EDA2CBF-E6FF-449F-ACDF-909526CEE140@gmx.de \
    --to=moeller0@gmx.de \
    --cc=cake@lists.bufferbloat.net \
    --cc=chromatix99@gmail.com \
    --cc=toke@toke.dk \
    /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