Re: [ovs-dev] [PATCH monitor_cond V9 00/12] Conditional monitoring implementation

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:48AM +0300, Liran Schour wrote: > This patch series implements conditional monitoring by introducing an OVSDB > RFC extension with 2 new JSON-RPC methods: "monitor_cond" and > "monitor_cond_change". Specification of this extension is defined in the > ovsdb-server (1) m

Re: [ovs-dev] [PATCH monitor_cond V9 04/12] ovsdb: generate update notifications for monitor_cond session

2016-07-18 Thread Ben Pfaff
On Tue, Jul 19, 2016 at 08:55:10AM +0300, Liran Schour wrote: > Hold session's conditions in ovsdb_monitor_session_condition. Pass it > to ovsdb_monitor for generating "update2" notifications. > Add functions that can generate "update2" notification for a > "monitor_cond" session. > JSON cache is e

Re: [ovs-dev] [ovs-dev, v23, 2/2] ovn-controller: Add incremental processing to lflow_run and physical_run

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 04:21:17PM -0500, Ryan Moats wrote: > This code changes to allow incremental processing of the > logical flow and physical binding tables whenver possible. > > Note: flows created by physical_run for multicast_groups are > *NOT* handled incrementally due to to be solved iss

[ovs-dev] [PATCH monitor_cond V9 04/12] ovsdb: generate update notifications for monitor_cond session

2016-07-18 Thread Liran Schour
Hold session's conditions in ovsdb_monitor_session_condition. Pass it to ovsdb_monitor for generating "update2" notifications. Add functions that can generate "update2" notification for a "monitor_cond" session. JSON cache is enabled only for session's with true condition only. "monitor_cond" and "

Re: [ovs-dev] [ovs-dev, v23, 1/2] ovn-controller: Persist ovn flow tables

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 04:21:16PM -0500, Ryan Moats wrote: > Ensure that ovn flow tables are persisted so that changes to > them chan be applied incrementally - this is a prereq for > making lflow_run and physical_run incremental. > > As part of this change, add a one-to-many hindex for finding >

Re: [ovs-dev] [PATCH] ovn: Add 128-bit support for ct_label

2016-07-18 Thread Numan Siddique
On Mon, Jul 18, 2016 at 11:40 PM, Ben Pfaff wrote: > On Fri, Jul 15, 2016 at 08:33:59PM +0530, Numan Siddique wrote: > > To support 128-bits in ct_label, the value of the ct_label is expected > > as a hex string in the 'ct_commit' action. > > > > Added a new accessor in the 'mf_subvalue' struct t

[ovs-dev] [PATCH v1] Enable manager_option configuation for NB and SB

2016-07-18 Thread Amitabha Biswas
NB and SB dbs are handled by separate ovsdb-server processes. The ovsdb-server processes manage dbs based on the schemas for OVN NorthBound and SouthBound. These schemas do not have any Manager Options similar to the Open_vSwitch schema. As a result, for e.g., the frequency of inactivity probes sen

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread pravin shelar
On Monday, July 18, 2016, Jesse Gross wrote: > On Tue, Jul 19, 2016 at 12:14 AM, pravin shelar > wrote: > > On Mon, Jul 18, 2016 at 2:59 PM, Joe Stringer > wrote: > >> On 18 July 2016 at 14:42, pravin shelar > > wrote: > >>> On Mon, Jul 18, 2016 at 2:32 PM, Joe Stringer > wrote: > On 17 J

Re: [ovs-dev] [PATCH v2] ovn-sbctl: eliminate a spurious test case error cause

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/07/2016 04:30:01 PM: > From: Lance Richardson > To: dev@openvswitch.org > Date: 07/07/2016 04:30 PM > Subject: [ovs-dev] [PATCH v2] ovn-sbctl: eliminate a spurious test > case error cause > Sent by: "dev" > > The "ovn-sbctl" test fails occasionally due to log messages > simila

Re: [ovs-dev] [PATCH] netdev-dummy: fix crash with more than one passive connection

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 11:29:28 AM: > > > From: "Lance Richardson" > > To: dev@openvswitch.org > > Sent: Wednesday, July 6, 2016 7:39:52 PM > > Subject: [ovs-dev] [PATCH] netdev-dummy: fix crash with more than > one passive connection > > > > Investigation found that Some of the occasional

Re: [ovs-dev] [PATCH v2] ovn: Make it possible for CMS to detect when the OVN system is up-to-date.

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 01:30:10 PM: > From: Ben Pfaff > To: dev@openvswitch.org > Cc: Ben Pfaff > Date: 07/18/2016 01:30 PM > Subject: [ovs-dev] [PATCH v2] ovn: Make it possible for CMS to > detect when the OVN system is up-to-date. > Sent by: "dev" > > Until now, there has been no reliable

[ovs-dev] tjxwctglhe

2016-07-18 Thread Mail Delivery Subsystem
The original message was received at Tue, 19 Jul 2016 09:20:56 +0800 from [4.7.114.72] - The following addresses had permanent fatal errors - dev@openvswitch.org ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.4

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 5:50 PM, Jesse Gross wrote: > On Tue, Jul 19, 2016 at 12:13 AM, Pravin B Shelar wrote: >> Most of changes are related to ip-fragment API and genetlink >> API changes. >> >> Signed-off-by: Pravin B Shelar > > Acked-by: Jesse Gross Thanks for review. I pushed patch to ma

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.4

2016-07-18 Thread Jesse Gross
On Tue, Jul 19, 2016 at 12:13 AM, Pravin B Shelar wrote: > Most of changes are related to ip-fragment API and genetlink > API changes. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross ___ dev mailing list dev@openvswitch.org http://openvswitch

Re: [ovs-dev] [PATCHv3] netdev-provider: Apply batch object to netdev provider.

2016-07-18 Thread William Tu
Thanks for reviewing the patch. I've updated patchv4 at https://patchwork.ozlabs.org/patch/649854/ At netdev_dpdk_send__(), we do need to introduce extra variables, other than that, I've removed the rest of extra variables. Regards, William On Thu, Jul 14, 2016 at 5:34 PM, Daniele Di Proietto w

[ovs-dev] [PATCHv4] netdev-provider: Apply batch object to netdev provider.

2016-07-18 Thread William Tu
Commit 1895cc8dbb64 ("dpif-netdev: create batch object") introduces batch process functions and 'struct dp_packet_batch' to associate with batch-level metadata. This patch applies the packet batch object to the netdev provider interface (dummy, Linux, BSD, and DPDK) so that batch APIs can be used

Re: [ovs-dev] [PATCH] ovs : Implementation to add meter in ovs flows

2016-07-18 Thread Joe Stringer
On 15 July 2016 at 00:32, wrote: > From: Deepanshu Saxena > > This commit implements ovs meters in kernel space using the traffic > control (tc) module of Linux kernel. TC is used to configure Traffic > Control features like Policing, Shaping etc. Ovs meters are analogous to > traffic control

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-07-18 Thread Jan Scheurich
Hi Johnson, > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Yang, Yi > Sent: Monday, 18 July, 2016 12:37 > > On Mon, Jul 18, 2016 at 03:15:13PM +0900, Simon Horman wrote: > > Hi Johnson, > > > > On Wed, Jul 13, 2016 at 01:28:48AM +0800, Johnson Li wrote: > > > Signed-off-by: Johnson

[ovs-dev] Scanned image

2016-07-18 Thread reception244
Image data has been attached to this email. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread Jesse Gross
On Tue, Jul 19, 2016 at 12:14 AM, pravin shelar wrote: > On Mon, Jul 18, 2016 at 2:59 PM, Joe Stringer wrote: >> On 18 July 2016 at 14:42, pravin shelar wrote: >>> On Mon, Jul 18, 2016 at 2:32 PM, Joe Stringer wrote: On 17 July 2016 at 19:24, Pravin B Shelar wrote: > This series conta

Re: [ovs-dev] [PATCH net-next v11 5/6] openvswitch: add layer 3 flow/port support

2016-07-18 Thread pravin shelar
On Sun, Jul 17, 2016 at 9:50 PM, Simon Horman wrote: > [CC Jiri Benc for portion regarding GRE] > > Hi Pravin, > > On Fri, Jul 15, 2016 at 02:07:37PM -0700, pravin shelar wrote: >> On Wed, Jul 13, 2016 at 12:31 AM, Simon Horman >> wrote: >> > Hi Pravin, >> > >> > On Thu, Jul 07, 2016 at 01:54:15P

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 2:59 PM, Joe Stringer wrote: > On 18 July 2016 at 14:42, pravin shelar wrote: >> On Mon, Jul 18, 2016 at 2:32 PM, Joe Stringer wrote: >>> On 17 July 2016 at 19:24, Pravin B Shelar wrote: This series contains patches to advance kernel datapath support for kernel

[ovs-dev] [PATCH] datapath: Add support for kernel 4.4

2016-07-18 Thread Pravin B Shelar
Most of changes are related to ip-fragment API and genetlink API changes. Signed-off-by: Pravin B Shelar --- Fixed according to comments from Jesse. --- acinclude.m4 | 17 +++-- datapath/actions.c | 7 ++-- datapath/datap

Re: [ovs-dev] [PATCH 5/8] datapath: Add support for kernel 4.4

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 2:48 PM, Jesse Gross wrote: > On Mon, Jul 18, 2016 at 11:43 PM, pravin shelar wrote: >> On Mon, Jul 18, 2016 at 11:28 AM, Jesse Gross wrote: >>> On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: >>> /home/jgross/openvswitch/datapath/linux/vxlan.c:1905:45: warning:

Re: [ovs-dev] [PATCH] ovs : Implementation to add meter in ovs flows

2016-07-18 Thread Dustin Lundquist
On 7/15/16 12:32 AM, deepanshu.saxen...@gmail.com wrote: > From: Deepanshu Saxena > > This commit implements ovs meters in kernel space using the traffic > control (tc) module of Linux kernel. TC is used to configure Traffic > Control features like Policing, Shaping etc. Ovs meters are analogo

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread Joe Stringer
On 18 July 2016 at 14:42, pravin shelar wrote: > On Mon, Jul 18, 2016 at 2:32 PM, Joe Stringer wrote: >> On 17 July 2016 at 19:24, Pravin B Shelar wrote: >>> This series contains patches to advance kernel datapath >>> support for kernel upto 4.6. >> >> When I try to load the kernel module on Ubu

Re: [ovs-dev] [PATCH] ovs-vsctl: Add another Flow_Table example.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 02:24:35PM -0500, Ryan Moats wrote: > "dev" wrote on 07/18/2016 12:55:05 PM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Ben Pfaff > > Date: 07/18/2016 12:55 PM > > Subject: [ovs-dev] [PATCH] ovs-vsctl: Add another Flow_Table example. > > Sent by: "dev" >

Re: [ovs-dev] [PATCH monitor_cond V9 00/12] Conditional monitoring implementation

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:48AM +0300, Liran Schour wrote: > This patch series implements conditional monitoring by introducing an OVSDB > RFC extension with 2 new JSON-RPC methods: "monitor_cond" and > "monitor_cond_change". Specification of this extension is defined in the > ovsdb-server (1) m

Re: [ovs-dev] [PATCH monitor_cond V9 07/12] ovsdb-client: support monitor-cond method

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:55AM +0300, Liran Schour wrote: > Add monitor_cond method to ovsdb-client. Enable testing of monitor_cond_change > via unixctl command.Add unit tests for monitor_cond and monitor_cond_change. > See ovsdb-client(1) man page for details. > Replace monitor2 with monitor_c

[ovs-dev] Status

2016-07-18 Thread Bounced mail
Your message was not delivered due to the following reason: Your message could not be delivered because the destination server was unreachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely the

Re: [ovs-dev] [PATCH 5/8] datapath: Add support for kernel 4.4

2016-07-18 Thread Jesse Gross
On Mon, Jul 18, 2016 at 11:43 PM, pravin shelar wrote: > On Mon, Jul 18, 2016 at 11:28 AM, Jesse Gross wrote: >> On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: >> /home/jgross/openvswitch/datapath/linux/vxlan.c:1905:45: warning: >> incorrect type in assignment (different base types) >>

Re: [ovs-dev] [PATCH monitor_cond V9 05/12] ovsdb: enable jsonrpc-server to service "monitor_cond_change" request

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:53AM +0300, Liran Schour wrote: > ovsdb-server now accepts "monitor_cond_change" request. After conditions > change > we compose update notification according to the current state of the > database without using a change list before sending reply to the > monitor_con

Re: [ovs-dev] [PATCH 5/8] datapath: Add support for kernel 4.4

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 11:28 AM, Jesse Gross wrote: > On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: >> Most of changes are related to ip-fragment API and genetlink >> API changes. >> >> Signed-off-by: Pravin B Shelar > > I'm getting some sparse warnings with this patch. Not all of the

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 2:32 PM, Joe Stringer wrote: > On 17 July 2016 at 19:24, Pravin B Shelar wrote: >> This series contains patches to advance kernel datapath >> support for kernel upto 4.6. > > When I try to load the kernel module on Ubuntu's kernel-4.4, I get > complaints about GRE: > > [

Re: [ovs-dev] [PATCH 0/8] datapath: Add support for latest kernel release.

2016-07-18 Thread Joe Stringer
On 17 July 2016 at 19:24, Pravin B Shelar wrote: > This series contains patches to advance kernel datapath > support for kernel upto 4.6. When I try to load the kernel module on Ubuntu's kernel-4.4, I get complaints about GRE: [ 784.836361] openvswitch: Open vSwitch switching datapath 2.5.90 [

Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 03:56:58PM -0500, Ryan Moats wrote: > P.S. Ben, I'll be sending a rebased pair of incremental processing > patches later this evening, after doubles league. I'm getting back into "review mode" right now, so it's good timing. With luck, I'll just apply them quickly this time

[ovs-dev] [PATCH v23 2/2] ovn-controller: Add incremental processing to lflow_run and physical_run

2016-07-18 Thread Ryan Moats
This code changes to allow incremental processing of the logical flow and physical binding tables whenver possible. Note: flows created by physical_run for multicast_groups are *NOT* handled incrementally due to to be solved issues with GWs and local routers. Signed-off-by: Ryan Moats --- ovn/c

[ovs-dev] [PATCH v23 1/2] ovn-controller: Persist ovn flow tables

2016-07-18 Thread Ryan Moats
Ensure that ovn flow tables are persisted so that changes to them chan be applied incrementally - this is a prereq for making lflow_run and physical_run incremental. As part of this change, add a one-to-many hindex for finding desired flows by their parent's UUID. Also extend the mapping by match

[ovs-dev] [PATCH v23 0/2] ovn-controller: add incremental-processing

2016-07-18 Thread Ryan Moats
This patch set has the residual changes that didn't merge from v21. v22->v23: rebase v21->v22: Unlike previous versions, the persist ovn flow tables patch changes semantics of the desired physical flow hmap table from one-to-one to one-to-many, and uses the logical parent's UUID to disa

Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Russell Bryant
On Mon, Jul 18, 2016 at 4:56 PM, Ryan Moats wrote: > Russell Bryant wrote on 07/18/2016 03:47:34 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > > Date: 07/18/2016 03:47 PM > > Subject: [PATCH] ovn-controller: Drop remove_local_da

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 13:11, Ben Pfaff wrote: > On Mon, Jul 18, 2016 at 11:34:54AM -0700, Guru Shetty wrote: > > On 18 July 2016 at 10:35, Ben Pfaff wrote: > > > > > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > > > We should not use "peer" while connecting a router to a sw

Re: [ovs-dev] [PATCH 3/8] datapath: backport: allow output of MPLS packets on tunnel vports

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 7:35 AM, Jesse Gross wrote: > On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: >> Upstream commit: >> commit fe3a5f6c795810edb1646a840fec3c8c350c2a4e >> Author: Simon Horman >> >> openvswitch: allow output of MPLS packets on tunnel vports >> >> Curr

Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Ryan Moats
Russell Bryant wrote on 07/18/2016 03:47:34 PM: > From: Russell Bryant > To: dev@openvswitch.org > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > Date: 07/18/2016 03:47 PM > Subject: [PATCH] ovn-controller: Drop remove_local_datapath_by_binding(). > > ovn-controller has an hmap called 'local_

Re: [ovs-dev] [PATCH 1/8] datapath: Sync OVS recursive loop counter with upstream.

2016-07-18 Thread pravin shelar
On Mon, Jul 18, 2016 at 7:36 AM, Jesse Gross wrote: > On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: >> Signed-off-by: Pravin B Shelar >> --- >> datapath/actions.c | 31 --- >> 1 file changed, 12 insertions(+), 19 deletions(-) > > I have somewhat similar con

[ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Russell Bryant
ovn-controller has an hmap called 'local_datapaths' which tracks all OVN datapaths that have at least one port binding on the local chassis. This patch corrects the case where a port binding row is deleted from the southbound DB while it's still bound to the chassis, meaning it was deleted before

Re: [ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Russell Bryant
On Mon, Jul 18, 2016 at 4:22 PM, Ryan Moats wrote: > Russell Bryant wrote on 07/18/2016 03:01:41 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > > Date: 07/18/2016 03:01 PM > > Subject: [PATCH] ovn-controller: Remove local_datapat

Re: [ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Ryan Moats
Russell Bryant wrote on 07/18/2016 03:01:41 PM: > From: Russell Bryant > To: dev@openvswitch.org > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > Date: 07/18/2016 03:01 PM > Subject: [PATCH] ovn-controller: Remove local_datapaths_by_uuid. > > binding.c included a static local_datapaths_by_uui

Re: [ovs-dev] [PATCH monitor_cond V9 10/12] lib: add monitor_cond_change API to C IDL lib

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:58AM +0300, Liran Schour wrote: > Add to IDL API that allows the user to add and remove clauses on a table's > condition > iteratively. IDL maintain tables condition and send monitor_cond_change to > the server > upon condition change. > Add tests for conditional mon

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-07-18 Thread Jan Scheurich
> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Simon Horman > Sent: Monday, 18 July, 2016 14:14 > > > > Simon, very good guide, do push_eth and pop_eth also need to follow > > this? > > Not exactly. > > As of v12 of the patch-set there are two phases, compose and commit. > This was

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:34:54AM -0700, Guru Shetty wrote: > On 18 July 2016 at 10:35, Ben Pfaff wrote: > > > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > > We should not use "peer" while connecting a router to a switch. > > > (Doing so, will cause ovn-northd to consta

Re: [ovs-dev] [PATCH] ovn: Remove a completed TODO item.

2016-07-18 Thread Russell Bryant
On Mon, Jul 18, 2016 at 3:45 PM, Kyle Mestery wrote: > On Mon, Jul 18, 2016 at 2:32 PM, Russell Bryant wrote: > > This TODO item was implemented a while ago, so remove it from the TODO > > file. > > > > Fixes: 0ee8aaf658dd ("ovn: Send GARP on localnet.") > > Signed-off-by: Russell Bryant > > --

[ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Russell Bryant
binding.c included a static local_datapaths_by_uuid but it was not used for anything. In fact, the hash node used when inserting into this hmap is overwritten in another code path for a different hmap. Fixes: 263064aeaa31 ("Convert binding_run to incremental processing.") Signed-off-by: Russell B

Re: [ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling.

2016-07-18 Thread Russell Bryant
On Sat, Jul 16, 2016 at 10:31 PM, Ryan Moats wrote: > "dev" wrote on 07/14/2016 02:47:52 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Date: 07/14/2016 02:48 PM > > Subject: [ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling. > > Sent by: "dev" > > > > Remove the

Re: [ovs-dev] [PATCH] ovn: Remove a completed TODO item.

2016-07-18 Thread Kyle Mestery
On Mon, Jul 18, 2016 at 2:32 PM, Russell Bryant wrote: > This TODO item was implemented a while ago, so remove it from the TODO > file. > > Fixes: 0ee8aaf658dd ("ovn: Send GARP on localnet.") > Signed-off-by: Russell Bryant > --- > ovn/TODO | 6 -- > 1 file changed, 6 deletions(-) > > diff -

[ovs-dev] [PATCH] ovn: Remove a completed TODO item.

2016-07-18 Thread Russell Bryant
This TODO item was implemented a while ago, so remove it from the TODO file. Fixes: 0ee8aaf658dd ("ovn: Send GARP on localnet.") Signed-off-by: Russell Bryant --- ovn/TODO | 6 -- 1 file changed, 6 deletions(-) diff --git a/ovn/TODO b/ovn/TODO index 0a6225d..2077ccd 100644 --- a/ovn/TODO ++

[ovs-dev] Considering the possibility of integrating DPDK generic classifier APIs in OVS.

2016-07-18 Thread Chandran, Sugesh
Hi Dev Team, I submitted a RFC patch few months ago to optimize the tunneling performance in OVS-DPDK (specifically VxLAN performance) using Intel Fortville Flow director feature. More details about the patch can be found below. http://openvswitch.org/pipermail/dev/2016-March/067988.html The p

Re: [ovs-dev] [PATCH] ovs-vsctl: Add another Flow_Table example.

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 12:55:05 PM: > From: Ben Pfaff > To: dev@openvswitch.org > Cc: Ben Pfaff > Date: 07/18/2016 12:55 PM > Subject: [ovs-dev] [PATCH] ovs-vsctl: Add another Flow_Table example. > Sent by: "dev" > > Signed-off-by: Ben Pfaff > --- > utilities/ovs-vsctl.8.in | 9 -

Re: [ovs-dev] [PATCH 0/2] Python JSON parser improvements

2016-07-18 Thread Aaron Rosen
I'm facing this same issue with the _ovn-utils.c Python C extension patch I posted. If this gets approved I believe I need to so the same thing for the following files: ovn/lib/actions.h ovn/lib/expr.h ovn/lib/lex.h On Tue, Jul 12, 2016 at 2:37 PM, Terry Wilson wrote: > These patches should ad

Re: [ovs-dev] [PATCH 1/4] ovn: ovn-nbctl, the implementation of icmp4 reject actions

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 11:30:00 AM: > From: nickcooper-zhangtonghao > To: dev@openvswitch.org > Date: 07/18/2016 11:30 AM > Subject: [ovs-dev] [PATCH 1/4] ovn: ovn-nbctl, the implementation of > icmp4 reject actions > Sent by: "dev" > > Hi, > > Now that some reject functions have been implem

Re: [ovs-dev] vport-stt.ko

2016-07-18 Thread Jesse Gross
On Mon, Jul 18, 2016 at 12:12 PM, Amru OM wrote: > Greetings, > > > > I tried to load the STT module (vport-stt.ko), which is included with OVS > 2.5 distribution, in order to configure STT overlay tunnels between virtual > machines located in different hosts. However, when I insert vport-stt.ko >

[ovs-dev] [PATCH v1 2/2] ovn-northd: Add logical flows to support DHCPv6

2016-07-18 Thread Numan Siddique
OVN implements native DHCPv6. DHCPv6 options are stored in the 'DHCP_Options' NB table and logical ports refer to this table to configure the DHCPv6 options. For each logical port configured with DHCPv6 Options following flows are added - A logical flow which copies the DHCPv6 options to the DHCP

[ovs-dev] [PATCH v1 1/2] ovn-controller: Add 'put_dhcpv6_opts' action in ovn-controller

2016-07-18 Thread Numan Siddique
This patch adds a new OVN action 'put_dhcpv6_opts' to support native DHCPv6 in OVN. ovn-controller parses this action and adds a NXT_PACKET_IN2 OF flow with 'pause' flag set and the DHCPv6 options stored in 'userdata' field. When the valid DHCPv6 packet is received by ovn-controller, it frames a

[ovs-dev] [PATCH v1 0/2] ovn: Support native DHCPv6

2016-07-18 Thread Numan Siddique
This patch set implements native DHCPv6 in OVN. Patch 2 depends on the ovn-northd DHCPv4 patch - https://patchwork.ozlabs.org/patch/648487/ - Numan Siddique (2): ovn-controller: Add 'put_dhcpv6_opts' action in ovn-controller ovn-northd: Add logical flows to support DHCPv6

Re: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a router port is a switch port.

2016-07-18 Thread Ryan Moats
Guru Shetty wrote on 07/18/2016 01:48:36 PM: > From: Guru Shetty > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: ovs dev > Date: 07/18/2016 01:48 PM > Subject: Re: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a > router port is a switch port. > > On 18 July 2016 at 11:44, Ryan Moats wrote: > "d

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-07-18 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Lance Richardson" > Cc: "Amitabha Biswas" , dev@openvswitch.org > Sent: Monday, July 18, 2016 2:34:02 PM > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > > On Mon, Jul 18, 2016 at 01:50:40PM -0400, Lance Richardson wrote: > >

Re: [ovs-dev] [PATCH 7/8] datapath: Add support for kernel 4.6

2016-07-18 Thread Jesse Gross
On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/include/linux/if_ether.h > b/datapath/linux/compat/include/linux/if_ether.h > index b2cb56d..ac0f1ed 100644 > --- a/datapath/linux/compat/include/linux/if_ether.h > +++ b/datapath/linux/compat/include/lin

Re: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a router port is a switch port.

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 11:44, Ryan Moats wrote: > "dev" wrote on 07/18/2016 03:39:56 AM: > > > From: Gurucharan Shetty > > To: dev@openvswitch.org > > Date: 07/18/2016 01:39 PM > > Subject: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a > > router port is a switch port. > > Sent by: "dev" >

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 01:34:02 PM: > From: Ben Pfaff > To: Lance Richardson > Cc: dev@openvswitch.org > Date: 07/18/2016 01:34 PM > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > Sent by: "dev" > > On Mon, Jul 18, 2016 at 01:50:40PM -0400, Lance Richardson wrote: > > - O

Re: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a router port is a switch port.

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 03:39:56 AM: > From: Gurucharan Shetty > To: dev@openvswitch.org > Date: 07/18/2016 01:39 PM > Subject: [ovs-dev] [PATCH] ovn-northd: Warn when the peer of a > router port is a switch port. > Sent by: "dev" > > Signed-off-by: Gurucharan Shetty > --- Is just printing

Re: [ovs-dev] [PATCH monitor_cond V9 04/12] ovsdb: generate update notifications for monitor_cond session

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:45:52AM +0300, Liran Schour wrote: > Hold session's conditions in ovsdb_monitor_session_condition. Pass it > to ovsdb_monitor for generating "update2" notifications. > Add functions that can generate "update2" notification for a > "monitor_cond" session. > JSON cache is e

[ovs-dev] [PATCH] ovn-northd: Warn when the peer of a router port is a switch port.

2016-07-18 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 8 1 file changed, 8 insertions(+) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..0a093c7 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -2603,6 +2603,14 @@ build_lrouter

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 10:35, Ben Pfaff wrote: > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > We should not use "peer" while connecting a router to a switch. > > (Doing so, will cause ovn-northd to constantly create and destroy > > port_binding records which causes CPU utili

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 01:50:40PM -0400, Lance Richardson wrote: > - Original Message - > > From: "Ben Pfaff" > > To: "Amitabha Biswas" > > Cc: dev@openvswitch.org > > Sent: Monday, July 18, 2016 1:33:56 PM > > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > > > > On Fri,

Re: [ovs-dev] [PATCH 6/8] datapath: Add support for kernel 4.5

2016-07-18 Thread Jesse Gross
On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: > Signed-off-by: Pravin B Shelar > --- > acinclude.m4| 8 ++-- > datapath/linux/compat/include/linux/netdevice.h | 17 +++-- > datapath/vport-netdev.c | 2 +- > 3

Re: [ovs-dev] [patch_v8] ovn: Add local router support (RFC).

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/11/2016 10:07:03 AM: > From: Darrell Ball > To: dlu...@gmail.com, d...@openvswitch.com > Date: 07/11/2016 10:07 AM > Subject: [ovs-dev] [patch_v8] ovn: Add local router support (RFC). > Sent by: "dev" > > This patch adds local router support. The idea is to do openflow rule

Re: [ovs-dev] [PATCH] ovs-vsctl: Fix typo in comment.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 01:03:49PM -0500, Ryan Moats wrote: > "dev" wrote on 07/18/2016 11:15:19 AM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Ben Pfaff > > Date: 07/18/2016 11:15 AM > > Subject: [ovs-dev] [PATCH] ovs-vsctl: Fix typo in comment. > > Sent by: "dev" > > > > Signe

Re: [ovs-dev] [PATCH] ovn: Make it possible for CMS to detect when the OVN system is up-to-date.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 09:14:55AM -0700, Ben Pfaff wrote: > Until now, there has been no reliable for the CMS (or ovn-nbctl, or > anything else) to detect when changes made to the northbound configuration > have been passed through to the southbound database or to the hypervisors. > This commit ad

[ovs-dev] [PATCH v2] ovn: Make it possible for CMS to detect when the OVN system is up-to-date.

2016-07-18 Thread Ben Pfaff
Until now, there has been no reliable for the CMS (or ovn-nbctl, or anything else) to detect when changes made to the northbound configuration have been passed through to the southbound database or to the hypervisors. This commit adds this feature to the system, by adding sequence numbers to the no

Re: [ovs-dev] [PATCH 5/8] datapath: Add support for kernel 4.4

2016-07-18 Thread Jesse Gross
On Mon, Jul 18, 2016 at 4:24 AM, Pravin B Shelar wrote: > Most of changes are related to ip-fragment API and genetlink > API changes. > > Signed-off-by: Pravin B Shelar I'm getting some sparse warnings with this patch. Not all of them are new but I wasn't able to run it before due to my local ke

Re: [ovs-dev] [PATCH 4/9] datapath-windows: Fix bugs in Event.c around subscribe and lock

2016-07-18 Thread Sairam Venugopal
Hi Yin, Thanks for reviewing this. (1) has been addressed in a different patch. (2) We don¹t need the lock for OvsGetOpenInstance(). Thanks, Sairam On 7/18/16, 11:07 AM, "Yin Lin" wrote: >Thanks Sai for fixing the bugs. A few questions: > >1. There is also an unrelease lock in line 359, functi

Re: [ovs-dev] [PATCH 5/9] datapath-windows: Define new multicast conntrack events

2016-07-18 Thread Yin Lin
Acked-By: Yin Lin On Wed, Jul 13, 2016 at 4:38 PM, Sairam Venugopal wrote: > Define new Conntrack events (new and delete) and add support for > subscribing to these events. Parse out OVS_NL_ATTR_MCAST_GRP and store it > as part of OVS_EVENT_SUBSCRIBE structure. > > Signed-off-by: Sairam Venugop

Re: [ovs-dev] [PATCH] stream-unix: only use path-based socket names

2016-07-18 Thread Ben Pfaff
On Fri, Jul 08, 2016 at 10:01:53AM -0300, Thadeu Lima de Souza Cascardo wrote: > FreeBSD returns a socklen of sockaddr_storage when doing an accept on an unix > STREAM socket. The current code will assume it means a sun_path larger than 0. > > That breaks some tests like the one below which don't

Re: [ovs-dev] [PATCH 6/9] datapath-windows: Modify OvsCreateNlMsgFromCtEntry to make it reusable

2016-07-18 Thread Yin Lin
Acked-By: Yin Lin On Wed, Jul 13, 2016 at 4:38 PM, Sairam Venugopal wrote: > Tweak the OvsCreateNlMsgFromCtEntry() method to reuse it for creating > netlink messages from other files. Also define the function in Conntrack.h > to make it accessible. > > Signed-off-by: Sairam Venugopal > --- >

Re: [ovs-dev] [PATCH] ovn: Add 128-bit support for ct_label

2016-07-18 Thread Ben Pfaff
On Fri, Jul 15, 2016 at 08:33:59PM +0530, Numan Siddique wrote: > To support 128-bits in ct_label, the value of the ct_label is expected > as a hex string in the 'ct_commit' action. > > Added a new accessor in the 'mf_subvalue' struct to access ovs_be128 > values. > > Signed-Off-by: Numan Siddiqu

Re: [ovs-dev] [PATCH 4/9] datapath-windows: Fix bugs in Event.c around subscribe and lock

2016-07-18 Thread Yin Lin
Thanks Sai for fixing the bugs. A few questions: 1. There is also an unrelease lock in line 359, function OvsWaitEventIoctl. 2. Do we need the lock before OvsGetOpenInstance? If so, please release the lock inside "if (instance == NULL)"; if not, you should do the same thing on line 359. On Wed

Re: [ovs-dev] [PATCH] ovs-vsctl: Fix typo in comment.

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 11:15:19 AM: > From: Ben Pfaff > To: dev@openvswitch.org > Cc: Ben Pfaff > Date: 07/18/2016 11:15 AM > Subject: [ovs-dev] [PATCH] ovs-vsctl: Fix typo in comment. > Sent by: "dev" > > Signed-off-by: Ben Pfaff > --- > utilities/ovs-vsctl.c | 2 +- > 1 file changed, 1

Re: [ovs-dev] [PATCH v3] OVN: Fix ARP request flow in router ingresstable

2016-07-18 Thread Chandra Sekhar Vejendla
Ryan Moats/Omaha/IBM wrote on 07/18/2016 07:50:50 AM: > From: Ryan Moats/Omaha/IBM > To: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/18/2016 07:50 AM > Subject: Re: [ovs-dev] [PATCH v3] OVN: Fix ARP request flow in > router ingress table > > "dev" wrote on 07

[ovs-dev] Week 29

2016-07-18 Thread Bonesca - Jona
  [ View in browser ]( http://r.newsletter.bonescamail.nl/nru6qd7ioatrf.html )     This email was sent to dev@openvswitch.org You received this email because you are registered with Bonesca Import en Export BV   [ Unsubscribe here ]( http://r.newsletter.bonescamail.nl/nru6qd7i

[ovs-dev] [PATCH] ovs-vsctl: Add another Flow_Table example.

2016-07-18 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- utilities/ovs-vsctl.8.in | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/utilities/ovs-vsctl.8.in b/utilities/ovs-vsctl.8.in index 265ffde..4128801 100644 --- a/utilities/ovs-vsctl.8.in +++ b/utilities/ovs-vsctl.8.in @@ -875,9 +875,16 @@ Co

Re: [ovs-dev] Flow table Eviction group

2016-07-18 Thread Ben Pfaff
On Fri, Jul 15, 2016 at 03:55:23PM +0530, Kapil Adhikesavalu wrote: > Hello, > > Need some help in configuring flow table eviction group. > I tried to configure the eviction group, but i am seeing some warn/errors > and didn't come across any sample commands in web. > > 1. Can you please share so

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-07-18 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Amitabha Biswas" > Cc: dev@openvswitch.org > Sent: Monday, July 18, 2016 1:33:56 PM > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > > On Fri, May 20, 2016 at 11:17:57AM -0700, Amitabha Biswas wrote: > > This patch fixes the

Re: [ovs-dev] SFC: How about stages in both pipelines?

2016-07-18 Thread Henry Fourie
John, Work is currently in progress on a OVN LB that specifies a VIP to distribute load to a set of IP addresses: ovn-nbctl create load_balancer vips:30.0.0.1="172.16.1.2,172.16.1.3,172.16.1.4"` http://openvswitch.org/pipermail/dev/2016-February/066959.html What is needed to an OVN LB that w

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Flaviof
On Mon, Jul 18, 2016 at 3:00 AM, Gurucharan Shetty wrote: > We should not use "peer" while connecting a router to a switch. > (Doing so, will cause ovn-northd to constantly create and destroy > port_binding records which causes CPU utilization of ovn-controller to > spike up.) > > Fixes: 31114af7

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > We should not use "peer" while connecting a router to a switch. > (Doing so, will cause ovn-northd to constantly create and destroy > port_binding records which causes CPU utilization of ovn-controller to > spike up.) > > Fixes:

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-07-18 Thread Ben Pfaff
On Fri, May 20, 2016 at 11:17:57AM -0700, Amitabha Biswas wrote: > This patch fixes the "raceful" situation that occurs when ovn-nbctl > configures the OVN Northbound database till the point when the > ovn-controller picks up the corresponding logical flows and applies > them as OpenFlow rules on t

Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Chandra Sekhar Vejendla
Ryan Moats/Omaha/IBM wrote on 07/18/2016 10:03:35 AM: > From: Ryan Moats/Omaha/IBM > To: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > Cc: Guru Shetty , ovs dev > Date: 07/18/2016 10:03 AM > Subject: Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined > to router ip > > "dev" wrote on 07

[ovs-dev] [PATCH v3] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Chandra S Vejendla
When router ip is used as SNAT IP, traffic destined to router ip should not be dropped Fixes: 4685e523695c ("ovn: Support multiple addresses on a single logical router port.") Signed-off-by: Chandra Sekhar Vejendla --- ovn/northd/ovn-northd.c | 7 ++- 1 file changed, 6 insertions(+), 1 delet

Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 11:20:59 AM: > From: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > To: Guru Shetty > Cc: ovs dev > Date: 07/18/2016 11:21 AM > Subject: Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined > to router ip > Sent by: "dev" > > Guru Shetty wrote on 07/18/2016 08:03

[ovs-dev] [RFC 5/5] netdev: do not allow devices to be opened with conflicting types

2016-07-18 Thread Thadeu Lima de Souza Cascardo
When a device is already opened, netdev_open should verify that the types match, or else return an error. Otherwise, users might expect to open a device with a certain type and get a handle belonging to a different type. This also prevents certain conflicting configurations that would have a port

[ovs-dev] [RFC 4/5] dpctl: uses open_type when calling netdev_open

2016-07-18 Thread Thadeu Lima de Souza Cascardo
dpctl uses a user or database defined type when calling netdev_open. Instead, it should use the type from dpif_port_open_type. Otherwise, when using the internal type, it could open it with that type instead of the correct one, which would be tap or dummy. --- lib/dpctl.c | 17 - 1

  1   2   >