Re: [ovs-dev] About a race condition between handle_upcalls and revalidate

2017-03-20 Thread Joe Stringer
On 19 March 2017 at 08:44, Paul Blakey wrote: > > > On 19/03/2017 17:10, Paul Blakey wrote: >>> >>> Hi all, >>> >>> While using out patches for HW offload we've noticed we get a ovs >>> assertion at transition ukey, which tries to >>> transition the ukey state from EVICTED

Re: [ovs-dev] [patch_v1] ovs build: Fix memset with zero size warning.

2017-03-20 Thread Eric Garver
On Mon, Mar 20, 2017 at 06:39:15PM +, Darrell Ball wrote: > > > On 3/20/17, 8:27 AM, "ovs-dev-boun...@openvswitch.org on behalf of Eric > Garver" wrote: > > On Sun, Mar 19, 2017 at 10:11:09AM -0700, Darrell Ball wrote: > >

Re: [ovs-dev] [PATCH] ovsdb-server: Drop unnecessary find_db() function.

2017-03-20 Thread Andy Zhou
On Mon, Mar 20, 2017 at 10:50 AM, Ben Pfaff wrote: > 'all_dbs' maps from a schema name to its struct db, so there's no need to > iterate the whole thing to find a database by schema name; instead, just > use the shash in the usual way. > > Also, avoid trying too hard in

[ovs-dev] [PATCH] ofproto-dpif-upcall: Fix flow setup/delete race.

2017-03-20 Thread Joe Stringer
If a handler thread takes a long time to set up a set of flows, it is possible for one of the installed flows to be dumped and scheduled for deletion by a revalidator thread before the handler is able to transition the ukey into an operational state---Between the dpif_operate() above this function

[ovs-dev] sphinx on os x on travis?

2017-03-20 Thread Ilya Maximets
Hi Ben and Lance, > Based on the above, I think the issue is that sphinx 1.1 isn't actually > sufficient > to build the documentation. That is not true. I made some investigation and found that root cause of this issue is the version of python-docutils library. Described issue is a know bug

Re: [ovs-dev] [PATCH] ovsdb-server: Fix memory leak in update_remote_status() error path.

2017-03-20 Thread Andy Zhou
On Sun, Mar 19, 2017 at 10:22 AM, Ben Pfaff wrote: > Found by inspection. > > Signed-off-by: Ben Pfaff Acked-by: Andy Zhou ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [patch_v2] build: Fix memset with zero size warning.

2017-03-20 Thread Darrell Ball
On 3/20/17, 12:32 PM, "ovs-dev-boun...@openvswitch.org on behalf of Eric Garver" wrote: On Mon, Mar 20, 2017 at 11:24:49AM -0700, Darrell Ball wrote: > In file included from /usr/include/string.h:640:0, >

Re: [ovs-dev] [PATCH] tests: Add and improve stp tests.

2017-03-20 Thread Joe Stringer
On 20 March 2017 at 14:21, Ben Pfaff wrote: > On Mon, Mar 20, 2017 at 02:15:24PM -0700, Joe Stringer wrote: >> On 9 March 2017 at 03:49, nickcooper-zhangtonghao >> wrote: >> > +ovs-appctl time/warp 1000 >> > +AT_CHECK([ovs-appctl netdev-dummy/receive br0 \ >>

Re: [ovs-dev] [PATCH] tests: Add and improve stp tests.

2017-03-20 Thread Joe Stringer
On 9 March 2017 at 03:49, nickcooper-zhangtonghao wrote: > This patch makes the stp tests more stable and > adds the test for fdb/mdb flushing when topology > is changed. > > Signed-off-by: nickcooper-zhangtonghao > --- > tests/stp.at | 163 >

[ovs-dev] Conference Invitation, Bulgaria (EU) 2017

2017-03-20 Thread International Scientific Events, Bulgaria
Conference Invitation, Bulgaria (EU) 2017 We have the pleasure of inviting you to participate in conferences, part of International Scientific Events 2017, to be held in Hotel "Royal Castle" *, Elenite Holiday Village on the Bulgarian Black Sea Coast (EU). Conferences in June Agriculture

Re: [ovs-dev] [patch_v2] build: Fix memset with zero size warning.

2017-03-20 Thread Eric Garver
On Mon, Mar 20, 2017 at 11:24:49AM -0700, Darrell Ball wrote: > In file included from /usr/include/string.h:640:0, > from ./lib/string.h:20, > from /usr/include/netinet/icmp6.h:22, > from ../lib/flow.h:21, > from

Re: [ovs-dev] [PATCH] tests: Add and improve stp tests.

2017-03-20 Thread nickcooper-zhangtonghao
I tested it via command as below for a long time. while true; do make check TESTSUITEFLAGS='-k stp'; done But it’s ok. I try to find what might cause this. Thanks. Nick > On Mar 21, 2017, at 5:24 AM, Joe Stringer wrote: > > Right, Travis seems happy too so most likely it's

Re: [ovs-dev] [PATCH v2] netdev-dpdk: fix ARM cross compilation failure

2017-03-20 Thread Jianbo Liu
The 03/20/2017 16:43, Hemant Agrawal wrote: > configure with_dpdk and cross compile fails with: > "configure: error: cannot check for file existence > when cross compiling" > > This is due to the usages of AC_CHECK_FILES. > > AC_CHECK_FILES only works when not cross compiling. It test > a feature

Re: [ovs-dev] [PATCH] netdev-dpdk: leverage the mempool offload support

2017-03-20 Thread Jianbo Liu
The 03/20/2017 17:29, Hemant Agrawal wrote: > DPDK 16.07 introduced the support for mempool offload support. > rte_pktmbuf_pool_create is the recommended method for creating pktmbuf > pools. Buffer pools created with rte_mempool_create may not get offloaded > to the underlying offloaded mempools.

[ovs-dev] [PATCH] travis: Break Mac OS build for format specifier warnings.

2017-03-20 Thread Ben Pfaff
Until now, the Travis build for Mac OS X has been configured to ignore format specifier warnings. These warnings have now been fixed, so this commit changes such warnings to error. Suggested-by: Daniele Di Proietto Signed-off-by: Ben Pfaff ---

Re: [ovs-dev] [PATCH 1/2] ofproto: Log when learn limit reached.

2017-03-20 Thread Joe Stringer
On 17 March 2017 at 16:04, Ben Pfaff wrote: > On Fri, Mar 17, 2017 at 11:38:34AM -0700, Joe Stringer wrote: >> This commit provides more visibility into conditions where learn limits >> are reached when the functionality from patch 4c71600d2256 >> ("ofp-actions: Add limit to learn

Re: [ovs-dev] [PATCH 2/2] ofproto: Clean up style in ofproto_flow_mod_learn.

2017-03-20 Thread Joe Stringer
On 17 March 2017 at 16:06, Ben Pfaff wrote: > On Fri, Mar 17, 2017 at 11:38:35AM -0700, Joe Stringer wrote: >> It's slightly more cognitive load to read an inverse condition and then >> invert it again to understand the 'else' condition. >> >> Signed-off-by: Joe Stringer

[ovs-dev] [branch-2.6] netdev-dpdk: leverage the mempool offload support

2017-03-20 Thread Hemant Agrawal
DPDK 16.07 introduced the support for mempool offload support. rte_pktmbuf_pool_create is the recommended method for creating pktmbuf pools. Buffer pools created with rte_mempool_create may not get offloaded to the underlying offloaded mempools. This patch, changes the rte_mempool_create to use

Re: [ovs-dev] About a race condition between handle_upcalls and revalidate

2017-03-20 Thread Paul Blakey
On 19/03/2017 17:10, Paul Blakey wrote: On 19/03/2017 16:28, Paul Blakey wrote: Hi all, While using out patches for HW offload we've noticed we get a ovs assertion at transition ukey, which tries to transition the ukey state from EVICTED back to OPERATIONAL. With furthur investigation it

[ovs-dev] [PATCH v2] netdev-dpdk: fix ARM cross compilation failure

2017-03-20 Thread Hemant Agrawal
configure with_dpdk and cross compile fails with: "configure: error: cannot check for file existence when cross compiling" This is due to the usages of AC_CHECK_FILES. AC_CHECK_FILES only works when not cross compiling. It test a feature of the host machine, and therefore, die when

Re: [ovs-dev] [PATCH ovs V4 00/24] Introducing HW offload support for openvswitch

2017-03-20 Thread Simon Horman
On Mon, Mar 20, 2017 at 10:56:34AM +0200, Roi Dayan wrote: > > > On 20/03/2017 10:33, Roi Dayan wrote: > > > > > >On 16/03/2017 17:40, Simon Horman wrote: > >>On Mon, Mar 13, 2017 at 03:36:50PM +0200, Roi Dayan wrote: > >>>This patch series introduces rule offload functionality to dpif-netlink >

[ovs-dev] [PATCH] netdev-dpdk: leverage the mempool offload support

2017-03-20 Thread Hemant Agrawal
DPDK 16.07 introduced the support for mempool offload support. rte_pktmbuf_pool_create is the recommended method for creating pktmbuf pools. Buffer pools created with rte_mempool_create may not get offloaded to the underlying offloaded mempools. This patch, changes the rte_mempool_create to use

Re: [ovs-dev] sphinx on os x on travis?

2017-03-20 Thread Lance Richardson
> From: "Ilya Maximets" > To: ovs-dev@openvswitch.org, "Lance Richardson" , "Ben > Pfaff" > Sent: Monday, March 20, 2017 2:54:55 AM > Subject: [ovs-dev] sphinx on os x on travis? > > Hi Ben and Lance, > > > Based on the above, I think

Re: [ovs-dev] [PATCH ovs V4 00/24] Introducing HW offload support for openvswitch

2017-03-20 Thread Or Gerlitz
On 3/17/2017 4:14 PM, Simon Horman wrote: Hi again, I think that once the TODO items above are resolved - both of which I have provided some feedback on separately - some consideration should be given to merging this patchset. Any thoughts/comments on that? Roi just commented that he will

Re: [ovs-dev] [PATCH] netdev-dpdk: Enable INDIRECT_DESC on DPDK vHostUser.

2017-03-20 Thread O Mahony, Billy
Hi Maxime, > -Original Message- > From: Maxime Coquelin [mailto:maxime.coque...@redhat.com] > Sent: Friday, March 17, 2017 9:48 AM > To: O Mahony, Billy ; d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] netdev-dpdk: Enable INDIRECT_DESC on > DPDK vHostUser.

Re: [ovs-dev] [PATCH ovs V4 00/24] Introducing HW offload support for openvswitch

2017-03-20 Thread Roi Dayan
On 16/03/2017 17:40, Simon Horman wrote: On Mon, Mar 13, 2017 at 03:36:50PM +0200, Roi Dayan wrote: This patch series introduces rule offload functionality to dpif-netlink via netdev ports new flow offloading API. The user can specify whether to enable rule offloading or not via OVS

Re: [ovs-dev] [PATCH] CONTRIBUTING.rst: Fix links.

2017-03-20 Thread Stephen Finucane
On Fri, 2017-03-17 at 11:27 -0700, Joe Stringer wrote: > Signed-off-by: Joe Stringer Acked-by: Stephen Finucane ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] Earn Extra Monthly by working from your Mobile device.

2017-03-20 Thread Coordenação-Geral de Identificação e Delimitação - CGID
Earn $3000 Monthly by working from your Mobile device. For more inquiry,EMAIL: SUNFLAG IRON & STEEL CO. LTD 970 285 4210 sunfla...@gmail.com www.sunflagsteel.com

[ovs-dev] Salut,

2017-03-20 Thread Véronique DESCHAMPS via dev
Salut, Excusez moi pour le dérangement, je voudrais faire votre connaissance et lier une amitié sincère avec vous, prière de me répondre. Bisous, Véronique DESCHAMPS ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH] netdev-dpdk: Enable INDIRECT_DESC on DPDK vHostUser.

2017-03-20 Thread O Mahony, Billy
Hi Kevin, > -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Thursday, March 16, 2017 6:35 PM > To: O Mahony, Billy ; d...@openvswitch.org > Cc: Loftus, Ciara ; Maxime Coquelin > >

[ovs-dev] [PATCH] ovsdb-server: Drop unnecessary find_db() function.

2017-03-20 Thread Ben Pfaff
'all_dbs' maps from a schema name to its struct db, so there's no need to iterate the whole thing to find a database by schema name; instead, just use the shash in the usual way. Also, avoid trying too hard in ovsdb_replication_init() in a similar way. Signed-off-by: Ben Pfaff ---

Re: [ovs-dev] How do we set fields for selection_method=hash?

2017-03-20 Thread Yang, Yi Y
Thanks Simon, I know how I can use it correctly from tests/ofproto.at, but it requires OpenFlow 1.5. I tried a new field I added, it is ok. This is really a good feature:-) -Original Message- From: Simon Horman [mailto:simon.hor...@netronome.com] Sent: Monday, March 20, 2017 9:28 PM

Re: [ovs-dev] How do we set fields for selection_method=hash?

2017-03-20 Thread Simon Horman
Thanks, I was pretty pleased with the design :) On Mon, Mar 20, 2017 at 01:36:27PM +, Yang, Yi Y wrote: > Thanks Simon, I know how I can use it correctly from tests/ofproto.at, but it > requires OpenFlow 1.5. I tried a new field I added, it is ok. This is really > a good feature:-) > >

Re: [ovs-dev] About a race condition between handle_upcalls and revalidate

2017-03-20 Thread Paul Blakey
On 19/03/2017 16:28, Paul Blakey wrote: Hi all, While using out patches for HW offload we've noticed we get a ovs assertion at transition ukey, which tries to transition the ukey state from EVICTED back to OPERATIONAL. With furthur investigation it seem that this can happen without our HW

Re: [ovs-dev] [PATCH] Documentation: Fix the ovs-ifup and ovs-ifdown examples in kvm.rst

2017-03-20 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 05:14:49PM +0100, Timothy Redaelli wrote: > Signed-off-by: Timothy Redaelli Thanks, applied to master and branch-2.7. Do these scripts need to be marked executable? If so, should we add a command to do that? Thanks, Ben.

Re: [ovs-dev] sphinx on os x on travis?

2017-03-20 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 09:54:55AM +0300, Ilya Maximets wrote: > Hi Ben and Lance, > > > Based on the above, I think the issue is that sphinx 1.1 isn't actually > > sufficient > > to build the documentation. > > That is not true. > > I made some investigation and found that root cause of this

Re: [ovs-dev] How do we set fields for selection_method=hash?

2017-03-20 Thread Simon Horman
On Tue, Mar 14, 2017 at 06:32:30AM +, Yang, Yi Y wrote: > Hi, Simon > > I notice you Netronome implemented selection_method support in ovs, now I > just want to use the specified match fields as hash input, I checked > ovs-ofctl man pages, for such use case, I must use "selection_method=hash

[ovs-dev] End of quarter offer.

2017-03-20 Thread Dora Delgado
Hello, Get fresh and accurate data for your marketing campaign. We have recently updated our Microsoft Azure users list and have added new contacts to our master file. We can provide you contacts across the globe and we can filter the contacts by industry and other filters. We provide

Re: [ovs-dev] [PATCH] compat: Remove rpl_dev_queue_xmit() backport.

2017-03-20 Thread Simon Horman
On Thu, Mar 02, 2017 at 03:00:37PM -0800, Joe Stringer wrote: > On 8 February 2017 at 16:50, Joe Stringer wrote: > > The MPLS portions of this were inadvertently broken in v2.4 due to > > 433637881ca5 ("datapath: define compat __skb_gso_segment()") which > > inverts the

Re: [ovs-dev] [patch_v1] ovs build: Fix memset with zero size warning.

2017-03-20 Thread Eric Garver
On Sun, Mar 19, 2017 at 10:11:09AM -0700, Darrell Ball wrote: > In file included from /usr/include/string.h:640:0, > from ./lib/string.h:20, > from /usr/include/netinet/icmp6.h:22, > from ../lib/flow.h:21, > from ../lib/flow.c:18:

Re: [ovs-dev] [PATCH] build: Only re-gen HTML docs when needed.

2017-03-20 Thread Russell Bryant
On Thu, Mar 16, 2017 at 4:58 PM, Ben Pfaff wrote: > On Thu, Mar 16, 2017 at 01:57:20PM -0700, Ben Pfaff wrote: >> On Thu, Mar 16, 2017 at 01:01:32PM -0400, Russell Bryant wrote: >> > When sphinx-build is installed, the docs were being re-generated during >> > every invocation of

Re: [ovs-dev] [PATCH 2/2] stp: Add the 'ovs-appctl stp/show' command.

2017-03-20 Thread Ben Pfaff
On Sat, Mar 18, 2017 at 01:13:56AM -0700, nickcooper-zhangtonghao wrote: > The stp/show command will help users and developers to > get more details about stp. This patch works together with > the previous patch "stp: Change the api for next patch." > > Signed-off-by: nickcooper-zhangtonghao

Re: [ovs-dev] sphinx on os x on travis?

2017-03-20 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 09:21:21AM -0700, Ben Pfaff wrote: > On Mon, Mar 20, 2017 at 09:54:55AM +0300, Ilya Maximets wrote: > > Hi Ben and Lance, > > > > > Based on the above, I think the issue is that sphinx 1.1 isn't actually > > > sufficient > > > to build the documentation. > > > > That is

[ovs-dev] [PATCH] travis: Fix build on Travis by installing new-enough docutils.

2017-03-20 Thread Ben Pfaff
This yields a clean build: https://travis-ci.org/blp/ovs-reviews/builds/213085784 Suggested-by: Ilya Maximets Signed-off-by: Ben Pfaff --- .travis/linux-prepare.sh | 1 + .travis/osx-prepare.sh | 1 + 2 files changed, 2 insertions(+) diff --git

[ovs-dev] [patch_v2] build: Fix memset with zero size warning.

2017-03-20 Thread Darrell Ball
In file included from /usr/include/string.h:640:0, from ./lib/string.h:20, from /usr/include/netinet/icmp6.h:22, from ../lib/flow.h:21, from ../lib/flow.c:18: In function 'memset', inlined from 'flow_push_vlan_uninit' at

Re: [ovs-dev] sphinx on os x on travis?

2017-03-20 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 10:39:16AM -0700, Ben Pfaff wrote: > On Mon, Mar 20, 2017 at 09:21:21AM -0700, Ben Pfaff wrote: > > On Mon, Mar 20, 2017 at 09:54:55AM +0300, Ilya Maximets wrote: > > > Hi Ben and Lance, > > > > > > > Based on the above, I think the issue is that sphinx 1.1 isn't actually

Re: [ovs-dev] [PATCH] travis: Fix build on Travis by installing new-enough docutils.

2017-03-20 Thread Lance Richardson
> From: "Ben Pfaff" > To: d...@openvswitch.org > Cc: "Ilya Maximets" , "Ben Pfaff" > Sent: Monday, March 20, 2017 2:31:22 PM > Subject: [ovs-dev] [PATCH] travis: Fix build on Travis by installing > new-enough docutils. > > This yields a

Re: [ovs-dev] [PATCH] travis: Fix build on Travis by installing new-enough docutils.

2017-03-20 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 02:38:08PM -0400, Lance Richardson wrote: > > From: "Ben Pfaff" > > To: d...@openvswitch.org > > Cc: "Ilya Maximets" , "Ben Pfaff" > > Sent: Monday, March 20, 2017 2:31:22 PM > > Subject: [ovs-dev] [PATCH] travis: Fix