From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [PATCH iproute2-next 0/9] TC more JSON support Date: Fri, 13 Jul 2018 16:48:28 -0400 Message-ID: <9ad5c235-8c66-4a62-0517-af4ae6bdddac@gmail.com> References: <20180709194856.18922-1-stephen@networkplumber.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Stephen Hemminger , Jiri Pirko , Jakub Kicinski , Cong Wang , Jamal Hadi Salim To: Stephen Hemminger , netdev@vger.kernel.org Return-path: Received: from mail-pf0-f195.google.com ([209.85.192.195]:37421 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731342AbeGMVEs (ORCPT ); Fri, 13 Jul 2018 17:04:48 -0400 Received: by mail-pf0-f195.google.com with SMTP id a26-v6so1178281pfo.4 for ; Fri, 13 Jul 2018 13:48:32 -0700 (PDT) In-Reply-To: <20180709194856.18922-1-stephen@networkplumber.org> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 7/9/18 3:48 PM, Stephen Hemminger wrote: > From: Stephen Hemminger > > Update core of TC command and library to do more JSON. > Most of this patch set is about getting tc utility functions > to be more friendly to the json_print infrastructure. > > Stephen Hemminger (9): > tc: use JSON in error handling > tc: use const char in util > tc: convert stats print to json > tc/cbq: use sprint_rate > tc/util: remove print_rate > tc/util: remove unused print_size > tc/util: remove unused print_time > tc/util: add print helpers for JSON > tc/sfq: add json support > > tc/q_cbq.c | 15 ++++----- > tc/q_sfq.c | 65 +++++++++++++++++++++--------------- > tc/tc.c | 19 ++++++----- > tc/tc_util.c | 94 +++++++++++++++++++++++++++------------------------- > tc/tc_util.h | 11 +++--- > 5 files changed, 109 insertions(+), 95 deletions(-) > I'd prefer some tc folks to take a look at the json output and verify everything is good. Jamal, Jiri, Jakub, Cong, others? There is a second set with 30 patches as well.