From: David Miller <davem@davemloft.net>
To: jiri@resnulli.us
Cc: netdev@vger.kernel.org, nogahf@mellanox.com, yuvalm@mellanox.com,
idosch@mellanox.com, mlxsw@mellanox.com, jhs@mojatatu.com,
xiyou.wangcong@gmail.com, kubakici@wp.pl
Subject: Re: [patch net-next 00/10] mlxsw: Offload multi-queue RED support
Date: Wed, 28 Feb 2018 12:15:29 -0500 (EST) [thread overview]
Message-ID: <20180228.121529.1824677671698775640.davem@davemloft.net> (raw)
In-Reply-To: <20180228094507.22354-1-jiri@resnulli.us>
From: Jiri Pirko <jiri@resnulli.us>
Date: Wed, 28 Feb 2018 10:44:57 +0100
> Support a two level hierarchy of offloaded qdiscs in mlxsw, with sch_prio
> being the root qdisc and sch_red as the children.
Ok I applied this series.
However, we really need to have a real discussion about those stats
issues.
I understand the kinds of constraints under which you operate when
offloading, but what, if anything, can we do to close the gap
of inconsistent stats exports in the long term?
Thanks.
prev parent reply other threads:[~2018-02-28 17:15 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-28 9:44 [patch net-next 00/10] mlxsw: Offload multi-queue RED support Jiri Pirko
2018-02-28 9:44 ` [patch net-next 01/10] mlxsw: spectrum: qdiscs: Support qdisc per tclass Jiri Pirko
2018-02-28 9:44 ` [patch net-next 02/10] mlxsw: spectrum: Add priority counters Jiri Pirko
2018-02-28 9:45 ` [patch net-next 03/10] mlxsw: spectrum: qdiscs: Add priority map per qdisc Jiri Pirko
2018-02-28 9:45 ` [patch net-next 04/10] mlxsw: spectrum: qdiscs: Collect stats for sch_red based on priomap Jiri Pirko
2018-02-28 9:45 ` [patch net-next 05/10] mlxsw: spectrum: qdiscs: Update backlog handling of a child qdiscs Jiri Pirko
2018-02-28 9:45 ` [patch net-next 06/10] net: sch: Don't warn on missmatching qlen and backlog for offloaded qdiscs Jiri Pirko
2018-02-28 9:45 ` [patch net-next 07/10] mlxsw: spectrum: Update sch_prio stats to include sch_red related drops Jiri Pirko
2018-02-28 9:45 ` [patch net-next 08/10] mlxsw: spectrum: qdiscs: prio: Delete child qdiscs when removing bands Jiri Pirko
2018-02-28 9:45 ` [patch net-next 09/10] net: sch: prio: Add offload ability for grafting a child Jiri Pirko
2018-03-02 3:38 ` Alexander Aring
2018-03-02 3:48 ` Jakub Kicinski
2018-03-02 8:37 ` Jiri Pirko
2018-03-02 14:21 ` Alexander Aring
2018-03-02 14:45 ` Marcelo Ricardo Leitner
2018-02-28 9:45 ` [patch net-next 10/10] mlxsw: spectrum: qdiscs: prio: Handle graft command Jiri Pirko
2018-02-28 17:15 ` 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=20180228.121529.1824677671698775640.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=idosch@mellanox.com \
--cc=jhs@mojatatu.com \
--cc=jiri@resnulli.us \
--cc=kubakici@wp.pl \
--cc=mlxsw@mellanox.com \
--cc=netdev@vger.kernel.org \
--cc=nogahf@mellanox.com \
--cc=xiyou.wangcong@gmail.com \
--cc=yuvalm@mellanox.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 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).