netdev - Re: [patch iproute2 00/11] tc: jsonify couple of qdics, filter and actions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20171126212612.GA1965@nanopsycho.orion>
Date:   Sun, 26 Nov 2017 22:26:12 +0100
From:   Jiri Pirko <jiri@...nulli.us>
To:     Stephen Hemminger <stephen@...workplumber.org>
Cc:     netdev@...r.kernel.org, jhs@...atatu.com, mlxsw@...lanox.com
Subject: Re: [patch iproute2 00/11] tc: jsonify couple of qdics, filter and
 actions

Sun, Nov 26, 2017 at 09:44:17PM CET, stephen@...workplumber.org wrote:
>On Sat, 25 Nov 2017 15:48:24 +0100
>Jiri Pirko <jiri@...nulli.us> wrote:
>
>> From: Jiri Pirko <jiri@...lanox.com>
>> 
>> An example json output:
>> 
>> $ tc -s -j filter show dev ens8 egress
>> [{
>>         "protocol": "ip",
>>         "pref": 6001,
>>         "kind": "flower",
>>         "chain": 0
>>     },{
>>         "protocol": "ip",
>>         "pref": 6001,
>>         "kind": "flower",
>>         "chain": 0,
>>         "options": {
>>             "handle": 1,
>>             "keys": {
>>                 "eth_type": "ipv4",
>>                 "dst_ip": "192.168.250.1"
>>             },
>>             "not_in_hw": true,
>>             "actions": [{
>>                     "order": 1,
>>                     "kind": "gact",
>>                     "control_action": {
>>                         "type": "drop"
>>                     },
>>                     "prob": {
>>                         "random_type": "none",
>>                         "control_action": {
>>                             "type": "pass"
>>                         },
>>                         "val": 0
>>                     },
>>                     "index": 1,
>>                     "ref": 1,
>>                     "bind": 1,
>>                     "installed": 1667830,
>>                     "last_used": 1667830,
>>                     "stats": {
>>                         "bytes": 0,
>>                         "packets": 0,
>>                         "drops": 0,
>>                         "overlimits": 0,
>>                         "requeues": 0,
>>                         "backlog": 0,
>>                         "qlen": 0,
>>                         "requeues": 0
>>                     },
>>                     "cookie": "a1b2c3d4aaaaaaaabb"
>>                 }
>>             }
>>         }
>>     }
>> ]
>> $ tc -s filter show dev ens8 egress
>> filter pref 6001 flower chain 0 
>> filter pref 6001 flower chain 0 handle 0x1 
>>   eth_type ipv4
>>   dst_ip 192.168.250.1
>>   not_in_hw
>>         action order 1: gact action drop
>>          random type none pass val 0
>>          index 1 ref 1 bind 1 installed 16689 sec used 16689 sec
>>         Action statistics:
>>         Sent 0 bytes 0 pkt (dropped 0, overlimits 0 requeues 0) 
>>         backlog 0b 0p requeues 0
>>         cookie a1b2c3d4aaaaaaaabb
>> 
>> ---
>> To be applied on top of my cookie fix patchset
>> 
>> Jiri Pirko (11):
>>   tc: jsonify qdisc core
>>   tc: jsonify stats2
>>   tc: jsonify fq_codel qdisc
>>   tc: jsonify htb qdisc
>>   tc: jsonify filter core
>>   tc: jsonify flower filter
>>   tc: jsonify matchall filter
>>   tc: jsonify actions core
>>   tc: jsonify gact action
>>   tc: jsonify mirred action
>>   tc: jsonify vlan action
>> 
>>  tc/f_flower.c   | 287 +++++++++++++++++++++++++++++++++-----------------------
>>  tc/f_matchall.c |  12 +--
>>  tc/m_action.c   |  22 +++--
>>  tc/m_gact.c     |  18 ++--
>>  tc/m_mirred.c   |  46 +++++++--
>>  tc/m_vlan.c     |  26 +++--
>>  tc/q_fq_codel.c |  25 +++--
>>  tc/q_htb.c      |  20 ++--
>>  tc/tc.c         |   5 +-
>>  tc/tc_filter.c  |  47 ++++++----
>>  tc/tc_qdisc.c   |  52 ++++++----
>>  tc/tc_util.c    |  66 +++++++++----
>>  tc/tc_util.h    |   1 +
>>  13 files changed, 396 insertions(+), 231 deletions(-)
>> 
>
>
>Applied, but other qdisc need some jsonification as well.

and classes/actions. I agree.