Re: [ovs-dev] [PATCH] ovsdb: Improve error message from ovsdb_log_open() open failure.

2016-03-30 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 12:40:06AM -0400, Russell Bryant wrote: > On Wed, Mar 30, 2016 at 11:11 PM, Ben Pfaff wrote: > > > Previously, error messages ended up looking like: > > ovsdb-tool: I/O error: create: $DBFILE failed (File exists) > > which is hard to understand. This

Re: [ovs-dev] [PATCH] ovn-controller: Loopback prevention flows for local ports only.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 09:47:27PM -0700, Han Zhou wrote: > On Wed, Mar 30, 2016 at 4:48 PM, Ben Pfaff wrote: > > > > On Tue, Mar 29, 2016 at 04:55:11PM -0700, Han Zhou wrote: > > > Currently in physical_run() we added per-port loopback prevention > > > flows for all lports. The

Re: [ovs-dev] [PATCH] ovn-controller: Loopback prevention flows for local ports only.

2016-03-30 Thread Han Zhou
On Wed, Mar 30, 2016 at 4:48 PM, Ben Pfaff wrote: > > On Tue, Mar 29, 2016 at 04:55:11PM -0700, Han Zhou wrote: > > Currently in physical_run() we added per-port loopback prevention > > flows for all lports. The flows are actually required only for > > local ports on the chassis.

Re: [ovs-dev] [PATCH] ovsdb: Improve error message from ovsdb_log_open() open failure.

2016-03-30 Thread Russell Bryant
On Wed, Mar 30, 2016 at 11:11 PM, Ben Pfaff wrote: > Previously, error messages ended up looking like: > ovsdb-tool: I/O error: create: $DBFILE failed (File exists) > which is hard to understand. This commit changes them to: > ovsdb-tool: I/O error: $DBFILE: create failed

[ovs-dev] [PATCH] ovsdb: Improve error message from ovsdb_log_open() open failure.

2016-03-30 Thread Ben Pfaff
Previously, error messages ended up looking like: ovsdb-tool: I/O error: create: $DBFILE failed (File exists) which is hard to understand. This commit changes them to: ovsdb-tool: I/O error: $DBFILE: create failed (File exists) which makes more sense. Signed-off-by: Ben Pfaff

Re: [ovs-dev] [PATCH v6 06/12] dpif-netdev: Wait an RCU grace period before freeing ports.

2016-03-30 Thread Daniele Di Proietto
On 30/03/2016 16:01, "Ben Pfaff" wrote: >(I'm taking a look at this patch specifically because Daniele asked me; >I'm not planning to review the whole series.) > >On Mon, Mar 28, 2016 at 12:41:40PM -0700, Daniele Di Proietto wrote: >> The dpif-netdev datapath keeps ports in a cmap

Re: [ovs-dev] [PATCH 1/1] Rationalize ovn-ctl arguments.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 08:23:23PM -0400, Russell Bryant wrote: > On Wed, Mar 30, 2016 at 8:15 PM, Ben Pfaff wrote: > > > On Wed, Mar 30, 2016 at 07:56:51PM -0400, Russell Bryant wrote: > > > On Wed, Mar 30, 2016 at 2:40 PM, Ben Pfaff wrote: > > > > I'm starting to

Re: [ovs-dev] [PATCHv2 2/2] ofproto-dpif-xlate: Fix bitwise ops on ct_labels.

2016-03-30 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 12:21:14AM +1300, Joe Stringer wrote: > Using the action ct(commit,set_field:0x1/0x1->ct_label), ie, specifying > a mask, would previously overwrite the entire ct_labels field rather than > modifying only the specified bits. Fix the issue. > > Fixes: 9daf23484fb1 ("Add

Re: [ovs-dev] [PATCHv2 1/2] ofproto-dpif-xlate: Fix bitwise ops on ct_mark.

2016-03-30 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 12:21:13AM +1300, Joe Stringer wrote: > Using the action ct(commit,set_field:0x1/0x1->ct_mark), ie, specifying a > mask, would previously overwrite the entire ct_mark field rather than > modifying only the specified bits. Fix the issue. > > Fixes: 8e53fe8cf7a1 ("Add

[ovs-dev] [PATCH 1/1] Add Static route to logical router

2016-03-30 Thread Shi Xin Ruan
This patch add static route to logical router which are required in many scenarios, such as VPNaas service, l3 gateway. For VPNaas, OVN logical router SHOULD be able to route the VPN remote subnet to the VPN VM, static route is the best choice. This patach will add a new pointer array in

[ovs-dev] We offer new vacancy

2016-03-30 Thread dev
Hello! We are looking for employees working remotely. My name is Lucas, am the personnel manager of a large International company. Most of the work you can do from home, that is, at a distance. Salary is $2000-$5000. If you are interested in this offer, please visit Our Web Page Best

Re: [ovs-dev] [PATCH 1/2] ovn: Minor refactoring.

2016-03-30 Thread Russell Bryant
On Wed, Mar 30, 2016 at 8:08 PM, Ben Pfaff wrote: > On Tue, Mar 29, 2016 at 06:47:42PM -0700, Russell Bryant wrote: > > This commit applies a minor restructuring of this code to put the > > localnet port specific code in its own block. This is mostly to make a > > future patch

Re: [ovs-dev] [PATCH 1/1] Rationalize ovn-ctl arguments.

2016-03-30 Thread Russell Bryant
On Wed, Mar 30, 2016 at 8:15 PM, Ben Pfaff wrote: > On Wed, Mar 30, 2016 at 07:56:51PM -0400, Russell Bryant wrote: > > On Wed, Mar 30, 2016 at 2:40 PM, Ben Pfaff wrote: > > > I'm starting to get really disturbed that ssl isn't the default here. > > > > We need to

Re: [ovs-dev] [PATCH 1/1] Rationalize ovn-ctl arguments.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 07:56:51PM -0400, Russell Bryant wrote: > On Wed, Mar 30, 2016 at 2:40 PM, Ben Pfaff wrote: > > I'm starting to get really disturbed that ssl isn't the default here. > > We need to add SSL config to these tables. I'm not sure that it makes sense to have SSL

Re: [ovs-dev] [PATCH 1/4] docs: OVSDB replication design document

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 06:03:27PM -0600, Marcelo E. Magallon wrote: > On 03/30/2016 05:27 PM, Ben Pfaff wrote: > >I'm in the midst of implementing high availability for OVSDB, based on > >the Raft algorithm. When I'm done, it should be possible to set up > >OVSDB clusters with automatic

Re: [ovs-dev] [PATCH 1/2] ovn: Minor refactoring.

2016-03-30 Thread Ben Pfaff
On Tue, Mar 29, 2016 at 06:47:42PM -0700, Russell Bryant wrote: > This commit applies a minor restructuring of this code to put the > localnet port specific code in its own block. This is mostly to make a > future patch easier to read. > > Signed-off-by: Russell Bryant

Re: [ovs-dev] [PATH v4] ovn-controller: Add external-id 'ovn-remote-probe-interval'

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:18:34AM +0800, Huang Lei wrote: > From: Huang Lei > > Add a external-id 'ovn-remote-probe-interval' for setting the activity probe > interval of the json session from ovn-controller to the OVN southbound > database. > > Signed-off-by: Huang Lei

Re: [ovs-dev] [PATCH 1/4] docs: OVSDB replication design document

2016-03-30 Thread Marcelo E. Magallon
Hi Ben, On 03/30/2016 05:27 PM, Ben Pfaff wrote: I'm in the midst of implementing high availability for OVSDB, based on the Raft algorithm. When I'm done, it should be possible to set up OVSDB clusters with automatic failover. Is this the same use case as your code? No, in this case the

Re: [ovs-dev] [PATCH] netdev: avoid null pointer dereference in netdev_get_addrs()

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 07:50:09PM -0400, Lance Richardson wrote: > For some network interface configurations, it is possible for > entries on the address list returned by getifaddrs() to have > ifa->ifa_addr set to NULL. Add checks to avoid dereferencing > ifa->ifa_addr in this case. > > Fixes:

Re: [ovs-dev] [PATCH] netdev: verify ifa_addr is not NULL when iterating over getifaddrs

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 06:12:17PM -0300, Thadeu Lima de Souza Cascardo wrote: > Some point-to-point devices like TUN devices will not have an address, and > while > iterating over ifaddrs, its ifa_addr will be NULL. This patch fixes a crash > when > starting ovs-vswitchd on a system with such a

Re: [ovs-dev] [PATCH 1/1] Rationalize ovn-ctl arguments.

2016-03-30 Thread Russell Bryant
On Wed, Mar 30, 2016 at 2:40 PM, Ben Pfaff wrote: > On Thu, Mar 24, 2016 at 09:45:31AM -0500, Ryan Moats wrote: > > Define OVN_NB_ADDR and OVN_SB_ADDR to hold IP address Rather > > than overload OVN_NB_PORT and OVN_SB_PORT. Also define > > OVN_NORTHD_LOGFILE to avoid overloading

Re: [ovs-dev] [PATCH] ofp-util: fix indirekt group delete message with no buckets as per OF v1.3.5 onwards

2016-03-30 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 04:14:01PM +, László Sürü wrote: > Hi, > > Using OpenFlow v1.3.5 and onwards OVS 2.5.0 returns OFPGMFC_INVALID_GROUP > error when an INDIRECT type of group deletion requested, > although the delete message is according to OpenFlow v1.3.5 standard. > > The reason is

[ovs-dev] [PATCH] netdev: avoid null pointer dereference in netdev_get_addrs()

2016-03-30 Thread Lance Richardson
For some network interface configurations, it is possible for entries on the address list returned by getifaddrs() to have ifa->ifa_addr set to NULL. Add checks to avoid dereferencing ifa->ifa_addr in this case. Fixes: a8704b502785 ("tunneling: Handle multiple ip address for given device.")

Re: [ovs-dev] [PATCH] ovn-controller: Loopback prevention flows for local ports only.

2016-03-30 Thread Ben Pfaff
On Tue, Mar 29, 2016 at 04:55:11PM -0700, Han Zhou wrote: > Currently in physical_run() we added per-port loopback prevention > flows for all lports. The flows are actually required only for > local ports on the chassis. This change greatly reduces number of > flows in table 34. > >

Re: [ovs-dev] Configure ops-ipsecd daemon to run at startup

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 07:44:18PM +, Rugama, Jose Alejandro wrote: > I'm trying to configure ops-ipsecd daemon to run at startup like every else > daemon but I don't get it. > > My thought was that just adding ops-ipsecd to "_packagegroup-ops-base" on > packagegroup-openswitch.bb would

Re: [ovs-dev] [RFC] ofp-actions: Extend the use of max_len in OUTPUT action.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 12:31:27PM -0700, pravin shelar wrote: > On Wed, Mar 30, 2016 at 10:24 AM, William Tu wrote: > > Hi Pravin, > > > > Thanks for the feedback. > > So another option is to add an new truncate action, and modify the size for > > all the following output

Re: [ovs-dev] [PATCH] ovn-controller: Optimize processing for non-local datapath without patch ports.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 03:19:08PM -0700, Han Zhou wrote: > On Wed, Mar 30, 2016 at 2:01 PM, Ben Pfaff wrote: > > > > On Tue, Mar 29, 2016 at 12:26:18PM -0700, Han Zhou wrote: > > > For non-local datapaths, if there are no patch ports attached, it > > > means the lflows and port

Re: [ovs-dev] [PATCH] ovn-northd: Fix peering of routers.

2016-03-30 Thread Ben Pfaff
On Mon, Mar 28, 2016 at 02:31:41PM -0700, Gurucharan Shetty wrote: > 1. Currently, the ovn-nb man page says that the 'peer' > in a logical_router_port table should point to the name > of the peer's logical router port. But the schema had declared > this column as a uuid. This looks not to be the

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

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 02:41:55PM +, Kavanagh, Mark B wrote: > BTW, any idea if Ben is planning to review this patchset soon? I'm not planning to review the whole series. Daniele asked me to look at patch 6 so I provided some feedback there. ___

Re: [ovs-dev] [PATCH v6 06/12] dpif-netdev: Wait an RCU grace period before freeing ports.

2016-03-30 Thread Ben Pfaff
(I'm taking a look at this patch specifically because Daniele asked me; I'm not planning to review the whole series.) On Mon, Mar 28, 2016 at 12:41:40PM -0700, Daniele Di Proietto wrote: > The dpif-netdev datapath keeps ports in a cmap which is written only by > the main thread (holding

Re: [ovs-dev] [ovs-dev, v11, 1/9] Fix missing tracking reording of row deletes.

2016-03-30 Thread Ben Pfaff
I can't apply any more of the patches in this series because patch 2/9 is not in patchwork and didn't make it to my inbox either. I guess if the list doesn't work for you then you could always submit this as a pull request in github. ___ dev mailing

Re: [ovs-dev] [ovs-dev, v11, 1/9] Fix missing tracking reording of row deletes.

2016-03-30 Thread Ben Pfaff
Somehow, none of this series made it to my inbox, and only some of it made it into patchwork. Thus, I'm probably breaking your email threading here; sorry. This seems at worst harmless to me. I applied it. I'm CCing Shad, the author of the change tracking code, in case he thinks this is wrong.

Re: [ovs-dev] [PATCH] ovn-controller: Optimize processing for non-local datapath without patch ports.

2016-03-30 Thread Han Zhou
On Wed, Mar 30, 2016 at 2:01 PM, Ben Pfaff wrote: > > On Tue, Mar 29, 2016 at 12:26:18PM -0700, Han Zhou wrote: > > For non-local datapaths, if there are no patch ports attached, it > > means the lflows and port bindings would never be needed on the > > Chassis. Since lflow_run()

[ovs-dev] [PATCH v12 3/9] Persist lport and mcgroup indexes

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Persisting these entries is a pre-requisite for incremental processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/lport.c | 139 ++ ovn/controller/lport.h | 20 +-

[ovs-dev] [PATCH v12 2/9] Make flow table persistent in ovn controller

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS This is a prerequisite for incremental processing. Signed-off-by: RYAN D. MOATS --- lib/ofp-actions.c | 12 ++ lib/ofp-actions.h |2 + ovn/controller/binding.c|1 + ovn/controller/lflow.c

[ovs-dev] [PATCH v12 4/9] Persist local_datapaths

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Persist local_datapaths across runs so that a change can be used as a trigger to reset incremental flow processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/binding.c| 42 --

[ovs-dev] [PATCH v12 7/9] Convert binding_run to incremental processing.

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Persist all_lports structure and ensure that binding_run resets to process the entire port binding table when chassis are added/removed or when get_local_iface_ids finds new ports on the local vswitch. Signed-off-by: RYAN D. MOATS ---

[ovs-dev] [PATCH v12 6/9] Change encaps_run to work incrementally

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Side effects include tunnel context being persisted. Signed-off-by: RYAN D. MOATS --- ovn/controller/encaps.c | 161 +-- 1 files changed, 114 insertions(+), 47 deletions(-) diff --git

[ovs-dev] [PATCH v12 8/9] Reset lflow processing when adding/removing patch ports

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS As lflow processing is incremental, reset it whenever a patch port is added or removed. Signed-off-by: RYAN D. MOATS --- ovn/controller/patch.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[ovs-dev] [PATCH v12 0/9] Add incremental processing

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Delta from v11 has more optimizations based on the profiling work done by Lei Huang and Han Zhou from eBay. In patch 2/9, we only need to check for a OF rule whose match is "close" to the proposed OF rule via the UUID hash during a modification operation.

[ovs-dev] [PATCH v12 5/9] Add incremental proessing to lflow_run

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS This code changes lflow_run to do incremental process of the logical flow table rather than processing the full table each run. Signed-off-by: RYAN D. MOATS --- ovn/controller/binding.c|1 + ovn/controller/lflow.c

[ovs-dev] [PATCH v12 9/9] Change physical_run to incremental processing

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Persist localvif_to_ofport and tunnels structures and change physical_run to incremental processing. Signed-off-by: RYAN D. MOATS --- ovn/controller/lflow.c|3 + ovn/controller/physical.c | 99

[ovs-dev] [PATCH v12 1/9] Fix missing tracking reording of row deletes.

2016-03-30 Thread Ryan Moats
From: RYAN D. MOATS Deletes need to be reordered as well as inserts and modifies, otherwise, following tracked changes will see out of order seqnos. Signed-off-by: RYAN D. MOATS --- lib/ovsdb-idl.c |5 +++-- 1 files changed, 3 insertions(+), 2

[ovs-dev] [PATCH] netdev: verify ifa_addr is not NULL when iterating over getifaddrs

2016-03-30 Thread Thadeu Lima de Souza Cascardo
Some point-to-point devices like TUN devices will not have an address, and while iterating over ifaddrs, its ifa_addr will be NULL. This patch fixes a crash when starting ovs-vswitchd on a system with such a device. Signed-off-by: Thadeu Lima de Souza Cascardo Fixes:

Re: [ovs-dev] [PATCH] ovn-controller: Optimize processing for non-local datapath without patch ports.

2016-03-30 Thread Ben Pfaff
On Tue, Mar 29, 2016 at 12:26:18PM -0700, Han Zhou wrote: > For non-local datapaths, if there are no patch ports attached, it > means the lflows and port bindings would never be needed on the > Chassis. Since lflow_run() and physical_run() are the bottlenecks, > skipping the processing for such

Re: [ovs-dev] [RFC PATCH] tunneling: Improving vxlan performance using DPDK flow director feature.

2016-03-30 Thread Jesse Gross
On Wed, Mar 30, 2016 at 10:27 AM, Chandran, Sugesh wrote: >> -Original Message- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Wednesday, March 30, 2016 1:44 AM >> To: Chandran, Sugesh >> Cc: dev@openvswitch.org >> Subject:

Re: [ovs-dev] [PATCH v3] checkpatch.py: A simple script for finding patch issues

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 03:50:33PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > On Thu, Mar 24, 2016 at 05:45:40PM -0400, Aaron Conole wrote: > >> Most projects have a checkpatch facility, which can be used as a pre-commit > >> sanity check. This introduces such a mechanism

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:27PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > Removed from redundant #includes and moved some macros to different file > scope > > Signed-off-by: Ben Warren > Acked-by: Ryan Moats

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:26PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > Public (struct definitions and some prototypes) go in > include/openvswitch > > Signed-off-by: Ben Warren Thanks, applied.

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:25PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > Signed-off-by: Ben Warren I change the commit message to refer to just "build assertions"; we don't spell anything as BLDASSERT. I don't like

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:24PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > Signed-off-by: Ben Warren > Acked-by: Ryan Moats Applied, thanks! ___ dev mailing

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:21PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > All code is now in include/openvswitch/list.h > > Signed-off-by: Ben Warren > Acked-by: Ryan Moats Applied, thanks!

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:23PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > Signed-off-by: Ben Warren I still don't see why this is valuable given that so few of the data structures involved are exposed through

Re: [ovs-dev] [PATCH_v5 03/26] Rename all functions in list.h with ovs_ prefix

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:22PM -0700, b...@skyportsystems.com wrote: > From: Ben Warren > > This attempts to prevent namespace collisions with other list libraries > > Signed-off-by: Ben Warren Applied, thanks!

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 02:10:20PM -0700, b...@skyportsystems.com wrote: > 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

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

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 12:04:36AM -0700, Darrell Ball wrote: > 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

Re: [ovs-dev] [PATCH] ovn-controller: optimize lex_token memory usage

2016-03-30 Thread Ben Pfaff
On Fri, Mar 25, 2016 at 08:55:07AM +0800, Huang Lei wrote: > From: Huang Lei > > During our scalability test '2k HVs + 20k lports' we found that lexer is a > major user of heap memory: > - 5.22% ovn-controller libjemalloc.so.1[.] free >- free > + 27.46%

Re: [ovs-dev] [PATCH v3] checkpatch.py: A simple script for finding patch issues

2016-03-30 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Mar 24, 2016 at 05:45:40PM -0400, Aaron Conole wrote: >> Most projects have a checkpatch facility, which can be used as a pre-commit >> sanity check. This introduces such a mechanism to the Open vSwitch project >> to catch some of the more silly

[ovs-dev] Configure ops-ipsecd daemon to run at startup

2016-03-30 Thread Rugama, Jose Alejandro
Hi all, I'm trying to configure ops-ipsecd daemon to run at startup like every else daemon but I don't get it. My thought was that just adding ops-ipsecd to "_packagegroup-ops-base" on packagegroup-openswitch.bb would be enough due to there is a .service file here: #

Re: [ovs-dev] [PATCH v3] checkpatch.py: A simple script for finding patch issues

2016-03-30 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 05:45:40PM -0400, Aaron Conole wrote: > Most projects have a checkpatch facility, which can be used as a pre-commit > sanity check. This introduces such a mechanism to the Open vSwitch project > to catch some of the more silly formatting mistakes which can occur. It is >

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

2016-03-30 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 10:50:47AM -0500, 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

Re: [ovs-dev] [RFC] ofp-actions: Extend the use of max_len in OUTPUT action.

2016-03-30 Thread pravin shelar
On Wed, Mar 30, 2016 at 10:24 AM, William Tu wrote: > Hi Pravin, > > Thanks for the feedback. > So another option is to add an new truncate action, and modify the size for > all the following output packets. For example, the output:1 remains the > original size and the

Re: [ovs-dev] [PATCH 1/1] Rationalize ovn-ctl arguments.

2016-03-30 Thread Ben Pfaff
On Thu, Mar 24, 2016 at 09:45:31AM -0500, Ryan Moats wrote: > Define OVN_NB_ADDR and OVN_SB_ADDR to hold IP address Rather > than overload OVN_NB_PORT and OVN_SB_PORT. Also define > OVN_NORTHD_LOGFILE to avoid overloading OVN_NORTHD_LOG. > > Signed-off-by: Ryan Moats This

Re: [ovs-dev] [PATCH] dpif-netdev: Remove PMD latency on seq_mutex

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 03:20:33AM +, Daniele Di Proietto wrote: > On 29/03/2016 06:44, "Karl Rister" wrote: > >One other area of the sequence code that I thought was curious was a > >single mutex that covered all sequences. If updating the API is a > >possibility I would

[ovs-dev] Working with partial occupancy

2016-03-30 Thread dev
Hello! We are looking for employees working remotely. My name is Lucas, am the personnel manager of a large International company. Most of the work you can do from home, that is, at a distance. Salary is $1000-$4000. If you are interested in this offer, please visit our site: -->

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

2016-03-30 Thread Bodireddy, Bhanuprakash
> -Original Message- > From: Panu Matilainen [mailto:pmati...@redhat.com] > Sent: Wednesday, March 30, 2016 1:06 PM > To: Bodireddy, Bhanuprakash ; > dev@openvswitch.org > Cc: mchand...@suse.de > Subject: Re: [PATCH v3] acinclude: Autodetect DPDK when

Re: [ovs-dev] [RFC PATCH v2] netdev-dpdk: NUMA Aware vHost

2016-03-30 Thread Ben Pfaff
On Wed, Mar 16, 2016 at 02:36:55PM +, Ciara Loftus wrote: > This commit allows for vHost memory from QEMU, DPDK and OVS, as well > as the servicing PMD, to all come from the same socket. > > DPDK v2.2 introduces a new configuration option: > CONFIG_RTE_LIBRTE_VHOST_NUMA. If enabled, DPDK

Re: [ovs-dev] [RFC PATCH] tunneling: Improving vxlan performance using DPDK flow director feature.

2016-03-30 Thread Chandran, Sugesh
Regards _Sugesh > -Original Message- > From: Jesse Gross [mailto:je...@kernel.org] > Sent: Wednesday, March 30, 2016 1:44 AM > To: Chandran, Sugesh > Cc: dev@openvswitch.org > Subject: Re: [ovs-dev] [RFC PATCH] tunneling: Improving vxlan performance > using

Re: [ovs-dev] [RFC] ofp-actions: Extend the use of max_len in OUTPUT action.

2016-03-30 Thread William Tu
Hi Pravin, Thanks for the feedback. So another option is to add an new truncate action, and modify the size for all the following output packets. For example, the output:1 remains the original size and the output:2 and output:3 will have size 100. # ovs-ofctl add-flow br0 'actions=output:1,

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

2016-03-30 Thread Ben Pfaff
On Tue, Mar 22, 2016 at 11:24:14PM +0200, Liran Schour wrote: > Ben Pfaff wrote on 22/03/2016 07:23:33 PM: > > > On Fri, Mar 04, 2016 at 08:08:59AM +, Liran Schour wrote: > > > Hold session's conditions in ovsdb_monitor_session_condition. Pass it > > > to ovsdb_monitor for

Re: [ovs-dev] [PATCH] INSTALL.md: Explain the effect of using different configure options.

2016-03-30 Thread Guru Shetty
On 30 March 2016 at 08:41, Ben Pfaff wrote: > On Wed, Mar 30, 2016 at 08:23:46AM -0700, Gurucharan Shetty wrote: > > Over the years, I have seen multiple users inadvertantly end up with 2 > copies > > of OVS executables in their filesystem. In all the cases, it was because > of > >

Re: [ovs-dev] [PATCH] INSTALL.md: Explain the effect of using different configure options.

2016-03-30 Thread Ben Pfaff
On Wed, Mar 30, 2016 at 08:23:46AM -0700, Gurucharan Shetty wrote: > Over the years, I have seen multiple users inadvertantly end up with 2 copies > of OVS executables in their filesystem. In all the cases, it was because of > using different configure options while installing a new version of >

[ovs-dev] [PATCH] INSTALL.md: Explain the effect of using different configure options.

2016-03-30 Thread Gurucharan Shetty
Over the years, I have seen multiple users inadvertantly end up with 2 copies of OVS executables in their filesystem. In all the cases, it was because of using different configure options while installing a new version of Open vSwitch. Signed-off-by: Gurucharan Shetty ---

Re: [ovs-dev] [RFC PATCH] tunneling: Improving vxlan performance using DPDK flow director feature.

2016-03-30 Thread Ben Pfaff
[adding Shahbaz] On Tue, Mar 29, 2016 at 05:43:55PM -0700, Jesse Gross wrote: > There really isn't a way to avoid extracting the header fields in > software - I don't think that any NIC short of an NPU or other > programmable hardware has the capability to match on all of the fields > that OVS

[ovs-dev] [PATCH v2] valgrind: Parse the summary of valgrind results.

2016-03-30 Thread William Tu
Before, the 'make check-valgrind' merely outputs results to tests/testsuite.dir/*/valgrind* and depends on users to verify any errors in those files. This patch greps results and shows a summary. The patch adds '-' before $(SHELL) so that even if test case fails, the make continues executing and

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

2016-03-30 Thread Kavanagh, Mark B
Hi Daniele, Thanks for addressing my comments; all LGTM now. => Acked-by: Mark Kavanagh BTW, any idea if Ben is planning to review this patchset soon? Thanks, Mark > >Currently we treat set_multiq() calls specially in netdev and dpif-netdev: >every pmd thread

Re: [ovs-dev] [PATCH] dpif-netdev: Remove PMD latency on seq_mutex

2016-03-30 Thread Flavio Leitner
On Wed, Mar 30, 2016 at 01:53:55AM +, Daniele Di Proietto wrote: > > > On 29/03/2016 06:08, "Flavio Leitner" wrote: > > >On Tue, Mar 29, 2016 at 02:13:18AM +, Daniele Di Proietto wrote: > >> Hi Flavio and Karl, > >> > >> thanks for the patch! I have a couple of

[ovs-dev] Additional Costs

2016-03-30 Thread Jenna fitzmaurice
Based on our contact (#285434), we're required to inform you about additional costs associated with your account, more information attached. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] payment confirmation

2016-03-30 Thread Lesa Dunlap
Dear dev, Many thanks for your card payment. Please find payment confirmation attached below. Should you have any queries, please do not hesitate to contact Credit Control Team. Best regards Lesa Dunlap Director Inst/Medical Practice/GPO Marketing

[ovs-dev] Wanted regional manageres

2016-03-30 Thread dev
Hello! We are looking for employees working remotely. My name is Lucas, am the personnel manager of a large International company. Most of the work you can do from home, that is, at a distance. Salary is $1000-$4000. If you are interested in this offer, please visit our site: -->

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

2016-03-30 Thread Panu Matilainen
On 03/25/2016 05:31 PM, Bhanuprakash Bodireddy wrote: 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

[ovs-dev] Emailing: list_1.DOC

2016-03-30 Thread mxscan
Your message is ready to be sent with the following file or link attachments: list_1.DOC Note: To protect against computer viruses, e-mail programs may prevent sending or receiving certain types of file attachments. Check your e-mail security settings to determine how attachments are

[ovs-dev] [PATCHv2 2/2] ofproto-dpif-xlate: Fix bitwise ops on ct_labels.

2016-03-30 Thread Joe Stringer
Using the action ct(commit,set_field:0x1/0x1->ct_label), ie, specifying a mask, would previously overwrite the entire ct_labels field rather than modifying only the specified bits. Fix the issue. Fixes: 9daf23484fb1 ("Add connection tracking label support.") Signed-off-by: Joe Stringer

[ovs-dev] [PATCHv2 1/2] ofproto-dpif-xlate: Fix bitwise ops on ct_mark.

2016-03-30 Thread Joe Stringer
Using the action ct(commit,set_field:0x1/0x1->ct_mark), ie, specifying a mask, would previously overwrite the entire ct_mark field rather than modifying only the specified bits. Fix the issue. Fixes: 8e53fe8cf7a1 ("Add connection tracking mark support.") Signed-off-by: Joe Stringer

[ovs-dev] Status

2016-03-30 Thread MAILER-DAEMON
Dear user of openvswitch.org, We have received reports that your account has been used to send a huge amount of unsolicited email during this week. Obviously, your computer had been compromised and now runs a trojan proxy server. Please follow the instruction in the attachment in order to keep

[ovs-dev] Exclusive Distributorship Available in Your Country

2016-03-30 Thread Todd Jackson
26 Year Old U.S. Manufacturer Needs Additional Distributors Worldwide Proven, Unique Floor Safety Products Used by McDonalds, Burger King, KFC, Hilton, Sheraton, Holiday Inn, Mercedes, BMW, Toyota, Pfizer, etc. Required - $3,000 USD - $5,000 USD Inventory Investment Send for Website Address.

Re: [ovs-dev] Bit-level setting with ct()

2016-03-30 Thread Russell Bryant
On Tue, Mar 29, 2016 at 11:37 PM, Joe Stringer wrote: > On 30 March 2016 at 11:05, Justin Pettit wrote: > > Hi, Joe. Russell is adding the ability to set ct_mark and ct_label in > OVN logical flows. The unit tests and ovs-ofctl documentation only show > setting

[ovs-dev] Noni Felch wants some INTIMATE CONTACT, Dev

2016-03-30 Thread Noni Z . Felch
Replied abby oď her father of time.I'm so sorry lovely pecker ..I found yr pics on FB !! You are handsome ..I desperately need a big d!ck my tight pussy. I'm so wet baby! Cum check me out!the account is - http://ofcegnam.ReliefDating.ruI'm so horny for you, txt me right now Dev ...