[Cerowrt-devel] 3.10.11-2 development build debloat bug

Sebastian Moeller moeller0 at gmx.de
Fri Sep 13 06:01:40 EDT 2013


Hi Dave,



On Sep 12, 2013, at 06:18 , Dave Taht <dave.taht at gmail.com> wrote:

> Well, actually, I don't know when the syntax changed, but now the -b option needs a 
>                             -
> for reading from standard input. Boy this file is getting crufty...
> 
> cero2 at snapon:~/src/ceropackages-3.3/net/debloat/files$ git diff debloat
> diff --git a/net/debloat/files/debloat b/net/debloat/files/debloat
> index e675008..d1cf939 100755
> --- a/net/debloat/files/debloat
> +++ b/net/debloat/files/debloat
> @@ -29,7 +29,7 @@ params = { "MDISC", "BIGDISC", "NORMDISC", "BINS", "MAX_HWQ_BY
>  -- Useful defaults
>  
>  env = { ["TC"] = "/sbin/tc", 
> -       ["TCARG"] = "-b", 
> +       ["TCARG"] = "-b -", 
>         ["INSMOD"] = "/sbin/modprobe",
>          ["ETHTOOL"] = "/sbin/ethtool", 
>         ["LSMOD"] = "/sbin/lsmod",
> (END)

	Thanks that fixed the non-ge00 interfaces. As to the abysmal performance with macosx over 5GHz wlan, that still is there, but I suspect the macbok to be the culprit here (plus the wlan connection is somewhere ion the edge between changing transmigrates so might be a moving target).


Many thanks
	Sebastian



> 
> 
> 
> 
> On Wed, Sep 11, 2013 at 9:10 PM, Dave Taht <dave.taht at gmail.com> wrote:
> 
> 
> 
> On Wed, Sep 11, 2013 at 1:36 AM, Sebastian Moeller <moeller0 at gmx.de> wrote:
> Hi Dave,
> 
> so I ant for the shiny 3.10.11-2, worked great (using Fred's mtd -r method, thanks Fred)
> 
> 
> On Sep 10, 2013, at 02:28 , Dave Taht <dave.taht at gmail.com> wrote:
> 
> > + readlink fix (hopefully fixes sysupgrade)
>         I guess this will be testable at the next version update...
> 
> > + usual merge with openwrt head (tons of ath9k changes)
>         Oh, as if you knew that I had a number of:
>                 ath: phy1: Failed to stop TX DMA, queues=
>         lines in dmesg, quick testing did not allow me to get those with 3.10.11-2, but I will need to test further...
> 
> > + dnsmasq 2.67test10
> > + ipv6subtrees back in
> > + the final htb atm patches
> 
>         So I tested tc_stab and htb_private from the AQM tab, both work equally well.
> 
> > + eliminated maxpacket check in codel
> >
> > - did not fold in edumazet's new fq code
> > - 100% totally untested. May a braver soul than I give it a shot. I won't be near a cero box til thursday, otherwise.
> >
> > http://snapon.lab.bufferbloat.net/~cero2/cerowrt/wndr/3.10.10-1/
> >
> > -I'm not sure if I got the "last" of the aqm gui patches in there or not…
> 
>         I think so, at least it works :)
> 
> >
> > ...
> >
> > Anyway... I had hopes to get a stable release out in august. I AM very happy about the major stuff that got fixed, instead... but...
> >
> > Since we didn't... I now have a ton of other matters piled up. Not least of which is a pending trip to england and the eu.
> 
>         Have a great trip.
> 
> 
> oh, this guilts me!  ;)
>  
> >
> > So for the next month I don't see how I'm going to be able to put more than a day a week into cerowrt. Tops. So I have tagged up this "release" and pushed all the baked portions of the sources to github.
> 
>         Thanks a lot.
> 
> > I'm still a little dubious of the ipv6 subtrees bit….
> 
>         RRUL-Testing against Toke's server shows great results, local rrul testing between osx 10.8.4 machine on sw10 to a net server running on an linux x86_64 3.10.1 machine on se00 is quite bad though (I assume I now run into the wifi issues on the macbook or the router as this is the first time I test against a machine with considerable larger bandwidth than the wlan). The rrul plots still are quite interesting, as I could nicely see anticoorelation between up and down bandwidth (shared medium)
> 
> 
> No, its possible we have a new problem...
>  
> If I get round to it I would like to re-enable fq_codel on all interfaces (now it is just running at ge00/ifb0) to see whether this can ameliorate the issue at least a bit.
> 
> Note, I enabled the log for /usr/sbin/deblaot (by editing/etc/hotplug.d/iface/00-debloat) and got the following:
> root at nacktmulle:~# cat /tmp/debloat.log
> fq_codel_ll
> fq_codel_ll
> fq_codel_ll
> fq_codel_ll
> root at nacktmulle:~# cat /tmp/debloat2.log
> 
> 
> No. This behavior is new.
> 
> I used to be able to
> 
> contents of /tmp/wtf:
> 
> qdisc change dev sw10 parent 1:1 handle 10 fq_codel limit 500 quantum 1000
> qdisc change dev sw10 parent 1:2 handle 20 fq_codel limit 1000 quantum 1000
> qdisc change dev sw10 parent 1:3 handle 30 fq_codel limit 1000 quantum 1000
> qdisc change dev sw10 parent 1:4 handle 40 fq_codel limit 1000 quantum 1000
> 
> and then
> cat /tmp/wtf | tc
> 
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> 
> 
> 
> Which is how the debloat script historically did everything. Now the only syntax that works is:
> 
> root at cerowrt:/etc/hotplug.d/iface# tc -b /tmp/wtf
> 
> I think this is a regression in tc
> 
>  
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> Usage: tc [ OPTIONS ] OBJECT { COMMAND | help }
>        tc [-force] -batch filename
> where  OBJECT := { qdisc | class | filter | action | monitor }
>        OPTIONS := { -s[tatistics] | -d[etails] | -r[aw] | -p[retty] | -b[atch] [filename] }
> 
> 
> Not sure whether that is new, as I never enabled the logs before. I guess I will see what causes these… (I assume an improper set of arguments to tc). And now I am trying to ind my way around debloat, but lua is totally new to me...
> 
> 
> Best Regards & many thanks
>         Sebastian
> 
> 
> 
> >
> >
> >
> > --
> > Dave Täht
> >
> > Fixing bufferbloat with cerowrt: http://www.teklibre.com/cerowrt/subscribe.html
> > _______________________________________________
> > Cerowrt-devel mailing list
> > Cerowrt-devel at lists.bufferbloat.net
> > https://lists.bufferbloat.net/listinfo/cerowrt-devel
> 
> 
> 
> 
> -- 
> Dave Täht
> 
> Fixing bufferbloat with cerowrt: http://www.teklibre.com/cerowrt/subscribe.html
> 
> 
> 
> -- 
> Dave Täht
> 
> Fixing bufferbloat with cerowrt: http://www.teklibre.com/cerowrt/subscribe.html




More information about the Cerowrt-devel mailing list