netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David Täht" <dave.taht@gmail.com>
To: jhs@mojatatu.com
Cc: jamal <hadi@cyberus.ca>, Eric Dumazet <eric.dumazet@gmail.com>,
	Herbert Xu <herbert@gondor.apana.org.au>,
	David Miller <davem@davemloft.net>,
	jesse@nicira.com, netdev@vger.kernel.org, dev@openvswitch.org
Subject: Re: [GIT PULL v2] Open vSwitch
Date: Wed, 23 Nov 2011 14:13:33 +0100	[thread overview]
Message-ID: <4ECCF17D.5020509@gmail.com> (raw)
In-Reply-To: <1322052463.2039.135.camel@mojatatu>

[-- Attachment #1: Type: text/plain, Size: 2769 bytes --]

On 11/23/2011 01:47 PM, jamal wrote:
> On Wed, 2011-11-23 at 09:12 +0100, Eric Dumazet wrote:
>
>> I had no time to look at OVS, but current tc model is not scalable,
>> everything is performed under a queue lock.
>> Maybe its time to redesign a new model, based on modern techniques.
> Making the enqueur/dequeuer lockless would be a big win. What happened
> to your idea of ring buffer?

It's not so much 'modern tecniques', as modern environments.

High on my list would be a way to more easily expose QoS and AQM
features in the hardware all the way up the stack.

I'd like the hardware to be able to express 'I have FQ', or 'I have red',
much like we express many other features in ethtool, only abstractly 
enough so that a qdisc setup can be made generic.

> What other hot areas do you see? It used to be ingress/egress share
> the qdisc lock - but that is now gone.
I find the mapping from hardware queues to any sort of complex software 
queuing scheme hard to conceptualize. Also, as structured, tc cannot be 
easily applied to wireless APs.

>
>> By the way, we seriously lack good documentation on tc, not counting
>> many features. Code might be there, but without documenation, working
>> samples, who can use it ?
I find tc's concepts incredibly difficult to use effectively. They start 
with the presumption that what you are working with is a 1998 point to 
point link and get harder from there. That said I think I've almost 
managed to bend it to my will of late...

(this email written under the influence of Byte Queue Limits + QFQ + 
RED, on ethernet)

>>
>> Take a look at last cls_flow extension, and try to use it on a real
>> setup, you'll find its almost not possible...
>
> There's no tc-central.org unlike the nice effort the netfilter guys have
> put over the years. Documentation is there - sometimes a little too much
> with differing "opinions" (lartc that Herbert pointed to is a good
> starting point); but googling also helps.
> Unfortunately, sometimes the people who understand stuff have no
> motivation to do docs.
After burning the last several months getting good enough at the tc layer
to do stuff in it, I would certainly like to have a place to put 
documentation,
and also easily update what already exists.

If it helps any I could offer a redmine instance on bufferbloat.net for 
this.
redmine has bug tracking and a wiki...

It would be nice also if the iproute2 code contained more working examples,
and man pages.

It's a ton of doc work, but I'd be willing to do some of it.

>
> cheers,
> jamal
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Dave Täht


[-- Attachment #2: dave_taht.vcf --]
[-- Type: text/x-vcard, Size: 214 bytes --]

begin:vcard
fn;quoted-printable:Dave T=C3=A4ht
n;quoted-printable:T=C3=A4ht;Dave
email;internet:dave.taht@gmail.com
tel;home:1-239-829-5608
tel;cell:0638645374
x-mozilla-html:FALSE
version:2.1
end:vcard


  parent reply	other threads:[~2011-11-23 13:13 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-21 21:30 [GIT PULL v2] Open vSwitch Jesse Gross
     [not found] ` <1321911029-20707-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2011-11-21 21:30   ` [PATCH v2 1/5] genetlink: Add genl_notify() Jesse Gross
2011-11-21 21:30   ` [PATCH v2 2/5] genetlink: Add lockdep_genl_is_held() Jesse Gross
2011-11-21 21:30   ` [PATCH v2 3/5] genetlink: Add rcu_dereference_genl and genl_dereference Jesse Gross
2011-11-21 21:30   ` [PATCH v2 4/5] vlan: Move vlan_set_encap_proto() to vlan header file Jesse Gross
2011-11-21 21:30   ` [PATCH v2 5/5] net: Add Open vSwitch kernel components Jesse Gross
     [not found]     ` <1321911029-20707-6-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2011-11-21 21:59       ` Stephen Hemminger
     [not found]         ` <20111121135955.571254b1-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2011-11-21 23:18           ` Jesse Gross
     [not found]             ` <CAEP_g=8uoq7tJjUTAC_Sp3kOYwZJuKjD3J7Ratu67Kq56ZiyYA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-11-21 23:25               ` Stephen Hemminger
     [not found]                 ` <20111121152518.79e82eb8-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2011-11-21 23:49                   ` Michał Mirosław
2011-11-21 22:12       ` Stephen Hemminger
     [not found]         ` <20111121141235.71a5f8fd-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2011-11-21 23:23           ` Jesse Gross
2011-11-22  0:27       ` Stephen Hemminger
2011-11-22 17:03         ` Jesse Gross
2011-11-22 20:50   ` [GIT PULL v2] Open vSwitch David Miller
2011-11-22 23:18     ` Stephen Hemminger
     [not found]       ` <20111122151854.198da33d-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2011-11-23  5:34         ` Chris Wright
2011-11-23  7:54     ` Herbert Xu
     [not found]       ` <20111123075433.GA7928-lOAM2aK0SrRLBo1qDEOMRrpzq4S04n8Q@public.gmane.org>
2011-11-23  8:12         ` Eric Dumazet
2011-11-23  8:21           ` Herbert Xu
2011-11-23 12:47           ` jamal
2011-11-23 12:55             ` Eric Dumazet
2011-11-23 13:44               ` Jamal Hadi Salim
2011-11-23 16:05                 ` John Fastabend
     [not found]                   ` <4ECD19AC.8090505-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2011-11-24 13:19                     ` Jamal Hadi Salim
2011-11-27 19:34                       ` Lennert Buytenhek
     [not found]                         ` <20111127193438.GV795-OLH4Qvv75CYX/NnBR394Jw@public.gmane.org>
2011-11-27 21:31                           ` jamal
2011-11-23 13:13             ` David Täht [this message]
     [not found]               ` <4ECCF17D.5020509-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-11-23 13:36                 ` jamal
2011-11-23 14:15                   ` Eric Dumazet
2011-11-24 13:04                     ` Jamal Hadi Salim
2011-11-27 14:14                       ` WANG Cong
2011-11-23 12:22       ` jamal
2011-11-28 13:04         ` Herbert Xu
     [not found]           ` <20111128130409.GB16828-lOAM2aK0SrRLBo1qDEOMRrpzq4S04n8Q@public.gmane.org>
2011-11-28 13:54             ` Fischer, Anna
2011-11-28 14:51               ` Herbert Xu
     [not found]                 ` <20111128145157.GA17678-lOAM2aK0SrRLBo1qDEOMRrpzq4S04n8Q@public.gmane.org>
2011-11-30  6:21                   ` Jesse Gross
2011-11-30  7:02                     ` Herbert Xu
     [not found]                       ` <20111130070219.GB32630-lOAM2aK0SrRLBo1qDEOMRrpzq4S04n8Q@public.gmane.org>
2011-12-01  7:24                         ` Simon Horman
2011-12-01  7:52                           ` Herbert Xu
     [not found]                             ` <20111201075237.GA12799-lOAM2aK0SrRLBo1qDEOMRrpzq4S04n8Q@public.gmane.org>
2011-12-01  8:06                               ` Simon Horman
2011-12-02 23:00                               ` Jesse Gross
     [not found]               ` <0199E0D51A61344794750DC57738F58E7586A74137-1IhDuF6AwYvulpxXP3Mx0dVKv6DIAtwysh7EHKopUjU@public.gmane.org>
2011-11-28 14:07                 ` Issues with openflow protocol WAS(RE: " Jamal Hadi Salim
2011-11-28 18:44                   ` Justin Pettit
     [not found]                     ` <20124540-D566-41B0-B86F-0BCA19B948AA-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2011-11-28 18:54                       ` Fischer, Anna
2011-11-28 22:55                       ` Jamal Hadi Salim
2011-11-28 16:04                 ` Ben Pfaff
     [not found]                   ` <20111128160400.GB6349-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2011-11-28 18:52                     ` Fischer, Anna
2011-11-28 14:02             ` Jamal Hadi Salim
2011-11-28 15:27               ` Martin Casado
2011-11-28 15:32                 ` [ovs-dev] " Jamal Hadi Salim
2011-11-28 15:50                   ` Martin Casado
2011-11-28 16:01               ` Ben Pfaff
     [not found]                 ` <20111128160117.GA6349-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2011-11-28 22:21                   ` Jamal Hadi Salim
2011-11-28 23:14                     ` [ovs-dev] " Ben Pfaff
2011-11-30  6:18             ` Jesse Gross
2011-11-30  7:06               ` Herbert Xu
     [not found]               ` <CAEP_g=-+F8bpkb8Qe1bPk65PQVNxz+VO7NoUrBCw6=GDUFbOFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-11-30 13:23                 ` 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=4ECCF17D.5020509@gmail.com \
    --to=dave.taht@gmail.com \
    --cc=davem@davemloft.net \
    --cc=dev@openvswitch.org \
    --cc=eric.dumazet@gmail.com \
    --cc=hadi@cyberus.ca \
    --cc=herbert@gondor.apana.org.au \
    --cc=jesse@nicira.com \
    --cc=jhs@mojatatu.com \
    --cc=netdev@vger.kernel.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).