Re: [ovs-dev] openvswitch.org is down

2014-06-17 Thread Flavio Leitner
On Tue, Jun 17, 2014 at 03:44:52PM -0700, Justin Pettit wrote: > Thanks, Flavio. I'm working with IT to get them to kick the box that's > running the site. We're working on a plan to prevent this from happening > in the future. No worries. I saw couple people asking about it on the channel and

[ovs-dev] [PATCH 2/2] revalidator: Don't use xcache side-effects on delete.

2014-06-17 Thread Joe Stringer
The xcache holds the correct set of modules to attribute stats for a flow that is being deleted, but it may hold an outdated view of the side-effects to perform for the flow. We have no way of knowing whether the flow is valid in this context, so use xcache to attribute stats, but do not use it for

[ovs-dev] [PATCH 1/2] revalidator: Simplify push_dump_ops__().

2014-06-17 Thread Joe Stringer
Commit acaa8dac49 (revalidator: Eliminate duplicate flow handling.) ensured that a ukey will always exist for a given flow, even if it is about to be deleted. This means that push_dump_ops__() no longer needs to handle the case where there is no ukey. This commit removes the redundant code. Signed

Re: [ovs-dev] [PATCH v2] dpif-netdev: Polling threads directly call ofproto upcall functions.

2014-06-17 Thread Ryan Wilson 76511
I think the key issue here is that Mike believes the limiting factor for DPDK is CPU while Ethan believes it to be memory (transferring memory between cores, to be precise). I would agree Mike is right where there is a heavy load (i.e. 100% utilization of 10 GbE links) while Ethan is right in the a

Re: [ovs-dev] openvswitch.org is down

2014-06-17 Thread Justin Pettit
Thanks, Flavio. I'm working with IT to get them to kick the box that's running the site. We're working on a plan to prevent this from happening in the future. --Justin On Tue, Jun 17, 2014 at 2:55 PM, Flavio Leitner wrote: > > FYI, openvswitch.org is down for a few hours. > fbl > ___

[ovs-dev] openvswitch.org is down

2014-06-17 Thread Flavio Leitner
FYI, openvswitch.org is down for a few hours. fbl ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] dpif-netdev: Polling threads directly call ofproto upcall functions.

2014-06-17 Thread Jarno Rajahalme
On Jun 17, 2014, at 11:33 AM, Ethan Jackson wrote: >> A good reason to offload an ofproto upcall function in polling mode is to >> allow a different CPU to do a time consuming inexact rule matches while the >> polling thread maintains fast packet switching. At low data and packet rates >> or

Re: [ovs-dev] [PATCH v3 3/3] datapath: add layer 3 flow/port support

2014-06-17 Thread Lori Jakab
Hi Jesse, On 5/23/14, 2:07 AM, Jesse Gross wrote: On Tue, May 20, 2014 at 9:27 PM, Lori Jakab wrote: On 5/21/14, 4:10 AM, Jesse Gross wrote: On Tue, May 13, 2014 at 7:02 AM, Lorand Jakab wrote: Implementation of the pop_eth and push_eth actions in the kernel, and layer 3 flow support. Sign

Re: [ovs-dev] [PATCH v2] dpif-netdev: Polling threads directly call ofproto upcall functions.

2014-06-17 Thread Ethan Jackson
> A good reason to offload an ofproto upcall function in polling mode is to > allow a different CPU to do a time consuming inexact rule matches while the > polling thread maintains fast packet switching. At low data and packet rates > or low rate Ethernet interface (1 GbE and lower) this does no

Re: [ovs-dev] [PATCH v3 2/7] flow: Add support for IGMP protocol

2014-06-17 Thread Flavio Leitner
On Wed, Jun 11, 2014 at 11:31:37AM -0700, Ben Pfaff wrote: > On Sat, May 31, 2014 at 01:53:16AM -0300, Flavio Leitner wrote: > > Add IGMP support to struct flow, flow compose and flow extract. > > > > Signed-off-by: Flavio Leitner > > If the ofproto code is going to match on particular IGMP type

Re: [ovs-dev] [PATCH] ovs-vsctl: Fix OF protocol name capitalization in manual page

2014-06-17 Thread Justin Pettit
On Tue, Jun 17, 2014 at 9:42 AM, Thomas Graf wrote: > On 06/17/14 at 09:19am, Justin Pettit wrote: > > I think the way it was worked, since we use strcasecmp(), but we may as > > well be consistent with how it's documented elsewhere. It's been > applied. > > Thanks! > > I think it's the actual s

Re: [ovs-dev] [PATCH] ovs-vsctl: Fix OF protocol name capitalization in manual page

2014-06-17 Thread Thomas Graf
On 06/17/14 at 09:19am, Justin Pettit wrote: > I think the way it was worked, since we use strcasecmp(), but we may as > well be consistent with how it's documented elsewhere. It's been applied. Thanks! I think it's the actual schema validation that enforces correct capitalization: $ sudo ovs-v

Re: [ovs-dev] [PATCH] ovs-vsctl: Fix OF protocol name capitalization in manual page

2014-06-17 Thread Justin Pettit
I think the way it was worked, since we use strcasecmp(), but we may as well be consistent with how it's documented elsewhere. It's been applied. Thanks, --Justin On Tue, Jun 17, 2014 at 7:58 AM, Thomas Graf wrote: > Signed-off-by: Thomas Graf > --- > utilities/ovs-vsctl.8.in | 2 +- >

Re: [ovs-dev] [PATCH] AUTHORS: Update entry for Thomas Graf

2014-06-17 Thread Justin Pettit
Applied. Thanks. --Justin On Tue, Jun 17, 2014 at 7:57 AM, Thomas Graf wrote: > Signed-off-by: Thomas Graf > --- > AUTHORS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/AUTHORS b/AUTHORS > index c03c705..33de7b1 100644 > --- a/AUTHORS > +++ b/AUTHORS > @@ -121,7

Re: [ovs-dev] [PATCH v2] dpif-netdev: Polling threads directly call ofproto upcall functions.

2014-06-17 Thread Polehn, Mike A
A good reason to offload an ofproto upcall function in polling mode is to allow a different CPU to do a time consuming inexact rule matches while the polling thread maintains fast packet switching. At low data and packet rates or low rate Ethernet interface (1 GbE and lower) this does not matter

[ovs-dev] [PATCH] AUTHORS: Update entry for Thomas Graf

2014-06-17 Thread Thomas Graf
Signed-off-by: Thomas Graf --- AUTHORS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AUTHORS b/AUTHORS index c03c705..33de7b1 100644 --- a/AUTHORS +++ b/AUTHORS @@ -121,7 +121,7 @@ SUGYO Kazushi sugyo@gmail.com Tadaaki Nagao na...@stratosphere.co.jp

[ovs-dev] [PATCH] ovs-vsctl: Fix OF protocol name capitalization in manual page

2014-06-17 Thread Thomas Graf
Signed-off-by: Thomas Graf --- utilities/ovs-vsctl.8.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-vsctl.8.in b/utilities/ovs-vsctl.8.in index 1701b48..50d8879 100644 --- a/utilities/ovs-vsctl.8.in +++ b/utilities/ovs-vsctl.8.in @@ -973,7 +973,7 @@ Deconfigu

Re: [ovs-dev] [PATCH v2.60] datapath: Add basic MPLS support to kernel

2014-06-17 Thread Simon Horman
Hi Jesse, I think this is getting pretty close. Is there anything I can do to help edge it over the line? On Fri, Jun 06, 2014 at 07:28:51PM +0900, Simon Horman wrote: > Allow datapath to recognize and extract MPLS labels into flow keys > and execute actions which push, pop, and set labels on pac