[Cake] [PATCH 2/2] tc print_uint format fixes

Toke Høiland-Jørgensen toke at toke.dk
Sun Mar 11 17:22:21 EDT 2018


Kevin Darbyshire-Bryant <kevin at darbyshire-bryant.me.uk> writes:

>> On 11 Mar 2018, at 20:50, Toke Høiland-Jørgensen <toke at toke.dk> wrote:
>> 
>> Kevin Darbyshire-Bryant <ldir at darbyshire-bryant.me.uk> writes:
>> 
>>> Signed-off-by: Kevin Darbyshire-Bryant <ldir at darbyshire-bryant.me.uk>
>>> ---
>>> tc/tc_qdisc.c |  3 ++-
>>> tc/tc_util.c  | 24 ++++++++++++------------
>>> 2 files changed, 14 insertions(+), 13 deletions(-)
>> 
>> Care to submit this to upstream iproute2? Amending the commit message
>> with an explanation of why the change is needed and sending the same
>> patch to netdev at vger.kernel.org should be enough, I think :)
>
> I wanted a sanity check first :-)  And there’s more to do.  A LOT more!
>
> Is it sane?

Apart from the fact that it is really grating that a function called
"print_uint()" needs this weird 64-bit format string include. But that
is probably an issue that upstream should deal with.

Also, the inttypes.h include is missing from tc_util.c in your patch...

Other than that, compiles and works for me on x86_64...

-Toke


More information about the Cake mailing list