[Cake] [PATCH] q_cake: Update xstats format to use per-tin structure

Kevin Darbyshire-Bryant kevin at darbyshire-bryant.me.uk
Wed Mar 7 06:08:11 EST 2018


There were some useful stats column re-alignment changes as well, wonder if you got those?

> On 7 Mar 2018, at 10:36, Toke Høiland-Jørgensen <toke at toke.dk> wrote:
> 
> Toke Høiland-Jørgensen <toke at toke.dk> writes:
> 
>> Kevin Darbyshire-Bryant <kevin at darbyshire-bryant.me.uk> writes:
>> 
>>> Hi All,
>>> 
>>> 
>>> 
>>>> On 7 Mar 2018, at 08:50, Toke Høiland-Jørgensen <toke at toke.dk> wrote:
>>>> 
>>>> Jonathan Morton <chromatix99 at gmail.com> writes:
>>>> 
>>>>>> On 6 Mar, 2018, at 11:06 pm, Toke Høiland-Jørgensen <toke at toke.dk> wrote:
>>>>>> 
>>>>>> ...on the iproute2 side the only
>>>>>> thing missing before we can attempt an upstream submission is an update
>>>>>> of the man page, as far as I can tell. Any volunteers to do that? :)
>>>>> 
>>>>> I could look into that tomorrow.
>>>> 
>>>> Awesome! Is there anything else you want to add on the kernel side
>>>> before we submit?
>>> 
>>> I think there was mention of the repo that Dave started on the last
>>> round of getting cake in upstream iproute2. There were a few man page
>>> tweaks & output formatting tweaks which ideally shouldn’t be lost.
>>> I’ve forked that fork on github here
>>> https://github.com/ldir-EDB0/iproute2-cake-next/tree/cake rebased onto
>>> release tag v4.15.0 and pulled in the the recent changes (overhead
>>> handling paradigm, xstats & the JSON output).
>>> 
>>> In theory I guess there should be no/little difference to tc-adv/master now but….
>>> 
>>> Anyway, it’s there if it’s of any interest… it’ll be the basis of what
>>> gets sent into LEDE a bit later today.
>> 
>> Please don't put something different into LEDE than what we're working
>> on upstreaming. It is difficult enough to keep track of the different
>> versions as they are. The tc-adv repo is already rebased on the upstream
>> iproute2-next, so if there is anything else that needs to be changed, it
>> should be changed in that repo and not in a fork...
> 
> I pulled in the man page changes from Dave's iproute2-next fork to the
> tc-adv repo now...
> 
> -Toke


Cheers,

Kevin D-B

012C ACB2 28C6 C53E 9775  9123 B3A2 389B 9DE2 334A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Message signed with OpenPGP
URL: <https://lists.bufferbloat.net/pipermail/cake/attachments/20180307/066ca284/attachment.sig>


More information about the Cake mailing list