From: jamal <hadi@cyberus.ca>
To: John Fastabend <john.r.fastabend@intel.com>
Cc: "shemminger@vyatta.com" <shemminger@vyatta.com>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"tgraf@infradead.org" <tgraf@infradead.org>,
"eric.dumazet@gmail.com" <eric.dumazet@gmail.com>,
"davem@davemloft.net" <davem@davemloft.net>
Subject: Re: [RFC PATCH v1] iproute2: add IFLA_TC support to 'ip link'
Date: Fri, 03 Dec 2010 06:06:52 -0500 [thread overview]
Message-ID: <1291374412.10126.17.camel@mojatatu> (raw)
In-Reply-To: <4CF7F8B4.4060807@intel.com>
On Thu, 2010-12-02 at 11:51 -0800, John Fastabend wrote:
> On 12/2/2010 2:40 AM, jamal wrote:
> I viewed the HW QOS as L2 link attributes more than a queuing discipline per se.
> Plus 'ip link' is already used to set things outside of ip.
> For example 'txqueuelen' and 'vf x'.
the vf one maybe borderline-ok txquelen is probably inherited from
ifconfig (and not sure a single queue a scheduler qualifies)
> However thinking about this a bit more qdisc support seems cleaner.
> For one we can configure QOS policies per class with Qdisc_class_ops.
> And then also aggregate statistics with dump_stats. I would avoid the
> "hardware-kinda-8021q-sched" name though to account for schedulers that
> may not be 802.1Q compliant maybe 'mclass-sched' for multi-class scheduler.
Typically the scheduler would be a very familiar one implemented
per-spec by many vendors and will have a name acceptable by all.
So pick an appropriate noun so the user expectation matches it.
> I'll look into this. Thanks for the suggestion!
>
> On egress the skb priority is mapped to a class which is associated with a
> range of queues (qoffset:qoffset + qcount).
> In the 802.1Q case this queue range is mapped to the 802.1Qp
> traffic class in hardware. So the hardware traffic class is mapped 1-1
> with the software class. Additionally in software the VLAN egress mapping
> is used to map the skb priority to the 802.1Q priority. Here I expect user
> policies to configure this to get a consistent mapping. On ingress the
> skb priority is set using the 802.1Q ingress mapping. This case is
> something a userspace policy could configure if egress/ingress mappings
> should be symmetric.
>
Sounds sensible.
> In the simpler case of hardware rate limiting (not 802.1Q) this is not
> really a concern at all. With this mechanism we can identify traffic
> and push it to the correct queues that are grouped into a rate limited class.
Ok, so you can do rate control as well?
> If there are egress/ingress mappings then those will apply skb priority tags
> on egress and the correct skb priority on ingress.
Curious how you would do this in a rate controlled environment. EX: on
egress, do you use whatever skb prio you get to map to a specific rate
queue in h/ware? Note: skb prio has a strict priority scheduling
semantics so a 1-1 mapping doesnt sound reasonable..
> Currently everything works reasonably well with this scheme and the mq qdisc.
> The mq qdisc uses pfifo and the driver then pauses the queues as needed.
> Using the enhanced transmission selection algorithm (ETS - 802.1Qaz pre-standard)
> in hardware we see variations from expected bandwidth around +-5% with TCP/UDP.
> Instrumenting HW rate limiters gives similar variations. I tested this is with
> ixgbe and the 82599 device.
>
> Bit long winded but hopefully that answers your question.
I am curious about the rate based scheme - and i hope you are looking at
a different qdisc for that?
cheers,
jamal
next prev parent reply other threads:[~2010-12-03 11:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-01 18:27 [RFC PATCH v1] iproute2: add IFLA_TC support to 'ip link' John Fastabend
2010-12-01 18:38 ` Stephen Hemminger
2010-12-01 18:48 ` David Miller
2010-12-01 19:27 ` Stephen Hemminger
2010-12-01 19:30 ` David Miller
2010-12-01 20:57 ` John Fastabend
2010-12-02 10:40 ` jamal
2010-12-02 19:51 ` John Fastabend
2010-12-03 11:06 ` jamal [this message]
2010-12-09 19:58 ` John Fastabend
2010-12-15 13:19 ` jamal
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=1291374412.10126.17.camel@mojatatu \
--to=hadi@cyberus.ca \
--cc=davem@davemloft.net \
--cc=eric.dumazet@gmail.com \
--cc=john.r.fastabend@intel.com \
--cc=netdev@vger.kernel.org \
--cc=shemminger@vyatta.com \
--cc=tgraf@infradead.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).