From mboxrd@z Thu Jan 1 00:00:00 1970 From: Phil Sutter Subject: Re: [iproute PATCH] tc: htb: Print default value in hex Date: Mon, 22 Oct 2018 19:28:01 +0200 Message-ID: <20181022172801.GG30406@orbyte.nwl.cc> References: <20181019154255.25749-1-phil@nwl.cc> <20181022095623.6ffabb31@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jiri Pirko , netdev@vger.kernel.org To: Stephen Hemminger Return-path: Received: from orbyte.nwl.cc ([151.80.46.58]:46046 "EHLO orbyte.nwl.cc" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728629AbeJWBr1 (ORCPT ); Mon, 22 Oct 2018 21:47:27 -0400 Content-Disposition: inline In-Reply-To: <20181022095623.6ffabb31@xeon-e3> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Oct 22, 2018 at 09:56:23AM -0700, Stephen Hemminger wrote: > On Fri, 19 Oct 2018 17:42:55 +0200 > Phil Sutter wrote: > > > Value of 'default' is assumed to be hexadecimal when parsing, so > > consequently it should be printed in hex as well. This is a regression > > introduced when adding JSON output. > > > > Fixes: f354fa6aa5ff0 ("tc: jsonify htb qdisc") > > Signed-off-by: Phil Sutter > > --- > > tc/q_htb.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tc/q_htb.c b/tc/q_htb.c > > index c8b2941d945b7..c69485db8ef7d 100644 > > --- a/tc/q_htb.c > > +++ b/tc/q_htb.c > > @@ -332,7 +332,7 @@ static int htb_print_opt(struct qdisc_util *qu, FILE *f, struct rtattr *opt) > > if (RTA_PAYLOAD(tb[TCA_HTB_INIT]) < sizeof(*gopt)) return -1; > > > > print_int(PRINT_ANY, "r2q", "r2q %d", gopt->rate2quantum); > > - print_uint(PRINT_ANY, "default", " default %u", gopt->defcls); > > + print_uint(PRINT_ANY, "default", " default %x", gopt->defcls); > > print_uint(PRINT_ANY, "direct_packets_stat", > > " direct_packets_stat %u", gopt->direct_pkts); > > if (show_details) { > > The output should be in hex. Because you used print_uint it will still be in decimal > in the JSON output. The correct fix here is to use print_0xhex instead. > > Please fix that, test it, and resubmit. I did test, but not JSON output. It is not a regression in the latter because it has been like this since JSON output was added to HTB. Assuming the output is programmatically parsed, it shouldn't matter too much as well. Grepping through the code I found three more cases where print_uint() was used for data printed in %x. Should I fix those as well? Thanks, Phil