[Cake] [PATCH] q_cake: Update xstats format to use per-tin structure
toke at toke.dk
Wed Mar 7 05:36:14 EST 2018
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...
More information about the Cake