[ovs-dev] Bug#691438: openvswitch packages do not work well together with upstream's kernel module

2012-11-04 Thread Arno Toell
This is a short state update. I'm currently lacking time to explain in detail but I will, once I have some time to show some evidence of what I mean. > No. From the FAQ available at http://openvswitch.org/faq/: ... >- Bridge compatibility, that is, support for the ovs-brcompatd >

Re: [ovs-dev] [PATCH] datapath: add ipv6 'set' action

2012-11-04 Thread Ansis Atteka
On Tue, Oct 30, 2012 at 1:11 AM, Jesse Gross wrote: > On Sun, Oct 28, 2012 at 2:21 PM, Ansis Atteka wrote: >> This patch adds ipv6 set action functionality. It allows to change >> traffic class, flow label, hop-limit, ipv6 source and destination >> address fields. >> >> Signed-off-by: Ansis Attek

[ovs-dev] Bug#691438: openvswitch packages do not work well together with upstream's kernel module

2012-11-04 Thread Ben Pfaff
On Sun, Nov 04, 2012 at 04:13:40PM +0100, Arno Toell wrote: > This is a short state update. I'm currently lacking time to explain in > detail but I will, once I have some time to show some evidence of what I > mean. > > > No. From the FAQ available at http://openvswitch.org/faq/: > ... > > >

[ovs-dev] [PATCH 3/6] Add support for dec_mpls_ttl action

2012-11-04 Thread Simon Horman
This adds support for the OpenFlow 1.1+ dec_mpls_ttl action. And also adds an NX dec_mpls_ttl action. The handling of the TTL modification is entirely handled in userspace. Signed-off-by: Simon Horman --- v2.6 * Rebase v2.5 * First post --- include/openflow/nicira-ext.h |1 + lib/flow.c

[ovs-dev] [PATCH 6/6] Add support for copy_ttl_in action

2012-11-04 Thread Simon Horman
This adds support for the OpenFlow 1.1+ copy_ttl_out action. And also adds an NX copy_ttl_out action. The implementation does not support copying in the case where the outermost header is IP as it is unclear to me that Open vSwtich has a notion of an inner IP header to copy the TLL from. The impl

[ovs-dev] [PATCH 5/6] Add support for copy_ttl_out action

2012-11-04 Thread Simon Horman
This adds support for the OpenFlow 1.1+ copy_ttl_out action. And also adds an NX copy_ttl_out action. The implementation does not support copying in the case where the outermost header is IP as it is unclear to me that Open vSwtich has a notion of an inner IP header to copy the TLL from. The hand

[ovs-dev] [PATCH 0/6 v2.6] Basic MPLS actions and matches

2012-11-04 Thread Simon Horman
Hi, This series implements basic MPLS actions and matches based on work by Ravi K, Leo Alterman and Yamahata-san. The main limitation of this implementation is that it only supports manipulating the outer-most MPLS label. Key difference between the previous post, v2.5: * Dropped "flow: Set ttl

[ovs-dev] [PATCH 1/6] datapath: Add basic MPLS support to kernel

2012-11-04 Thread Simon Horman
Allow datapath to recognize and extract MPLS labels into flow keys and execute actions which push, pop, and set labels on packets. Based heavily on work by Leo Alterman and Ravi K. Cc: Isaku Yamahata Cc: Ravi K Cc: Leo Alterman Signed-off-by: Simon Horman --- v2.6 * As suggested by Yamahata

[ovs-dev] [PATCH 4/6] Add support for set_mpls_ttl action

2012-11-04 Thread Simon Horman
This adds support for the OpenFlow 1.1+ set_mpls_ttl action. And also adds an NX set_mpls_ttl action. The handling of the TTL modification is entirely handled in userspace. Signed-off-by: Simon Horman --- v2.6 * Rebase v2.5 * First post --- include/openflow/nicira-ext.h | 12 lib/

Re: [ovs-dev] [PATCH 4/8] User-Space MPLS actions and matches

2012-11-04 Thread Ben Pfaff
OK. I think we need Jesse's input now. On Sat, Nov 03, 2012 at 05:17:14PM +0800, Simon Horman wrote: > Sorry to be a bit slow, I am on holidays. > > What I was trying to say in my reply to Yamahata-san, > is that I don't think the problem he sees manifests when > using the MPLS patches on top of

Re: [ovs-dev] [PATCH 03/18] ofp-util: Add version bitmap support to hello messages

2012-11-04 Thread Ben Pfaff
On Thu, Nov 01, 2012 at 08:33:04AM +0800, Simon Horman wrote: > On Wed, Oct 31, 2012 at 11:16:06AM -0700, Ben Pfaff wrote: > > On Fri, Oct 26, 2012 at 11:35:51AM +0900, Simon Horman wrote: > > > Allow encoding and decoding of version bitmap in hello messages > > > as specified in Open Flow 1.3.1. >

Re: [ovs-dev] [PATCH 05/18] ofproto: Make set_pvconns() aware of OpenFlow versions

2012-11-04 Thread Ben Pfaff
On Fri, Oct 26, 2012 at 11:35:53AM +0900, Simon Horman wrote: > Paramatise the allowed OpenFlow verion for snoops. > This is in preparation for making the allow OpenFlow verions to > be configured. > > Signed-off-by: Simon Horman Snoops are pretty odd. With them, the goal is to allow some progr

Re: [ovs-dev] [PATCH 18/18] ofp-util: Optimize ofputil_version_bitmap_scanr

2012-11-04 Thread Ben Pfaff
On Fri, Oct 26, 2012 at 11:36:06AM +0900, Simon Horman wrote: > Make use of __builtin_clz if available which should optimize > ofputil_version_bitmap_scanr() by replacing a loop with > a single CLZ instruction when available. > > I'm unsure if this approach is worth it or not. > But a similar appr