[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