Cake - FQ_codel the next generation
 help / color / mirror / Atom feed
From: Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk>
To: Vincent Frentzel <vincent@frentzel.eu>, <cake@lists.bufferbloat.net>
Subject: Re: [Cake] wash option not in tc-adv
Date: Mon, 30 Nov 2015 11:22:21 +0000	[thread overview]
Message-ID: <565C316D.8070804@darbyshire-bryant.me.uk> (raw)
In-Reply-To: <CAO2eGh25Jtw=XuK_9e1uS8yKjiz=dn7ES7EFL+ccAiDHdKS5WA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3097 bytes --]



On 30/11/15 10:22, Vincent Frentzel wrote:
> Hi Kevin,
>
> Thanks for the pointer. 
>
> Im 99.9% sure of having done that.
>
> 1) I have a cero repo in my feed list.
> 2) I ran ./scripts/feeds update -a; install -a
> 3) made oldconfig reviewed the changes (tc-adv and sch-cake are here).
> 4) ran make dirclean; make
Looks good to me.  The 'install -a' is not required as 'tc-adv' is
(hopefully) already installed, and you (equally hopefully) just updated
it.  To make sure an old version of 'tc-adv' isn't hanging around for
some reason, 'rm dl/tc-adv-4.1.0-*' is another 'brute force' option.
>
> I ll check the Makefile for the version stanza tonight.
The Makefile is king so worth checking. 
>
> I use as option "wash diffserv4". Is this correct? I use this config
> on my egress queue (facing lan side).
In fact, as 'diffserv4' is the cake default, you can reduce this to just
'wash'.  And I just did 'tc qdisc change dev ifb4eth0 root cake wash
diffserv4' and it all worked as expected.

You are flashing the box (or uploading the new 'tc-adv' package') right?
:-)  I know it's a silly question :-)
>
> Thanks for your help,
No problem.
>
> BR,
> Vincent
>
> On Mon, Nov 30, 2015 at 11:10 AM Kevin Darbyshire-Bryant
> <kevin@darbyshire-bryant.me.uk <mailto:kevin@darbyshire-bryant.me.uk>>
> wrote:
>
>     Hi Vincent,
>
>     I suspect you've not updated the 'tc-adv' package.  The package
>     definition contains amongst many things, the revision to checkout from
>     the git repository containing 'tc-adv'.  If that's not updated,
>     you get
>     the old version.  If you run (from your openwrt base directory)
>     './scripts/feeds update -a' this will update all the packages in all
>     your configured feeds to their latest versions.  As 'tc-adv' is (or
>     should be) an already configured package then that should be
>     sufficient.
>
>     I'm guessing you've the 'kmod-sched-cake' and 'tc-adv' packages in a
>     custom feed, probably something like 'cero'.  Take a look in
>     'feeds/cero/net/tc-adv/Makefile' and you should find the
>     PKG_SOURCE_VERSION is 'f789a4897dff988a4d08e39fd360e9468a60fa65'
>
>
>     I hope that helps.
>
>     Kevin
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>     On 30/11/15 06:41, Vincent Frentzel wrote:
>     > Hi Everyone,
>     >
>     > Been trying to make use of the wash option the whole week but it
>     seems
>     > tc-adv does not accept the param. I cleaned up my build env with
>     > distclean, started from scratch, no luck. Am I missing something?
>     >
>     > Cheers,
>     > Vincent
>     >
>     >
>     > _______________________________________________
>     > Cake mailing list
>     > Cake@lists.bufferbloat.net <mailto:Cake@lists.bufferbloat.net>
>     > https://lists.bufferbloat.net/listinfo/cake
>
>
>     _______________________________________________
>     Cake mailing list
>     Cake@lists.bufferbloat.net <mailto:Cake@lists.bufferbloat.net>
>     https://lists.bufferbloat.net/listinfo/cake
>



[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4816 bytes --]

  reply	other threads:[~2015-11-30 11:22 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-29 22:59 Vincent Frentzel
2015-11-30  6:41 ` Vincent Frentzel
2015-11-30 10:10   ` Kevin Darbyshire-Bryant
2015-11-30 10:22     ` Vincent Frentzel
2015-11-30 11:22       ` Kevin Darbyshire-Bryant [this message]
2015-11-30 11:31         ` Kevin Darbyshire-Bryant
2015-11-30 11:36           ` Dave Taht
2015-11-30 12:09             ` Kevin Darbyshire-Bryant
2015-11-30 21:15         ` Vincent Frentzel
2015-11-30 21:49           ` Kevin Darbyshire-Bryant
2015-12-01  0:18           ` Toke Høiland-Jørgensen

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=565C316D.8070804@darbyshire-bryant.me.uk \
    --to=kevin@darbyshire-bryant.me.uk \
    --cc=cake@lists.bufferbloat.net \
    --cc=vincent@frentzel.eu \
    /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