Development issues regarding the cerowrt test router project
 help / color / mirror / Atom feed
From: Andy Furniss <adf.lists@gmail.com>
To: Sebastian Moeller <moeller0@gmx.de>
Cc: "lartc@vger.kernel.org" <lartc@vger.kernel.org>,
	Alan Goodman <notifications@yescomputersolutions.com>,
	"cerowrt-devel@lists.bufferbloat.net"
	<cerowrt-devel@lists.bufferbloat.net>
Subject: Re: [Cerowrt-devel] Correctly calculating overheads on unknown connections
Date: Wed, 24 Sep 2014 23:48:48 +0100	[thread overview]
Message-ID: <54234A50.7050102@gmail.com> (raw)
In-Reply-To: <16C372C8-70C4-4E62-A059-BF2C65A88CF0@gmx.de>

Sebastian Moeller wrote:

>> Maybe you mean overhead calculated by a script?
>
> Well in cerowrt’s SQM-scripts we expose the stab options so users can
> take link layer and overhead into account. If you naively determine
> the overhead, either with the help of the scrips I posted earlier or
> by looking it up on a table (if the encapsulation options are known)
> you will end up not handling the kernel’s auto-added overhead well.
> Currently SQM scripts does not expose PPP devices only ge00
> (ethernet) so -14 seems currently the best recommendation in
> combination with “please test”.

Oh, OK - I know nothing about wrt.

> What I am curious after your message
> is what happens if the kernel terminates a pppoe connection but is
> connected to a “modem” via ethernet, what does the kernel do. And
> thanks to your pointers I know have an idea of how to test that ;)

Well I can't say I know - testing is always best.

I think we are "seeing" skbs just as they enter an interface - so what 
form they take depends on the particular interface they have just been 
made for.

It's possible to have multiple pppoes/vlans on an eth and use the eth 
normally at the same time. What you see I suppose depends on where you 
are "attached". I guess shaping a pppoe on the eth rather than on the 
actual ppp is doable with a bit of filtering - in which case you may 
need to allow for the +14 macs/ethertype and that 8 ppp are already in 
the payload - a totally untested theory :-)


>> On ppp skb->len = ip len
>>
>> On eth skb->len = ip len + 14
>>
>> On vlan skb->len = ip len + 18
>
> So this is the information I actually wanted to find and then somehow
> thought qdisc_pkt_len_init() was the place. Do you by chance have any
> pointer where this assignment is handled?

No, sorry I don't know the code.



      reply	other threads:[~2014-09-24 22:48 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <541C9527.1070105@yescomputersolutions.com>
     [not found] ` <541DA8B5.70701@gmail.com>
2014-09-20 17:55   ` Dave Taht
2014-09-20 22:29     ` Andy Furniss
2014-09-21 18:35     ` Sebastian Moeller
     [not found]       ` <CAK1m8mPBWyg-sR-ekZGUhsOG-0HoZd3eJ-Q6HJpSLyN-J90kHg@mail.gmail.com>
2014-09-21 21:40         ` Alan Goodman
2014-09-22  9:05         ` Sebastian Moeller
2014-09-22 10:01       ` Andy Furniss
2014-09-22 10:20         ` Sebastian Moeller
2014-09-22 13:09       ` Alan Goodman
2014-09-22 19:52         ` Sebastian Moeller
2014-09-22 23:02           ` Alan Goodman
2014-09-23  9:32             ` Sebastian Moeller
2014-09-23 15:10               ` Andy Furniss
2014-09-23 17:47                 ` Sebastian Moeller
2014-09-23 19:05                   ` Andy Furniss
2014-09-23 22:16                     ` Sebastian Moeller
2014-09-24  9:17                       ` Andy Furniss
2014-09-24 16:23                         ` Sebastian Moeller
2014-09-24 22:48                           ` Andy Furniss [this message]

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=54234A50.7050102@gmail.com \
    --to=adf.lists@gmail.com \
    --cc=cerowrt-devel@lists.bufferbloat.net \
    --cc=lartc@vger.kernel.org \
    --cc=moeller0@gmx.de \
    --cc=notifications@yescomputersolutions.com \
    /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