[ovs-dev] [PATCH v2] OVN: Support BUM traffic in the VTEP schema

2016-03-25 Thread Darrell Ball
This patch implements BUM support in the VTEP schema. This relates to BUM traffic flowing from a gateway towards HVs. This code would be relevant to HW gateways and the ovs-vtep simulator. In order to do this, the mcast macs remote table in the VTEP schema is populated based on the OVN SB port bin

[ovs-dev] [PATCH] Patch v2: OVN: Support BUM traffic in the VTEP schema

2016-03-25 Thread Darrell Ball
This patch implements BUM support in the VTEP schema. This relates to BUM traffic flowing from a gateway towards HVs. This code would be relevant to HW gateways and the ovs-vtep simulator. In order to do this, the mcast macs remote table in the VTEP schema is populated based on the OVN SB port bind

[ovs-dev] [CudaMailTagged] Adding Session Layer to OVS

2016-03-25 Thread Amrane Ait Zeouay
Hi, I'm new to the OVS and i want to know how can i add some layers (Session, Presentation and Application layer), and where i can find the full packet not until 4th layer so i can extract all layers. Thank you. best regards. -- Amrane Ait Zeouay Engineer Student in The Development of Software

Re: [ovs-dev] RFCv2: OVN database options

2016-03-25 Thread Kyle Mestery
On Thu, Mar 24, 2016 at 7:35 PM, Justin Pettit wrote: > >> On Mar 24, 2016, at 1:29 PM, Russell Bryant wrote: >> >> My preferred approach is to continue on the current path and build on the >> success we've had with OVSDB so far. We should take regular checkpoints >> and still remain willing to

Re: [ovs-dev] [PATCH v7 2/6] datapath-windows: Added recirculation support.

2016-03-25 Thread Nithin Raju
Thanks for incorporating the review comments. There are a few bugs, but we can fix them in subsequent patches I think. Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sorin Vinturis Date: Thursday, March 24, 2016 at 9:53 AM To: "dev@openvswitch.org" Subject: [ovs-dev]

Re: [ovs-dev] [PATCH v7 4/6] datapath-windows: Support for OVS_ACTION_ATTR_HASH attribute

2016-03-25 Thread Nithin Raju
Sorin, Thanks for the explanation about dpHash. We¹ll see how it gets used. Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sorin Vinturis Date: Thursday, March 24, 2016 at 9:53 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v7 4/6] datapath-windows: Support fo

Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Expand RSS hash calculation by MAC addresses

2016-03-25 Thread Wojciechowicz, RobertX
Hi Jesse, please find my answers inline. > -Original Message- > From: Jesse Gross [mailto:je...@kernel.org] > Sent: Thursday, March 24, 2016 11:20 PM > To: Wojciechowicz, RobertX > Cc: ovs dev > Subject: Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Expand RSS hash > calculation by MAC address

[ovs-dev] [PATCH v8 1/6] datapath-windows: Percpu allocation support function

2016-03-25 Thread Sorin Vinturis
Signed-off-by: Sorin Vinturis Co-authored-by: Alin Gabriel Serdean Acked-by: Nithin Raju --- datapath-windows/ovsext/Util.c | 16 datapath-windows/ovsext/Util.h | 1 + 2 files changed, 17 insertions(+) diff --git a/datapath-windows/ovsext/Util.c b/datapath-windows/ovsext/Util

[ovs-dev] [PATCH v8 0/6] datapath-windows: Recirculation support

2016-03-25 Thread Sorin Vinturis
This patch series adds support for recirculation to the OVS extension. Sorin Vinturis (6): [PATCH v8 1/6] datapath-windows: Percpu allocation support function [PATCH v8 2/6] datapath-windows: Added recirculation support. [PATCH v8 3/6] datapath-windows: Allow recirculation feature probe. [

[ovs-dev] [PATCH v8 2/6] datapath-windows: Added recirculation support.

2016-03-25 Thread Sorin Vinturis
Recirculation support for the OVS extension. Tested using PING and iperf with Driver Verifier enabled. Signed-off-by: Sorin Vinturis Co-authored-by: Alin Gabriel Serdean Reported-by: Sorin Vinturis Reported-at: https://github.com/openvswitch/ovs-issues/issues/104 Acked-by: Nithin Raju --- v2:

[ovs-dev] [PATCH v8 6/6] datapath-windows: Updated FAQ regarding Hyper-V recirculation.

2016-03-25 Thread Sorin Vinturis
Updated FAQ document to reflect recirculation support for Hyper-V. Signed-off-by: Sorin Vinturis Acked-by: Sairam Venugopal Acked-by: Nithin Raju --- FAQ.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FAQ.md b/FAQ.md index 4830c44..04ffc84 100644 --- a/FAQ.md +++ b/FAQ.

[ovs-dev] [PATCH v8 4/6] datapath-windows: Support for OVS_ACTION_ATTR_HASH attribute

2016-03-25 Thread Sorin Vinturis
Signed-off-by: Sorin Vinturis Acked-by: Sairam Venugopal Acked-by: Nithin Raju --- datapath-windows/ovsext/Actions.c| 38 datapath-windows/ovsext/DpInternal.h | 1 + datapath-windows/ovsext/Flow.c | 14 + 3 files changed, 53 insertions

[ovs-dev] [PATCH v8 3/6] datapath-windows: Allow recirculation feature probe.

2016-03-25 Thread Sorin Vinturis
Recirculation feature probe messages sent from userspace are suppressed by the OVS extension. This patch changes the current behaviour to allow feature probe for recirculation. Signed-off-by: Sorin Vinturis Acked-by: Sairam Venugopal Acked-by: Nithin Raju --- datapath-windows/ovsext/Flow.c |

[ovs-dev] [PATCH v8 5/6] datapath-windows: Extract flow metadata in execute cmd handler.

2016-03-25 Thread Sorin Vinturis
Signed-off-by: Sorin Vinturis Acked-by: Sairam Venugopal Acked-by: Nithin Raju --- datapath-windows/ovsext/DpInternal.h | 1 + datapath-windows/ovsext/Flow.c | 19 +++ datapath-windows/ovsext/Flow.h | 2 ++ datapath-windows/ovsext/User.c | 7 +++ 4 files

[ovs-dev] [PATCH v3] acinclude: Autodetect DPDK when configuring OVS

2016-03-25 Thread Bhanuprakash Bodireddy
When using DPDK datapath, the OVS configure script requires the DPDK build directory passed on --with-dpdk. This can be avoided if DPDK library, headers are in standard compiler search paths. This patch fixes the problem by searching for DPDK libraries in standard locations and configures OVS sour

[ovs-dev] [PATCH v9 2/6] datapath-windows: Added recirculation support.

2016-03-25 Thread Sorin Vinturis
Recirculation support for the OVS extension. Tested using PING and iperf with Driver Verifier enabled. Signed-off-by: Sorin Vinturis Co-authored-by: Alin Gabriel Serdean Reported-by: Sorin Vinturis Reported-at: https://github.com/openvswitch/ovs-issues/issues/104 Acked-by: Nithin Raju --- v2:

Re: [ovs-dev] [PATCH v8 0/6] datapath-windows: Recirculation support

2016-03-25 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:49:26PM +, Sorin Vinturis wrote: > This patch series adds support for recirculation to the OVS extension. > > Sorin Vinturis (6): > [PATCH v8 1/6] datapath-windows: Percpu allocation support function > [PATCH v8 2/6] datapath-windows: Added recirculation support.

Re: [ovs-dev] [PATCH v8 0/6] datapath-windows: Recirculation support

2016-03-25 Thread Sorin Vinturis
Hi Ben, I saw that an unwanted change to ovs-atomic.h header was introduced in "[PATCH v8 2/6] datapath-windows: Added recirculation support.". It is too late to revert and apply "[PATCH v9 2/6]" that I have just sent? -Sorin -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sen

Re: [ovs-dev] [PATCH RFC v3] ovs-ctl: Allow selective start for db and switch

2016-03-25 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 01:42:29PM -0400, Aaron Conole wrote: > Currently, ``ovs-ctl start'' will attempt to start both the DB and > vswitchd. This is quite convenient when the database already has all of > the configuration values required, and when using a single services file > for systemd integ

Re: [ovs-dev] [PATCH v8 0/6] datapath-windows: Recirculation support

2016-03-25 Thread Ben Pfaff
It's too late to amend the commit in-place. Please send a patch that reverts the undesired change. On Fri, Mar 25, 2016 at 03:35:47PM +, Sorin Vinturis wrote: > Hi Ben, > > I saw that an unwanted change to ovs-atomic.h header was introduced in > "[PATCH v8 2/6] datapath-windows: Added recir

Re: [ovs-dev] [PATCH v2] ovs-thread: Do not always end quiescent state in ovs_thread_create().

2016-03-25 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 11:31:09AM -0700, Daniele Di Proietto wrote: > A new thread must be started in a non quiescent state. There is a call > to ovsrcu_quiesce_end() in ovsthread_wrapper(), to enforce this. > > ovs_thread_create(), instead, is executed in the parent thread. It must > call ovsrc

Re: [ovs-dev] [PATCH v2] ovs-thread: Do not always end quiescent state in ovs_thread_create().

2016-03-25 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 11:31:09AM -0700, Daniele Di Proietto wrote: > A new thread must be started in a non quiescent state. There is a call > to ovsrcu_quiesce_end() in ovsthread_wrapper(), to enforce this. > > ovs_thread_create(), instead, is executed in the parent thread. It must > call ovsrc

[ovs-dev] [PATCH] datapath-windows: Revert unrelated change to ovs-atomic.h

2016-03-25 Thread Sorin Vinturis
There was an unwanted change to ovs-atomic.h header made by the recirculation patch, ee25964a60c6b2c6e60a4c5fbfc9e90cf304f970 commit. This patch reverts that change. Signed-off-by: Sorin Vinturis --- lib/ovs-atomic.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ovs-ato

Re: [ovs-dev] [PATCH] datapath-windows: Revert unrelated change to ovs-atomic.h

2016-03-25 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sorin Vinturis Date: Friday, March 25, 2016 at 8:56 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH] datapath-windows: Revert unrelated change to ovs-atomic.h >There was an unwanted change to ovs-atomic.h head

[ovs-dev] [PATCH] datapath-windows: fix alignment in Flow.c

2016-03-25 Thread Nithin Raju
Cosmetic changes. Acked-by: Nithin Raju --- datapath-windows/ovsext/Flow.c | 56 +- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c index f15a3de..02c41b7 100644 --- a/datapath

Re: [ovs-dev] [PATCH] datapath-windows: Revert unrelated change to ovs-atomic.h

2016-03-25 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 03:56:01PM +, Sorin Vinturis wrote: > There was an unwanted change to ovs-atomic.h header made by the > recirculation patch, ee25964a60c6b2c6e60a4c5fbfc9e90cf304f970 commit. > This patch reverts that change. > > Signed-off-by: Sorin Vinturis Applied, thanks! _

Re: [ovs-dev] [PATCH] datapath-windows: fix alignment in Flow.c

2016-03-25 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 09:34:17AM -0700, Nithin Raju wrote: > Cosmetic changes. > > Acked-by: Nithin Raju That should be a Signed-off-by, if it's your change. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] datapath-windows: Revert unrelated change to ovs-atomic.h

2016-03-25 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sorin Vinturis > Trimis: Friday, March 25, 2016 5:56 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH] datapath-windows: Revert unrelated change to > ovs-atomic.h > >

Re: [ovs-dev] [ovsdb-server multithreading RFC 2/9] ovsdb: Make 'remote' opaque in ovsdb_jsonrpc_session

2016-03-25 Thread Ben Pfaff
On Mon, Mar 21, 2016 at 01:00:53PM -0700, Andy Zhou wrote: > On Fri, Mar 18, 2016 at 5:20 PM, Ben Pfaff wrote: > > > On Thu, Mar 03, 2016 at 12:13:21AM -0800, Andy Zhou wrote: > > > It turns out there is no need for ovsdb_jsonrcp_session to have access > > > > s/jsonrcp/jsonrpc/ > > > > > to the

Re: [ovs-dev] [ovsdb-server multithreading RFC 5/9] ovsdb: Add a 'big lock' to serialize all OVSDB operations

2016-03-25 Thread Ben Pfaff
On Mon, Mar 21, 2016 at 01:05:09PM -0700, Andy Zhou wrote: > On Fri, Mar 18, 2016 at 6:08 PM, Ben Pfaff wrote: > > > On Thu, Mar 03, 2016 at 12:13:24AM -0800, Andy Zhou wrote: > > > Add a global lock to serialize all OVSDB operations. It is a simple > > > locking scheme to implement and to reason

Re: [ovs-dev] [ovsdb-server multithreading RFC 6/9] ovsdb: Add IPC messages for thread communication

2016-03-25 Thread Ben Pfaff
On Mon, Mar 21, 2016 at 01:20:35PM -0700, Andy Zhou wrote: > On Fri, Mar 18, 2016 at 6:24 PM, Ben Pfaff wrote: > > > On Thu, Mar 03, 2016 at 12:13:25AM -0800, Andy Zhou wrote: > > > Add a command queue for the main thread to pass commands to the > > > sessions thread. > > > > > > Defined an imple

[ovs-dev] [PATCH v2] datapath-windows: Revert unrelated change to ovs-atomic.h

2016-03-25 Thread Sorin Vinturis
There was an unwanted change to ovs-atomic.h header made by the recirculation patch, ee25964a60c6b2c6e60a4c5fbfc9e90cf304f970 commit. This patch reverts that change. Signed-off-by: Sorin Vinturis Acked-by: Nithin Raju Acked-by: Alin Gabriel Serdean --- lib/ovs-atomic.h | 2 +- 1 file changed,

Re: [ovs-dev] [PATCH RFC] OVN: Openstack floating ip support

2016-03-25 Thread Ryan Moats
"dev" wrote on 03/22/2016 04:19:07 PM: > From: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > To: dev@openvswitch.org > Date: 03/22/2016 04:19 PM > Subject: [ovs-dev] [PATCH RFC] OVN: Openstack floating ip support > Sent by: "dev" > > This patch adds distributed floating ip support for ovn. The as

[ovs-dev] [PATCH v2] datapath-windows: fix alignment in Flow.c

2016-03-25 Thread Nithin Raju
Cosmetic changes. Signed-off-by: Nithin Raju --- datapath-windows/ovsext/Flow.c | 56 +- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c index f15a3de..02c41b7 100644 --- a/dat

Re: [ovs-dev] [PATCH v2] datapath-windows: fix alignment in Flow.c

2016-03-25 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 10:16:08AM -0700, Nithin Raju wrote: > Cosmetic changes. > > Signed-off-by: Nithin Raju Applied, thanks! ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] datapath-windows: Add Windows thread atomic APIs for x64 binaries.

2016-03-25 Thread Sorin Vinturis
This patch modifies atomic operations library export header to include Windows thread atomic APIs, ovs-atomic-msvc.h instead of ovs-atomic-pthreads.h, for both x86 and x64 binaries. Signed-off-by: Sorin Vinturis --- lib/ovs-atomic.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

Re: [ovs-dev] [PATCH RFC] OVN: Openstack floating ip support

2016-03-25 Thread Guru Shetty
It would really help if you give packet walkthrough when following 2 simultaneous connections happen at the same time. 1. east-west without NAT between the private IP addresses 2 east-west with floating IP. You also mentioned in the meeting (if I remember correctly) on how you have to keep a part

Re: [ovs-dev] [PATCH] datapath-windows: Add Windows thread atomic APIs for x64 binaries.

2016-03-25 Thread Guru Shetty
On 25 March 2016 at 10:41, Sorin Vinturis wrote: > This patch modifies atomic operations library export header to > include Windows thread atomic APIs, ovs-atomic-msvc.h instead of > ovs-atomic-pthreads.h, for both x86 and x64 binaries. > > Signed-off-by: Sorin Vinturis > Have you gone through

[ovs-dev] [PATCH v2] datapath-windows: Update Recirculation to use the right parameters

2016-03-25 Thread Sairam Venugopal
Update OvsLookupFlow() to include flowKey->RecircId when computing hash. Use the right source port Id for checking if a packet is received or transmitted. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Actions.c | 2 +- datapath-windows/ovsext/Flow.c| 3 +++ 2 files changed, 4

[ovs-dev] [PATCH] datapath-windows: Remove unnecessary keylen computation in Flow.c

2016-03-25 Thread Sairam Venugopal
destKey->l2.keylen gets reset after this line. This change doesn't help with hash computation and can be removed. This is what destKey->l2.keyLen gets set to: destKey->l2.keyLen = OVS_WIN_TUNNEL_KEY_SIZE + OVS_L2_KEY_SIZE - destKey->l2.offset; Signed-off-by: Sairam Venugo

Re: [ovs-dev] [PATCH v2] ovs-thread: Do not always end quiescent state in ovs_thread_create().

2016-03-25 Thread Daniele Di Proietto
On 25/03/2016 08:46, "Ben Pfaff" wrote: >On Thu, Mar 24, 2016 at 11:31:09AM -0700, Daniele Di Proietto wrote: >> A new thread must be started in a non quiescent state. There is a call >> to ovsrcu_quiesce_end() in ovsthread_wrapper(), to enforce this. >> >> ovs_thread_create(), instead, is exe

Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Expand RSS hash calculation by MAC addresses

2016-03-25 Thread Jesse Gross
On Fri, Mar 25, 2016 at 7:45 AM, Wojciechowicz, RobertX wrote: > Hi Jesse, > > please find my answers inline. > >> -Original Message- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Thursday, March 24, 2016 11:20 PM >> To: Wojciechowicz, RobertX >> Cc: ovs dev >> Subject: Re: [ov

Re: [ovs-dev] [PATCH v4 03/22] Rename all functions in list.h with ovs_ prefix

2016-03-25 Thread Ryan Moats
"dev" wrote on 03/22/2016 06:17:34 PM: > From: b...@skyportsystems.com > To: dev@openvswitch.org > Cc: Ben Warren > Date: 03/22/2016 06:27 PM > Subject: [ovs-dev] [PATCH v4 03/22] Rename all functions in list.h > with ovs_ prefix > Sent by: "dev" > > From: Ben Warren > > This attempts to prev

Re: [ovs-dev] [PATCH v4 03/22] Rename all functions in list.h with ovs_ prefix

2016-03-25 Thread Ben Warren
> On Mar 25, 2016, at 12:47 PM, Ryan Moats wrote: > > "dev" wrote on 03/22/2016 06:17:34 PM: > > > From: b...@skyportsystems.com > > To: dev@openvswitch.org > > Cc: Ben Warren > > Date: 03/22/2016 06:27 PM > > Subject: [ovs-dev] [PATCH v4 03/22] Rename all functions in list.h > > with ovs_ p

[ovs-dev] [PATCH_v5 06/26] Move BLDASSERT macros to compiler header file

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/compiler.h | 30 ++ lib/util.h | 28 2 files changed, 30 insertions(+), 28 deletions(-) diff --git a/include/openvswitch/compiler.h b/include/openvswi

[ovs-dev] [PATCH_v5 02/26] Remove lib/list.h completely

2016-03-25 Thread ben
From: Ben Warren All code is now in include/openvswitch/list.h Signed-off-by: Ben Warren Acked-by: Ryan Moats --- lib/automake.mk | 1 - lib/bfd.c | 2 +- lib/dp-packet.h | 2 +- lib/dpif-netdev.c | 2 +- lib/fat-rwlock.c

[ovs-dev] [PATCH_v5 01/26] Move contents of lib/list.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Most of the list code is properly namespaced, so is OK to move to the global export directory. Some "lib/util.h" code had to move to the other directory as well, but I've tried to make that as small as possible Signed-off-by: Ben Warren Acked-by: Ryan Moats --- include/openv

[ovs-dev] [PATCH_v5 00/26] rework code base for third-party linking

2016-03-25 Thread ben
From: Ben Warren This patch set is an attempt at making a 'dev' distribution for OpenVswitch The goal is to provide shared libraries and header files so that third party applications can link in OVS rather than always using the command line utilities. Changes in v5: - Rebased against TOT - Moved

[ovs-dev] [PATCH_v5 07/26] Break tun-metadata.h into private and public parts

2016-03-25 Thread ben
From: Ben Warren Public (struct definitions and some prototypes) go in include/openvswitch Signed-off-by: Ben Warren --- include/openvswitch/automake.mk| 1 + include/openvswitch/tun-metadata.h | 101 + lib/tun-metadata.h | 80 +---

[ovs-dev] [PATCH_v5 04/26] Move ofp-parse.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/ofp-parse.h | 106 lib/automake.mk | 1 - lib/dpctl.c | 2 +- lib/learning-switch.c | 2 +- li

[ovs-dev] [PATCH_v5 05/26] Move lib/ofpbuf.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren Acked-by: Ryan Moats --- include/openvswitch/automake.mk | 1 + include/openvswitch/ofpbuf.h| 277 lib/automake.mk | 1 - lib/bfd.c | 2 +- lib/bundle.c

[ovs-dev] [PATCH_v5 09/26] Break packets.h into private and public parts

2016-03-25 Thread ben
From: Ben Warren Public (struct definitions and some prototypes) go in include/openvswitch Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/packets.h | 64 + lib/packets.h | 43 +---

[ovs-dev] [PATCH_v5 08/26] Misc cleanup with "util.h" header files

2016-03-25 Thread ben
From: Ben Warren Removed from redundant #includes and moved some macros to different file scope Signed-off-by: Ben Warren Acked-by: Ryan Moats --- include/openvswitch/compiler.h | 3 +++ include/openvswitch/util.h | 50 - lib/util.h

[ovs-dev] [PATCH_v5 16/26] Move lib/type-props.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/type-props.h | 55 lib/automake.mk | 1 - lib/classifier.h | 2 +- lib/fatal-signal.c | 2 +- li

[ovs-dev] [PATCH_v5 12/26] Remove inter-header dependencies in OVN files

2016-03-25 Thread ben
From: Ben Warren These includes were needed when "meta-flow.h" was refactored, even though neither of affected files included that header file directly. Signed-off-by: Ben Warren --- ovn/controller/ovn-controller.c | 2 ++ ovn/northd/ovn-northd.c | 1 + 2 files changed, 3 insertions(+)

[ovs-dev] [PATCH_v5 15/26] Break uuid.h into private and public parts

2016-03-25 Thread ben
From: Ben Warren Public (struct definitions and some prototypes) go in include/openvswitch Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/uuid.h | 35 +++ lib/ofp-print.c | 1 + lib/ofp-util.c

[ovs-dev] [PATCH_v5 11/26] Move lib/match.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/match.h | 214 lib/automake.mk | 1 - lib/classifier.h| 2 +- lib/dpctl.c | 2 +- li

[ovs-dev] [PATCH_v5 17/26] Break netdev.h into private and public parts

2016-03-25 Thread ben
From: Ben Warren Public (struct definitions and some prototypes) go in include/openvswitch Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/netdev.h| 87 + lib/netdev.h| 66 +---

[ovs-dev] [PATCH_v5 10/26] Break flow.h into private and public parts

2016-03-25 Thread ben
From: Ben Warren Public (struct definitions and some prototypes) go in include/openvswitch Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/flow.h | 197 lib/flow.h | 175 +-

[ovs-dev] [PATCH_v5 25/26] Move lib/ofp-prop.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/ofp-prop.h | 138 lib/automake.mk | 1 - lib/ofp-actions.c | 2 +- lib/ofp-prop.c | 3 +- li

[ovs-dev] [PATCH_v5 18/26] Assorted #include additions

2016-03-25 Thread ben
From: Ben Warren Necessary in preparation for making ofp-util.h public Signed-off-by: Ben Warren --- ofproto/ofproto-dpif-ipfix.c | 1 + ovn/controller/ofctrl.c | 3 +++ ovn/controller/physical.c| 2 ++ ovn/controller/pinctrl.c | 1 + ovn/lib/actions.c| 1 + ovn/lib/ex

[ovs-dev] [PATCH_v5 20/26] Debian: build openvswitch as shared libraries

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- debian/openvswitch-common.install | 3 +++ debian/rules | 5 - 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/debian/openvswitch-common.install b/debian/openvswitch-common.install index 3264ea5..73c8fc9 1006

[ovs-dev] [PATCH_v5 26/26] Move lib/ofp-print.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/ofp-print.h | 58 + lib/automake.mk | 1 - lib/dpif-netdev.c | 11 lib/dpif.c | 2 +-

[ovs-dev] [PATCH_v5 21/26] Debian: add libopenvswitch-dev build target

2016-03-25 Thread ben
From: Ben Warren Add shared objects and header files Signed-off-by: Ben Warren --- debian/automake.mk| 1 + debian/control| 15 +++ debian/libopenvswitch-dev.install | 5 + 3 files changed, 21 insertions(+) create mode 100644 debian/libope

[ovs-dev] [PATCH_v5 23/26] Move lib/ofp-msgs.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk | 1 + include/openvswitch/ofp-msgs.h | 752 lib/automake.mk | 5 +- lib/learning-switch.c | 14 +- lib/ofp-errors.c| 7 +- l

[ovs-dev] [PATCH_v5 24/26] Move lib/ofp-actions.h to include/openvswitch directory

2016-03-25 Thread ben
From: Ben Warren Signed-off-by: Ben Warren --- include/openvswitch/automake.mk |1 + include/openvswitch/ofp-actions.h | 1030 + lib/automake.mk |1 - lib/bundle.c | 10 +- lib/learn.c |

[ovs-dev] [PATCH_v5 13/26] Move a couple of bitmap macros to exportable header

2016-03-25 Thread ben
From: Ben Warren This allows lib/meta-flow.h to become exportable Signed-off-by: Ben Warren --- include/openvswitch/util.h | 4 lib/bitmap.h | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/openvswitch/util.h b/include/openvswitch/util.h index