netdev - Re: [ovs-dev] [PATCH net-next 4/4] net: Add Open vSwitch kernel components.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1322141102.1949.198.camel@mojatatu>
Date:	Thu, 24 Nov 2011 08:25:02 -0500
From:	jamal <hadi@...erus.ca>
To:	Justin Pettit <jpettit@...ira.com>
Cc:	Jesse Gross <jesse@...ira.com>, dev@...nvswitch.org,
	netdev@...r.kernel.org, "David S. Miller" <davem@...emloft.net>
Subject: Re: [ovs-dev] [PATCH net-next 4/4] net: Add Open vSwitch kernel
 components.

Hi Justin,

I apologize, I did not intend to demean your work. There
are lots of clever ideas in there and you may have wanted
to put out something in a short period out there.

The most basic IMO is to use netlink if you are doing
it from a programmatic interface. You seem to be doing that
already for other items (eg HTB) in the setup. There are
a few libraries out there you could use but i realize
that they may not match your license requirements.
Maybe you could isolate your netlink code and make it 
standalone based on the license you use and people who 
need that could use it.

The other thing, is you match every flow on the specific
virtual port - this may be design intent but it appears
very inflexible.

cheers,
jamal

On Wed, 2011-11-23 at 18:34 -0800, Justin Pettit wrote:
> On Nov 22, 2011, at 5:45 PM, Jamal Hadi Salim wrote:
> 
> > BTW, you  _are using some of the actions_ already (the policer for
> > example to do rate control; no disrespect intended but in a terrible
> > way). 
> 
> Hi, Jamal.  I did the initial implementation of the rate control in Open vSwitch.  
> Can you help us out with a few more specifics on the problems you see and how they 
> could be improved?
> 
> Thanks,
> 
> --Justin
> 
> 

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