From: Jiri Pirko <jiri@resnulli.us>
To: Stephen Hemminger <stephen@networkplumber.org>
Cc: netdev@vger.kernel.org, jhs@mojatatu.com, mlxsw@mellanox.com
Subject: Re: [patch iproute2 00/11] tc: jsonify couple of qdics, filter and actions
Date: Sun, 26 Nov 2017 22:26:12 +0100 [thread overview]
Message-ID: <20171126212612.GA1965@nanopsycho.orion> (raw)
In-Reply-To: <20171126124417.6f97f325@xeon-e3>
Sun, Nov 26, 2017 at 09:44:17PM CET, stephen@networkplumber.org wrote:
>On Sat, 25 Nov 2017 15:48:24 +0100
>Jiri Pirko <jiri@resnulli.us> wrote:
>
>> From: Jiri Pirko <jiri@mellanox.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.
prev parent reply other threads:[~2017-11-26 21:26 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-25 14:48 [patch iproute2 00/11] tc: jsonify couple of qdics, filter and actions Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 01/11] tc: jsonify qdisc core Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 02/11] tc: jsonify stats2 Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 03/11] tc: jsonify fq_codel qdisc Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 04/11] tc: jsonify htb qdisc Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 05/11] tc: jsonify filter core Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 06/11] tc: jsonify flower filter Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 07/11] tc: jsonify matchall filter Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 08/11] tc: jsonify actions core Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 09/11] tc: jsonify gact action Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 10/11] tc: jsonify mirred action Jiri Pirko
2017-11-25 14:48 ` [patch iproute2 11/11] tc: jsonify vlan action Jiri Pirko
2017-11-26 20:44 ` [patch iproute2 00/11] tc: jsonify couple of qdics, filter and actions Stephen Hemminger
2017-11-26 21:26 ` Jiri Pirko [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171126212612.GA1965@nanopsycho.orion \
--to=jiri@resnulli.us \
--cc=jhs@mojatatu.com \
--cc=mlxsw@mellanox.com \
--cc=netdev@vger.kernel.org \
--cc=stephen@networkplumber.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).