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

[Dave Taht <dave.taht@gmail.com>]

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

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@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)




On Wed, Sep 11, 2013 at 9:10 PM, Dave Taht <dave.taht@gmail.com> wrote:

>
>
>
> On Wed, Sep 11, 2013 at 1:36 AM, Sebastian Moeller <moeller0@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@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@nacktmulle:~# cat /tmp/debloat.log
>> fq_codel_ll
>> fq_codel_ll
>> fq_codel_ll
>> fq_codel_ll
>> root@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@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@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

[-- Attachment #2: Type: text/html, Size: 12343 bytes --]