Re: [Bloat] Adding CAKE "tc qdisc" options to NetworkManager

2020-10-05 Thread Toke Høiland-Jørgensen
Dave Collier-Brown writes: > Alas, once I looked under the covers, it doesn't look easy at all. > Definitely not a weekend project. > > I used to use QEF, and this reminds me of it. Looks eminently copy-paste'able from that sfq commit you linked, though :) -Toke ___

Re: [Bloat] Adding CAKE "tc qdisc" options to NetworkManager

2020-10-05 Thread Dave Collier-Brown
Alas, once I looked under the covers, it doesn't look easy at all. Definitely not a weekend project. I used to use QEF, and this reminds me of it. --dave On 2020-10-05 1:33 p.m., Jonathan Morton wrote: On 5 Oct, 2020, at 7:13 pm, David Collier-Brown wrote: By pure luck, I ended up chatting w

Re: [Bloat] Adding CAKE "tc qdisc" options to NetworkManager

2020-10-05 Thread Dave Collier-Brown
This appears to be adding a tree of static data structures so nmcli can understand what the options are, including what the defaults are. If so, I can probably stop it after the command parser runs, and not have a huge build-and-test-everything task. --dave On 2020-10-05 1:33 p.m., Jonathan M

Re: [Bloat] Adding CAKE "tc qdisc" options to NetworkManager

2020-10-05 Thread Jonathan Morton
> On 5 Oct, 2020, at 7:13 pm, David Collier-Brown wrote: > > By pure luck, I ended up chatting with one of the NetworkManager chaps, who > invited a merge request with the proper parameters for CAKE. > > He wrote > > Currently NM doesn't support configuring CAKE parameters. IOW, if you > set "

[Bloat] Adding CAKE "tc qdisc" options to NetworkManager

2020-10-05 Thread David Collier-Brown
By pure luck, I ended up chatting with one of the NetworkManager chaps, who invited a merge request with the proper parameters for CAKE. He wrote Currently NM doesn't support configuring CAKE parameters. IOW, if you set "root cake bandwidth 100Mbit", you will see in the tc output that cake was