Re: [ovs-dev] [PATCH v8] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery (kmestery)
On Dec 5, 2012, at 8:13 PM, Jesse Gross wrote: > On Wed, Dec 5, 2012 at 1:06 PM, Kyle Mestery wrote: >> Add support for VXLAN tunnels to Open vSwitch. Add support >> for setting the destination UDP port on a per-port basis. >> This is done by adding a "dst_port" parameter to the port >> configura

Re: [ovs-dev] [PATCH v8] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Jesse Gross
On Wed, Dec 5, 2012 at 1:06 PM, Kyle Mestery wrote: > Add support for VXLAN tunnels to Open vSwitch. Add support > for setting the destination UDP port on a per-port basis. > This is done by adding a "dst_port" parameter to the port > configuration. This is only applicable currently to VXLAN > tun

Re: [ovs-dev] [PATCH] rconn: Maintain original allowed_versions instead of substituting for zero.

2012-12-05 Thread Simon Horman
On Wed, Dec 05, 2012 at 05:03:40PM -0800, Ben Pfaff wrote: > On Thu, Dec 06, 2012 at 09:49:56AM +0900, Simon Horman wrote: > > On Tue, Dec 04, 2012 at 05:51:04PM -0800, Ben Pfaff wrote: > > > rconn_create() was substituting OFPUTIL_DEFAULT_VERSIONS if an > > > allowed_versions of 0 was passed in.

Re: [ovs-dev] Possible user-space datapath regression

2012-12-05 Thread Justin Pettit
Sorry for the delay. Here's a patch I just sent out that I think addresses the issue: http://openvswitch.org/pipermail/dev/2012-December/023652.html Let me know if it works or if you still see problems. Thanks! --Justin On Nov 28, 2012, at 4:06 PM, Simon Horman wrote: > Thanks, let me

[ovs-dev] [PATCH] netdev-linux: Use underlying tap device on netdev_linux_listen().

2012-12-05 Thread Justin Pettit
Commit acf608 (ofproto-dpif: Use a single underlying datapath across multiple bridges.) broke connectivity to userspace datapath devices. The code assumed the first caller to open a tap device with netdev_linux_open() wanted to write to it. This commit moves that logic to when netdev_linux_listen(

Re: [ovs-dev] [PATCH] rconn: Maintain original allowed_versions instead of substituting for zero.

2012-12-05 Thread Ben Pfaff
On Thu, Dec 06, 2012 at 09:49:56AM +0900, Simon Horman wrote: > On Tue, Dec 04, 2012 at 05:51:04PM -0800, Ben Pfaff wrote: > > rconn_create() was substituting OFPUTIL_DEFAULT_VERSIONS if an > > allowed_versions of 0 was passed in. At the same time, > > connmgr_set_controllers() compared the adjust

Re: [ovs-dev] [PATCH] rconn: Maintain original allowed_versions instead of substituting for zero.

2012-12-05 Thread Simon Horman
On Tue, Dec 04, 2012 at 05:51:04PM -0800, Ben Pfaff wrote: > rconn_create() was substituting OFPUTIL_DEFAULT_VERSIONS if an > allowed_versions of 0 was passed in. At the same time, > connmgr_set_controllers() compared the adjusted value of allowed_versions > against the original value, saw that th

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Ben Pfaff
On Wed, Dec 05, 2012 at 07:52:39PM +, Kyle Mestery (kmestery) wrote: > On Dec 5, 2012, at 1:47 PM, Ben Pfaff wrote: > > On Wed, Dec 05, 2012 at 10:23:43AM -0800, Jesse Gross wrote: > >> This will run on both userspace and kernel for the files that need to > >> be recompiled. The userspace ver

[ovs-dev] [PATCH v8] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery
Add support for VXLAN tunnels to Open vSwitch. Add support for setting the destination UDP port on a per-port basis. This is done by adding a "dst_port" parameter to the port configuration. This is only applicable currently to VXLAN tunnels. Please note this currently does not implement any sort o

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery (kmestery)
On Dec 5, 2012, at 1:47 PM, Ben Pfaff wrote: > On Wed, Dec 05, 2012 at 10:23:43AM -0800, Jesse Gross wrote: >> This will run on both userspace and kernel for the files that need to >> be recompiled. The userspace version is a little bit more >> temperamental because sparse requires header annotat

Re: [ovs-dev] OVS development - beginner

2012-12-05 Thread Ben Pfaff
I'd start by reading PORTING. On Wed, Dec 05, 2012 at 10:58:45AM -0800, Ra Alva wrote: > Sorry I guess I was not elaborate. Actually I need to implement a > garbage collector for an Openflow network. Would be great if you could > please let me know what my first steps should be to get familiar wit

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Ben Pfaff
On Wed, Dec 05, 2012 at 10:23:43AM -0800, Jesse Gross wrote: > This will run on both userspace and kernel for the files that need to > be recompiled. The userspace version is a little bit more > temperamental because sparse requires header annotations to work > properly and userspace headers don't

Re: [ovs-dev] OVS development - beginner

2012-12-05 Thread Ra Alva
Sorry I guess I was not elaborate. Actually I need to implement a garbage collector for an Openflow network. Would be great if you could please let me know what my first steps should be to get familiar with OVS and the code? And also if there is anything else important for me to go through. Reg

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Jesse Gross
On Wed, Dec 5, 2012 at 10:14 AM, Kyle Mestery (kmestery) wrote: > On Dec 5, 2012, at 12:10 PM, Jesse Gross wrote: >> On Wed, Dec 5, 2012 at 8:53 AM, Kyle Mestery wrote: >>> Add support for VXLAN tunnels to Open vSwitch. Add support >>> for setting the destination UDP port on a per-port basis. >>

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery (kmestery)
On Dec 5, 2012, at 12:10 PM, Jesse Gross wrote: > On Wed, Dec 5, 2012 at 8:53 AM, Kyle Mestery wrote: >> Add support for VXLAN tunnels to Open vSwitch. Add support >> for setting the destination UDP port on a per-port basis. >> This is done by adding a "dst_port" parameter to the port >> configur

Re: [ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Jesse Gross
On Wed, Dec 5, 2012 at 8:53 AM, Kyle Mestery wrote: > Add support for VXLAN tunnels to Open vSwitch. Add support > for setting the destination UDP port on a per-port basis. > This is done by adding a "dst_port" parameter to the port > configuration. This is only applicable currently to VXLAN > tun

Re: [ovs-dev] [PATCH 0/4] RFC: database backed logging console

2012-12-05 Thread Thomas Graf
On 12/04/2012 08:25 PM, Ben Pfaff wrote: It tells a human user that the operation failed. It could also notify a script that the operation failed, e.g. if we decided to return a nonzero exit status. (ovs-vsctl is intended for use by both humans and programs.) I'm perfectly willing to add anoth

[ovs-dev] [PATCH v7] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery
Add support for VXLAN tunnels to Open vSwitch. Add support for setting the destination UDP port on a per-port basis. This is done by adding a "dst_port" parameter to the port configuration. This is only applicable currently to VXLAN tunnels. Please note this currently does not implement any sort o

Re: [ovs-dev] [PATCH v6] datapath: Add support for VXLAN tunnels to Open vSwitch

2012-12-05 Thread Kyle Mestery (kmestery)
Thanks, I agree with all your comments below, I've addressed them all, re-tested, and will be sending a new patch now (with the correct ordering of commit messages vs. notes at the top). Thanks for the continued reviews of this series Jesse! Kyle On Dec 4, 2012, at 4:52 PM, Jesse Gross wrote: >

Re: [ovs-dev] [PATCH] FAQ: Add table of supported Linux versions.

2012-12-05 Thread Guillermo Ontañón
Hi devs, This is somewhat relevant to the conversation that we had not long ago about OVS kernel mod version and ubuntu. Below is the table supported OVS - Kernel version pairs. Regards, On Tue, Dec 4, 2012 at 9:38 PM, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff > --- > FAQ | 26 +

Re: [ovs-dev] [PATCH] docs: Add INSTALL.OpenNebula

2012-12-05 Thread Ben Pfaff
On Wed, Dec 05, 2012 at 12:51:08PM +0100, Jaime Melis wrote: > From: Jaime Melis > Date: Wed, 5 Dec 2012 12:33:40 +0100 > Subject: [PATCH] INSTALL.OpenNebula: New instructions for using with > OpenNebula. Thanks. Now we need a Signed-off-by-line. Here's the information from SubmittingPatches.

Re: [ovs-dev] [PATCH] docs: Add INSTALL.OpenNebula

2012-12-05 Thread Jaime Melis
Attaching as a patch after reading "Documentation/email-clients.txt" and realising that gmail (Web GUI) doesn't work for sending inline patches. On Wed, Dec 5, 2012 at 12:51 PM, Jaime Melis wrote: > From 32dd91bcc835b41e28b48fcc8f3278dc69f8851d Mon Sep 17 00:00:00 2001 > From: Jaime Melis >

[ovs-dev] [PATCH] docs: Add INSTALL.OpenNebula

2012-12-05 Thread Jaime Melis
>From 32dd91bcc835b41e28b48fcc8f3278dc69f8851d Mon Sep 17 00:00:00 2001 From: Jaime Melis Date: Wed, 5 Dec 2012 12:33:40 +0100 Subject: [PATCH] INSTALL.OpenNebula: New instructions for using with OpenNebula. --- INSTALL.OpenNebula | 177 + Mak

[ovs-dev] Good Morning

2012-12-05 Thread R . B . S
>From Mr. Stephen Hester. 36 St Andrew Square Edinburgh, Scotland EH2 2YB United Kingdom I am Mr. Stephen Hester, the Group Chief Executive of Royal Bank of Scotland. I have a proposal for you in the tune of SIXTY EIGHT MILLION DOLLARS during our Bank's Annual Year Account Auditing. After succe