Re: [ovs-dev] [ovs-dev, v2, 09/23] Break tun-metadata.h into private and public parts

Hi Ryan, > On Mar 17, 2016, at 10:22 AM, Ryan Moats wrote: > > > 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-metada

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

On 03/20/2016 10:38 PM, Bodireddy, Bhanuprakash wrote: Hello Panu, Thanks for the comments. I have follow up question on the auto discovery of DPDK install location. As the DPDK install location can vary with distros, do OVS configure script has to search for DPDK libraries in /usr/local/sh

Re: [ovs-dev] [PATCH v4 01/12] netdev-dpdk: Consistent variable naming.

A few comments inline. Best regards, Ilya Maximets. On 17.03.2016 02:39, Daniele Di Proietto wrote: > In different functions we use different variable names ('netdev_', 'netdev', > 'dev', 'vhost_dev', ...) for the same objects. > > This commit changes the code to comply with the following conven

[ovs-dev] [RFCv2] ovs-ctl: Allow selective start for db and switch

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 integration. The same goes for the ``ovs-ctl stop'' command. However, ther

Re: [ovs-dev] [PATCH v4 12/12] netdev-dpdk: Use ->reconfigure() call to change rx/tx queues.

One comment below. Best regards, Ilya Maximets. On 17.03.2016 02:40, Daniele Di Proietto wrote: > This introduces in dpif-netdev and netdev-dpdk the first use for the > newly introduce reconfigure netdev call. > > When a request to change the number of queues comes, netdev-dpdk will > remember t

Re: [ovs-dev] [PATCH v4 00/12] Reconfigure netdev at runtime

There are few style problems in patches. I've sent corresponding mails. Other than this: Tested-by: Ilya Maximets Acked-by: Ilya Maximets On 17.03.2016 02:39, Daniele Di Proietto wrote: > Currently we treat set_multiq() calls specially in netdev and dpif-netdev: > every pmd threa

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

When using DPDK datapath, the OVS configure script requires the DPDK build directory passed on --with-dpdk. This can be avoided if the DPDK is installed in standard location i.e /usr/local/share/dpdk (or) /usr/share/dpdk This patch fixes the problem by searching for DPDK libraries in standard loca

[ovs-dev] World largest tablets store

96 World biggest FDA shop p{margin:10px 0;padding:0;} table{border-collapse:collapse;} h1,h2,h3,h4,h5,h6{display:block;margin:0;padding:0;} img,a img{border:0;height:auto;outline:none;text-decoration:none;} body,#bodyTable,#bodyCell{height:100%;margin:0;padding:0;width:100%;} #outlo

[ovs-dev] [PATCH v3 0/3] ovn: Apply ACL changes to existing connections.

Prior to this commit, once a connection had been committed to the connection tracker, the connection would continue to be allowed, even if the policy defined in the ACL table changed. This patch changes the implementation so that existing connections are affected by policy changes. The implementa

[ovs-dev] [PATCH v3 1/3] ovn: Update ACL flow docs.

Apply some minor updates to the description of flows related to ACLs. Signed-off-by: Russell Bryant Acked-by: Han Zhou --- ovn/northd/ovn-northd.8.xml | 8 +--- ovn/ovn-sb.xml | 11 +-- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/ovn/northd/ovn-nort

[ovs-dev] [PATCH v3 2/3] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.

Update the "ct_commit;" logical flow action to optionally take one or two parameters, setting the value of "ct_mark" or "ct_label". Supported ct_commit syntax now includes: ct_commit; ct_commit(); ct_commit(ct_mark=1); ct_commit(ct_label=1); ct_commit(ct_mark=1, ct_label=1); S

[ovs-dev] [PATCH v3 3/3] ovn: Apply ACL changes to existing connections.

Prior to this commit, once a connection had been committed to the connection tracker, the connection would continue to be allowed, even if the policy defined in the ACL table changed. This patch changes the implementation so that existing connections are affected by policy changes. The implementa

[ovs-dev] [PATCH] ovn: Remove outdated debug message.

ovn-northd used to only call ovnsb_db_run() and ovnnb_db_run() when seqnos indicated that they had changed. It now always calls these functions from the main loop. Remove this outdated log message that indicates that we called this function because contents changed. Signed-off-by: Russell Bryant

[ovs-dev] [PATCH] [Patch V2] OVN: Logical-Physical Separation and L2 SW Gateways

The following patch series implements physical-logical separation to be used presently by gateways and localnet. The patch series also includes L2 SW Gateway support which depends on the physical-logical separation changes. The physical logical separation changes allow the physical network to be m

[ovs-dev] [PATCH v2] Physical-Logical Separation and L2 SW Gateway

Combined Physical-Logical Separation and L2 SW Gateway patches due to dependencies. The following patch series implements physical-logical separation to be used presently by gateways and localnet. The patch series also includes L2 SW Gateway support which depends on the physical-logical separati

Re: [ovs-dev] [ovs-dev, v2, 09/23] Break tun-metadata.h into private and public parts

On Thu, Mar 17, 2016 at 11:22:13AM -0600, Ryan Moats wrote: > > > 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

Re: [ovs-dev] [PATCH] list.h: Define OVS_LIST_POISON statically

Applied, thanks. On Sun, Mar 20, 2016 at 02:37:45AM +, Nithin Raju wrote: > I am fine with the suggestion and the patch you sent. Thanks Ben. > > -Original Message- > From: Ben Pfaff > Date: Friday, March 18, 2016 at 1:47 PM > To: Nithin Raju > Cc: "dev@openvswitch.org" , Yin Lin >

Re: [ovs-dev] [PATCH v2 04/23] Move contents of lib/list.h to include/openvswitch directory

Thanks. I'm starting to get too many patch rejects at this point in the series. Note that I skipped patch 1. That's because I still don't see how it is useful to clients at this point in the series. On Sat, Mar 19, 2016 at 03:26:22PM -0700, Ben Warren wrote: > Thanks! I'll resolve and re-post.

[ovs-dev] [PATCH] ofpbuf: Don't include unneeded "packets.h".

Signed-off-by: Ben Pfaff --- lib/ofpbuf.h| 3 ++- lib/rtnetlink.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/ofpbuf.h b/lib/ofpbuf.h index 8008cb4..0b66302 100644 --- a/lib/ofpbuf.h +++ b/lib/ofpbuf.h @@ -20,9 +20,10 @@ #include #include #include "list.h"

Re: [ovs-dev] [PATCH monitor_cond V4 02/17] ovsdb: add conditions utilities to support monitor_cond

On Thu, Feb 25, 2016 at 09:26:42AM +0200, Liran Schour wrote: > Ben Pfaff wrote on 25/02/2016 02:25:32 AM: > > On Thu, Feb 18, 2016 at 11:46:40AM +, Liran Schour wrote: > > > Change ovsdb_condition to be a 3-element json array or a boolean > value. > > > Conditions utilities will be used late

Re: [ovs-dev] [PATCH] [Patch V2] OVN: Logical-Physical Separation and L2 SW Gateways

On Mon, Mar 21, 2016 at 8:47 AM, Darrell Ball wrote: > The following patch series implements physical-logical separation > to be used presently by gateways and localnet. > The patch series also includes L2 SW Gateway support which depends on the > physical-logical separation changes. > > The phys

[ovs-dev] [PATCH] db-ctl-base: Fix a typo.

Signed-off-by: Russell Bryant --- lib/db-ctl-base.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c index 849883b..ffef041 100644 --- a/lib/db-ctl-base.c +++ b/lib/db-ctl-base.c @@ -62,7 +62,7 @@ static void (*ctl_exit_func)(int status) =

Re: [ovs-dev] Deletion of Indirect Groups on Ovs2.5

> On Mar 20, 2016, at 5:21 PM, Simon Horman wrote: > > On Fri, Mar 18, 2016 at 09:45:08AM -0700, Jarno Rajahalme wrote: >> OpenFlow 1.5 spec says: “The group type need not be specified for the delete >> request.” > > Thanks Jarno. What I am wondering at this point is what > omitting the group

[ovs-dev] [PATCH] openvswitch: Fix checking for new expected connections.

OVS should call into CT NAT for packets of new expected connections only when the conntrack state is persisted with the 'commit' option to the OVS CT action. The test for this condition is doubly wrong, as the CT status field is ANDed with the bit number (IPS_EXPECTED_BIT) rather than the mask (IP

Re: [ovs-dev] [PATCH v3 2/3] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.

On 21 March 2016 at 07:54, Russell Bryant wrote: > Update the "ct_commit;" logical flow action to optionally take > one or two parameters, setting the value of "ct_mark" or "ct_label". > Supported ct_commit syntax now includes: > > ct_commit; > ct_commit(); > ct_commit(ct_mark=1); >

Re: [ovs-dev] [patch v2] openvswitch: using a bit shift as a mask

I just sent the patch to netfilter-devel and kernel-janitors with you as Reported-by. Jarno > On Mar 19, 2016, at 5:09 AM, Dan Carpenter wrote: > > It doesn't make sense to apply my patch. Can you please send your patch > but give me a Reported-by: tag? > > regards, > dan carpenter > ___

Re: [ovs-dev] [PATCH v3 2/3] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.

On Mon, Mar 21, 2016 at 11:16 AM, Guru Shetty wrote: > > > On 21 March 2016 at 07:54, Russell Bryant wrote: > >> Update the "ct_commit;" logical flow action to optionally take >> one or two parameters, setting the value of "ct_mark" or "ct_label". >> Supported ct_commit syntax now includes: >> >

Re: [ovs-dev] linux-next: Tree for Mar 21 (openvswitch)

On 03/20/16 21:13, Stephen Rothwell wrote: > Hi all, > > Please do not add any v4.7 related material to your linux-next included > trees until after v4.6-rc1 is released. > > Changes since 20160318: > on i386: ERROR: "nf_nat_icmp_reply_translation" [net/openvswitch/openvswitch.ko] undefined!

Re: [ovs-dev] [PATCH monitor_cond V5 02/18] ovsdb: add conditions utilities to support monitor_cond

On Fri, Mar 04, 2016 at 08:08:57AM +, Liran Schour wrote: > Change ovsdb_condition to be a 3-element json array or a boolean value. > Conditions utilities will be used later for conditional monitoring. > > Signed-off-by: Liran Schour Compiler error building 32-bit: ../tests/test-ovsdb.c:979

Re: [ovs-dev] [ovs-dev, monitor_cond, V5, 00/18] ovsdb: create column index mapping between ovsdb row to monitor row

On Fri, Mar 18, 2016 at 09:48:04AM -0600, Ryan Moats wrote: > When I tried to apply these patches, they didn't go in clean, and when I > tried to look up the dependent patch sets, > the list was rather unmanageable. You can fetch it from: https://github.com/liranschour/ovs.git monitor_cond

Re: [ovs-dev] [PATCH monitor_cond V5 02/18] ovsdb: add conditions utilities to support monitor_cond

On Fri, Mar 04, 2016 at 08:08:57AM +, Liran Schour wrote: > Change ovsdb_condition to be a 3-element json array or a boolean value. > Conditions utilities will be used later for conditional monitoring. > > Signed-off-by: Liran Schour Thanks for the revision! Please add a description of the

[ovs-dev] [PATCH v3 00/21] rework code base for third-party linking

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 V3: - Removed patches that have b

[ovs-dev] [PATCH v3 01/21] Move contents of lib/list.h to include/openvswitch directory

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 v3 06/21] Move BLDASSERT macros to compiler header file

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 v3 08/21] Misc cleanup with "util.h" header files

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 v3 07/21] Break tun-metadata.h into private and public parts

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 v3 04/21] Move ofp-parse.h to include/openvswitch directory

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 v3 09/21] Break packets.h into private and public parts

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 v3 15/21] Move lib/type-props.h to include/openvswitch directory

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 v3 12/21] Remove inter-header dependencies in OVN files

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 v3 14/21] Break uuid.h into private and public parts

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 v3 17/21] Assorted #include additions

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 v3 16/21] Break netdev.h into private and public parts

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 v3 11/21] Move lib/match.h to include/openvswitch directory

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 v3 10/21] Break flow.h into private and public parts

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 v3 19/21] Debian: build openvswitch as shared libraries

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 v3 20/21] Debian: add libopenvswitch-dev build target

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 v3 02/21] Remove lib/list.h completely

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 v3 05/21] Move lib/ofpbuf.h to include/openvswitch directory

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] datapath: Check for sock argument to v6ops->fragment.

Ubuntu 3.13.0-83-generic has backported a patch that adds an intermediate version of the v6ops->fragment function that doesn't seem to ever been part of a released upstream kernel. This version is missing the sock argument to the fragment function. Since we already have a backported version of the

Re: [ovs-dev] [PATCH] Dynamically reconnect ovn-controller if ovn-remote value changes

On Mon, Mar 14, 2016 at 1:35 PM, Ryan Moats wrote: > From: RYAN D. MOATS > > Allows for auto detection and reconnect if the ovn-remote needs > to change. ovn-controller test case updated to include testing > this code > > Signed-off-by: RYAN D. MOATS > --- > ovn/controller/ovn-controller.c |

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On Mon, Mar 21, 2016 at 12:43 PM, Jesse Gross wrote: > Ubuntu 3.13.0-83-generic has backported a patch that adds an intermediate > version of the v6ops->fragment function that doesn't seem to ever been > part of a released upstream kernel. This version is missing the sock > argument to the fragme

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On Mon, Mar 21, 2016 at 12:49 PM, Russell Bryant wrote: > On Mon, Mar 21, 2016 at 12:43 PM, Jesse Gross wrote: >> >> Ubuntu 3.13.0-83-generic has backported a patch that adds an intermediate >> version of the v6ops->fragment function that doesn't seem to ever been >> part of a released upstream k

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

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 remote data structure. Make it opaque as a 'void *' pointer. > > The pointe

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

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 about correctness, without > > much changes to core of OVSDB impleme

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On Mon, Mar 21, 2016 at 12:59 PM, Jesse Gross wrote: > On Mon, Mar 21, 2016 at 12:49 PM, Russell Bryant wrote: > > On Mon, Mar 21, 2016 at 12:43 PM, Jesse Gross wrote: > >> > >> Ubuntu 3.13.0-83-generic has backported a patch that adds an > intermediate > >> version of the v6ops->fragment funct

Re: [ovs-dev] [PATCH monitor_cond V5 03/18] ovsdb: allow unmonitored columns in condition evaluation

On Fri, Mar 04, 2016 at 08:08:58AM +, Liran Schour wrote: > This commit allows to add unmonitored columns to a monitored table > due to condition update. > It will be used to evaluate conditions on unmonitored columns. > Update notification includes only monitored columns. > Due to the limited

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On Mon, Mar 21, 2016 at 1:07 PM, Russell Bryant wrote: > > > On Mon, Mar 21, 2016 at 12:59 PM, Jesse Gross wrote: >> >> On Mon, Mar 21, 2016 at 12:49 PM, Russell Bryant wrote: >> > On Mon, Mar 21, 2016 at 12:43 PM, Jesse Gross wrote: >> >> >> >> Ubuntu 3.13.0-83-generic has backported a patch t

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

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 implemented a set of IPC messages for creating, > > closing and managing jsonr

Re: [ovs-dev] [ovsdb-server multithreading RFC 7/9] ovsdb: Add --max-num-threads commad line option

On Fri, Mar 18, 2016 at 6:31 PM, Ben Pfaff wrote: > On Thu, Mar 03, 2016 at 12:13:26AM -0800, Andy Zhou wrote: > > See ovsdb-server.1.in changes for details. > > > > Signed-off-by: Andy Zhou > > s/commad/command/ in the title. > > will fix. Thanks. > Thanks! > > The option name --max-threads i

Re: [ovs-dev] [PATCH v10 1/1] Separating OVN NB and SB database processes

Russell Bryant wrote on 03/18/2016 07:52:14 PM: > From: Russell Bryant > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: ovs dev , arnaldimich...@gmail.com > Date: 03/18/2016 07:52 PM > Subject: Re: [ovs-dev] [PATCH v10 1/1] Separating OVN NB and SB > database processes > > On Mon, Mar 7, 2016 at 8:15 AM,

Re: [ovs-dev] [ovsdb-server multithreading RFC 1/9] ovsdb: Do not group sessions by remote.

On Fri, Mar 18, 2016 at 5:15 PM, Ben Pfaff wrote: > On Thu, Mar 03, 2016 at 12:13:20AM -0800, Andy Zhou wrote: > > Currently ovsdb_jsonrpc_session are grouped together in a linked > > list within 'ovsdb_jsonrpc_remote'. This makes sense since most > > session operations applies to sessions withi

[ovs-dev] [PATCH v11 0/1] Separating OVN NB and SB database processes

From: RYAN D. MOATS Changes from V10: Rebased to fix merge conflict in ovn-sbctl.at Added macro to ovs-macros.at to allow controlling by --target arguement as this is needed for th ovn-sbctl tests. Addressed comments from Russell Bryant RYAN D. MOATS (1): Separating OVN NB and SB database pro

[ovs-dev] [PATCH v11 1/1] Separating OVN NB and SB database processes

From: RYAN D. MOATS OVN NB & SB DB's should be run in separate ovsdb-server processes and should run with ovn-ctl start_northd / stop_northd. This patch includes changes to unit tests, tutorial and debian scripts to remain self-consistent. Signed-off-by: RYAN D. MOATS Signed-off-by: Michael Ar

Re: [ovs-dev] [PATCH monitor_cond V5 00/18] Implement conditional monitoring

On Fri, Mar 04, 2016 at 08:08:55AM +, 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_update". Specification of this extension is defined in the > ovsdb-server (1) m

Re: [ovs-dev] [PATCH v8 00/10] Implement incremental processing in ovn-controller

"Huang, Lei" wrote on 03/20/2016 11:23:44 PM: > From: "Huang, Lei" > To: "dev@openvswitch.org" , Ryan Moats/Omaha/IBM@IBMUS > Date: 03/20/2016 11:39 PM > Subject: Re: [PATCH v8 00/10] Implement incremental processing in > ovn-controller > > Hi Ryan, > > I plan to test this patch in our scalabili

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On 21/03/16 21:07, Russell Bryant wrote: On Mon, Mar 21, 2016 at 12:59 PM, Jesse Gross mailto:je...@kernel.org>> wrote: On Mon, Mar 21, 2016 at 12:49 PM, Russell Bryant mailto:russ...@ovn.org>> wrote: > On Mon, Mar 21, 2016 at 12:43 PM, Jesse Gross mailto:je...@kernel.org>> wrote:

Re: [ovs-dev] [PATCH] datapath: Check for sock argument to v6ops->fragment.

On Monday, March 21, 2016, Jakub Libosvar wrote: > On 21/03/16 21:07, Russell Bryant wrote: > >> >> >> On Mon, Mar 21, 2016 at 12:59 PM, Jesse Gross > > wrote: >> >> On Mon, Mar 21, 2016 at 12:49 PM, Russell Bryant > > wrote: >> > On Mo

Re: [ovs-dev] [PATCH v3 2/3] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.

On Mon, Mar 21, 2016 at 11:23 AM, Russell Bryant wrote: > > > On Mon, Mar 21, 2016 at 11:16 AM, Guru Shetty wrote: > >> >> >> On 21 March 2016 at 07:54, Russell Bryant wrote: >> >>> Update the "ct_commit;" logical flow action to optionally take >>> one or two parameters, setting the value of "c

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

Hi Sorin/Alin, I looked at the patch and had a few comments. Pls. find them inlined. In general, I was a little confused about how we use the recirc ID and the hash. We can discuss more during the IRC meeting. Thanks, -- Nithin -Original Message- From: dev on behalf of Sorin Vinturis

Re: [ovs-dev] [PATCH v2] ovn: Update the test case for localnet port.

On Thu, Mar 17, 2016 at 1:55 AM, Han Zhou wrote: > Localnet port is now able to connect vif ports on different HVs. > Change the test case accordingly. > > Signed-off-by: Han Zhou Thanks! I applied this to master. -- Russell Bryant ___ dev mailing

Re: [ovs-dev] [PATCH] [Patch V2] OVN: Logical-Physical Separation and L2 SW Gateways

On 3/21/16, 9:41 AM, "dev on behalf of Russell Bryant" wrote: >On Mon, Mar 21, 2016 at 8:47 AM, Darrell Ball wrote: > >> The following patch series implements physical-logical separation >> to be used presently by gateways and localnet. >> The patch series also includes L2 SW Gateway suppo

Re: [ovs-dev] [PATCH] db-ctl-base: Fix a typo.

> On Mar 21, 2016, at 10:42 AM, Russell Bryant wrote: > > Signed-off-by: Russell Bryant Acked-by: Justin Pettit --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ofpbuf: Don't include unneeded "packets.h".

> On Mar 21, 2016, at 9:23 AM, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn: Remove outdated debug message.

> On Mar 21, 2016, at 8:10 AM, Russell Bryant wrote: > > ovn-northd used to only call ovnsb_db_run() and ovnnb_db_run() when > seqnos indicated that they had changed. It now always calls these > functions from the main loop. Remove this outdated log message that > indicates that we called this

Re: [ovs-dev] [PATCH] [Patch V2] OVN: Logical-Physical Separation and L2 SW Gateways

On Mon, Mar 21, 2016 at 6:07 PM, Darrell Ball wrote: > > > > > > On 3/21/16, 9:41 AM, "dev on behalf of Russell Bryant" < > dev-boun...@openvswitch.org on behalf of russ...@ovn.org> wrote: > > >On Mon, Mar 21, 2016 at 8:47 AM, Darrell Ball wrote: > > > > > > >On a related topic, I have a need fo

Re: [ovs-dev] [PATCH] ovn: Remove outdated debug message.

On Mon, Mar 21, 2016 at 6:22 PM, Justin Pettit wrote: > > > On Mar 21, 2016, at 8:10 AM, Russell Bryant wrote: > > > > ovn-northd used to only call ovnsb_db_run() and ovnnb_db_run() when > > seqnos indicated that they had changed. It now always calls these > > functions from the main loop. Rem

Re: [ovs-dev] [PATCH] db-ctl-base: Fix a typo.

On Mon, Mar 21, 2016 at 6:08 PM, Justin Pettit wrote: > > > On Mar 21, 2016, at 10:42 AM, Russell Bryant wrote: > > > > Signed-off-by: Russell Bryant > > Acked-by: Justin Pettit Thanks! I applied this to master. -- Russell Bryant ___ dev mailing

Re: [ovs-dev] Deletion of Indirect Groups on Ovs2.5

On Mon, Mar 21, 2016 at 10:44:19AM -0700, Jarno Rajahalme wrote: > > > On Mar 20, 2016, at 5:21 PM, Simon Horman > > wrote: > > > > On Fri, Mar 18, 2016 at 09:45:08AM -0700, Jarno Rajahalme wrote: > >> OpenFlow 1.5 spec says: “The group type need not be specified for the > >> delete request.”

[ovs-dev] [PATCH] ovn: Add hostname to Chassis.

We currently use the system-id from the Open_vSwitch schema on each host to populate the unique name field of a Chassis in OVN_Southbound. On most systems, this is a UUID. It would be very convenient to also have the hostname available as that will allow people to more quickly identify which host

[ovs-dev] GANJARAN TELEKOM

Telekom Malaysia Berhad Level 51, North Wing, Menara TM, Jalan Pantai Baru, 50672, Kuala Lumpur, WP Kuala Lumpur, Malaysia. Hadiah Bonus Tahun Baru. Pihak Telekom Malaysia @Program Kemenangan yang telah diadakan pada 9th Mac 2016 di mana alamat email anda yang disertakan beraama Tiket Kemenanga

[ovs-dev] Few observations on low performance of 64 byte packet size

Hi All, While sending 2G traffic of 64bytes size, I see a low performance for Phy-Phy setup using OVS with DPDK. I get around 1500/1500 throughput. Following are the Platform and setup details: NOTE: Used OVS 2.4 and DPDK2.0.0 1. Intel(R) Atom(TM) 2. I assigned 1core to vswitchd process 3. Boun

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

Acked-by: Nithin Raju This patch is part of a series though and cannot be committed till the previous ones in the series get committed. -Original Message- From: dev on behalf of Sorin Vinturis Date: Friday, March 18, 2016 at 7:58 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATC

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

How is key->dphash being used after it gets stored as part of the key. Shouldn¹t it be used for flow lookup? Is it purely for reporting purposes? Thanks, -- Nithin -Original Message- From: dev on behalf of Sorin Vinturis Date: Friday, March 18, 2016 at 7:58 AM To: "dev@openvswitch.org"

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

Acked-by: Nithin Raju I was wondering if this should be part of the recirc patch itself. -Original Message- From: dev on behalf of Sorin Vinturis Date: Friday, March 18, 2016 at 7:58 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v6 5/6] datapath-windows: Extract flow metadat

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

Acked-by: Nithin Raju Dependent on previous patches in the series getting checked in. -Original Message- From: dev on behalf of Sorin Vinturis Date: Friday, March 18, 2016 at 7:58 AM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v6 6/6] datapath-windows: Updated FAQ regarding Hy