From: David Miller <davem@davemloft.net>
To: jiri@resnulli.us
Cc: netdev@vger.kernel.org, nogahf@mellanox.com, jhs@mojatatu.com,
xiyou.wangcong@gmail.com, mlxsw@mellanox.com, andrew@lunn.ch,
vivien.didelot@savoirfairelinux.com, f.fainelli@gmail.com,
michael.chan@broadcom.com, ganeshgr@chelsio.com,
saeedm@mellanox.com, matanb@mellanox.com, leonro@mellanox.com,
idosch@mellanox.com, jakub.kicinski@netronome.com,
simon.horman@netronome.com, pieter.jansenvanvuuren@netronome.com,
john.hurley@netronome.com, alexander.h.duyck@intel.com,
ogerlitz@mellanox.com, john.fastabend@gmail.com
Subject: Re: [patch net-next 0/9] qdisc RED offload
Date: Wed, 08 Nov 2017 12:28:55 +0900 (KST) [thread overview]
Message-ID: <20171108.122855.673986719935863302.davem@davemloft.net> (raw)
In-Reply-To: <20171106062349.4873-1-jiri@resnulli.us>
From: Jiri Pirko <jiri@resnulli.us>
Date: Mon, 6 Nov 2017 07:23:40 +0100
> From: Jiri Pirko <jiri@mellanox.com>
>
> Nogah says:
>
> Add an offload support for RED qdisc for mlxsw driver.
> The first patch adds the ability to offload RED qdisc by using
> ndo_setup_tc. It gives RED three commands, to offload, change or delete
> the qdisc, to get the qdisc generic stats and to get it's RED xstats.
> There is no enforcement on a driver to offload or not offload the qdisc and
> it is up to the driver to decide.
> RED qdisc is first being created and only later graft to a parent (unless
> it is a root qdisc). For that reason the return value of the offload
> replace command that is called in the init process doesn't reflect actual
> offload state. The offload state is determined in the dump function so it
> can be reflected to the user. This function is also responsible for stats
> update.
>
> The patchses 2-3 change the name of TC_SETUP_MQPRIO & TC_SETUP_CBS to match
> with the new convention of QDISC prefix.
> The rest of the patchset is driver support for the qdisc. Currently only
> as root qdisc that is being set on the default traffic class. It supports
> only the following parameters of RED: min, max, probability and ECN mode.
> Limit and burst size related params are being ignored at this moment.
Series applied, thanks!
prev parent reply other threads:[~2017-11-08 3:29 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-06 6:23 [patch net-next 0/9] qdisc RED offload Jiri Pirko
2017-11-06 6:23 ` [patch net-next 1/9] net_sch: red: Add offload ability to RED qdisc Jiri Pirko
2017-11-07 1:39 ` Simon Horman
2017-11-06 6:23 ` [patch net-next 2/9] net_sch: mqprio: Change TC_SETUP_MQPRIO to TC_SETUP_QDISC_MQPRIO Jiri Pirko
2017-11-07 1:40 ` Simon Horman
2017-11-06 6:23 ` [patch net-next 3/9] net_sch: cbs: Change TC_SETUP_CBS to TC_SETUP_QDISC_CBS Jiri Pirko
2017-11-07 1:40 ` Simon Horman
2017-11-07 18:54 ` Vinicius Costa Gomes
2017-11-06 6:23 ` [patch net-next 4/9] mlxsw: reg: Add cwtp & cwtpm registers Jiri Pirko
2017-11-06 6:23 ` [patch net-next 5/9] mlxsw: spectrum: Support RED qdisc offload Jiri Pirko
2017-11-06 6:23 ` [patch net-next 6/9] mlxsw: reg: Add ext and tc-cong counter groups Jiri Pirko
2017-11-06 6:23 ` [patch net-next 7/9] mlxsw: spectrum: Collect tclass related stats periodically Jiri Pirko
2017-11-06 6:23 ` [patch net-next 8/9] mlxsw: spectrum: Support RED xstats Jiri Pirko
2017-11-06 6:23 ` [patch net-next 9/9] mlxsw: spectrum: Support general qdisc stats Jiri Pirko
2017-11-08 3:28 ` David Miller [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=20171108.122855.673986719935863302.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=alexander.h.duyck@intel.com \
--cc=andrew@lunn.ch \
--cc=f.fainelli@gmail.com \
--cc=ganeshgr@chelsio.com \
--cc=idosch@mellanox.com \
--cc=jakub.kicinski@netronome.com \
--cc=jhs@mojatatu.com \
--cc=jiri@resnulli.us \
--cc=john.fastabend@gmail.com \
--cc=john.hurley@netronome.com \
--cc=leonro@mellanox.com \
--cc=matanb@mellanox.com \
--cc=michael.chan@broadcom.com \
--cc=mlxsw@mellanox.com \
--cc=netdev@vger.kernel.org \
--cc=nogahf@mellanox.com \
--cc=ogerlitz@mellanox.com \
--cc=pieter.jansenvanvuuren@netronome.com \
--cc=saeedm@mellanox.com \
--cc=simon.horman@netronome.com \
--cc=vivien.didelot@savoirfairelinux.com \
--cc=xiyou.wangcong@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.