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

[jamal <hadi-fAAogVwAN2Kw5LPnMra/2Q@public.gmane.org>]

Reading your website - it seems you indicate the code to be portable
across other platforms. That is a noble reason - but i dont think
it is good enough a reason in my opinion to have your own
replicated infrastructure on top of Linux. 

cheers,
jamal

On Mon, 2011-11-21 at 07:20 -0500, jamal wrote:

> 
> So the last time we had a discussion on this on the list, we seemed
> to agree that you could use the tc classifier-action infrastructure.
> For simplicity, we agreed you will need to do a speacilized classifier.
> You may need to add a few more actions. What happened since?
> 
> You are replicating a lot of code and semantic that exist (not just on
> classifier actions). You could improve the exisiting infrastructure
> instead. We are eventually going to have two competing interfaces as 
> a result. You may only need 1 or 2 different classification schemes
> today and try to justify you need it for simplicity - but in a few
> months you'll need one more then another action and another and 
> you'll keep adding to your infrastructure.
> 
> cheers,
> jamal
>