Re: [ovs-dev] [patch_v6 4/8] dpdk: Userspace Datapath: Introduce NAT Support.

On 2/21/17, 1:12 PM, "ovs-dev-boun...@openvswitch.org on behalf of Flavio Leitner" wrote: On Thu, Feb 16, 2017 at 12:47:35AM -0800, Darrell Ball wrote: > This patch introduces NAT support for the userspace datapath. >

Re: [ovs-dev] [BUG] upcall handler thread crash

Again, thanks for reporting this bug! I just posted a more complete patch to fix this issue to the ova-dev list. Please have a look. Jarno > On Feb 7, 2017, at 4:36 AM, wangyunjian wrote: > > I have tested patch without issue. Will you submit it as an official

[ovs-dev] [PATCH] mirror: Allow concurrent lookups.

Handler threads use a selection of mirror functions with the assumption that the data referred to is RCU protected, while the implementation has not provided for this, which can lead to an OVS crash. This patch fixes this by making the mbundle lookup RCU-safe by using cmap instead of hmap and

[ovs-dev] [branch-2.5 2/2] Prepare for 2.5.3.

Signed-off-by: Justin Pettit --- NEWS | 4 configure.ac | 2 +- debian/changelog | 7 +++ 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 7c697db..6c1d998 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,7 @@ +v2.5.3 - xx xxx

[ovs-dev] [branch-2.5 1/2] Set release date for 2.5.2.

Signed-off-by: Justin Pettit --- NEWS | 3 ++- debian/changelog | 9 +++-- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/NEWS b/NEWS index 27c15af..7c697db 100644 --- a/NEWS +++ b/NEWS @@ -1,10 +1,11 @@ -v2.5.2 - xx xxx +v2.5.2 - 21 Feb 2017

[ovs-dev] [PATCH] ovs-ctl: allow passing user:group to daemons

The Open vSwitch daemons allow passing --user user[:group] to allow spawning under different user privileges. ovs-ctl now accepts --ovs-user in the same form to pass this argument on, as well as create databases and data directories with the appropriate privileges. Signed-off-by: Aaron Conole

[ovs-dev] [branch-2.7] Set release date for 2.7.0.

Signed-off-by: Justin Pettit --- NEWS | 2 +- debian/changelog | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 89bb026..4ec96de 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -v2.7.0 - xx xxx +v2.7.0 - 21 Feb 2017

Re: [ovs-dev] [PATCH] msi: add ovs-vswitchd trigger info This patch changes the service ovs-vswitchd from "auto" execution to "demand" start.

> -Original Message- > From: Alin Serdean > Sent: Friday, February 17, 2017 1:26 PM > To: d...@openvswitch.org > Cc: Alin Serdean > Subject: [PATCH] msi: add ovs-vswitchd trigger info This patch changes the > service ovs-vswitchd from "auto" execution to

[ovs-dev] [PATCH v2] msi: add ovs-vswitchd trigger info

From: Alin Serdean This patch changes the service ovs-vswitchd from "auto" execution to "demand" start. This patch also introduces a custom action for the ovs-vswitchd service in which the following command will be executed before the service startup: sc

Re: [ovs-dev] [PATCH 1/3] ofp-msgs: Remove unnecessary #include.

On 21 February 2017 at 11:37, Joe Stringer wrote: > On 17 February 2017 at 17:47, Yi-Hung Wei wrote: >> Remove unnecessary #include in ofp-msgs.h so that make will not complain. >> >> Signed-off-by: Yi-Hung Wei >> --- > > Not sure about

Re: [ovs-dev] [PATCH] doc: Describe backporting process.

On 16 February 2017 at 04:36, Stephen Finucane wrote: > On Wed, 2017-02-15 at 15:05 -0800, Joe Stringer wrote: >> This patch documents the backporting process, and provides a >> walkthrough >> for developers who would like to backport upstream Linux patches into >> the Open

Re: [ovs-dev] [PATCH] docs: fix typo in testing.rst

On Mon, Feb 20, 2017 at 01:12:30PM -0500, Lance Richardson wrote: > Coverage section refers to "check-lcoc" target, should be > "check-lcov". > > Signed-off-by: Lance Richardson > --- Thanks for pointing me to that doc :-) Acked-by: Flavio Leitner

Re: [ovs-dev] [patch_v6 2/8] dpdk: Parse NAT netlink for userspace datapath.

On Thu, Feb 16, 2017 at 12:47:33AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball > --- Looks good to me. Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [patch_v6 8/8] dpdk: Update feature alert documentation

On Thu, Feb 16, 2017 at 12:47:39AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball > --- Thanks for fixing it! Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH 3/3] Makefile.am: Avoid including internal library in public header files.

On 17 February 2017 at 17:47, Yi-Hung Wei wrote: > Add a build check that public openvswitch header file should not include > internal library. > > Suggested-by: Joe Stringer > Suggested-by: Daniele Di Proietto > Signed-off-by: Yi-Hung

Re: [ovs-dev] [PATCH 2/3] meta-flow: Remove dependency of cmap from meta-flow.h.

On 17 February 2017 at 17:47, Yi-Hung Wei wrote: > Previous patch 04f48a68 ("ofp-actions: Fix variable length meta-flow OXMs.") > introduced dependency of an internal library (cmap.h) to ovs public > interface (meta-flow.h) that may cause potential building problem. In this

Re: [ovs-dev] [PATCH 1/3] ofp-msgs: Remove unnecessary #include.

On 17 February 2017 at 17:47, Yi-Hung Wei wrote: > Remove unnecessary #include in ofp-msgs.h so that make will not complain. > > Signed-off-by: Yi-Hung Wei > --- Not sure about make complaining, but this include does seem to be both unnecessary and a

[ovs-dev] [PATCH] ofp-actions: Fix translation of set_field for nw_ecn

When using set_field for nw_ecn with OF1.0 or OF1.1, you get an error instead of a proper translation. This use to work before 4b684612d900 ("ofp-actions: Translate mod_nw_ecn action to OF1.1 properly.") because it would fallback to using NXM. e.g. [root@rhel7 ~]# ovs-ofctl -O OpenFlow10

Re: [ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

On Tue, Feb 21, 2017 at 04:29:51PM +0100, Timothy Redaelli wrote: > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1412694 > Signed-off-by: Timothy Redaelli > --- Looks good to me. Acked-by: Flavio Leitner

[ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1412694 Signed-off-by: Timothy Redaelli --- Documentation/intro/install/fedora.rst | 1 + Vagrantfile| 1 + rhel/openvswitch-fedora.spec.in| 54 --

[ovs-dev] [PATCH 1/4] id-pool: Allocate the lowest available ids.

This simple change makes id-pool to always allocate the lowest possible id from the pool. No any other code affected because, actually, there is no users of 'id_pool_free_id' in OVS. This behaviour of id-pool will be used in the next patch. Signed-off-by: Ilya Maximets

[ovs-dev] [PATCH 0/4] Incremental addition/deletion of PMD threads.

Ilya Maximets (4): id-pool: Allocate the lowest available ids. dpif-netdev: Incremental addition/deletion of PMD threads. dpif-netdev: Avoid port's reconfiguration on pmd-cpu-mask changes. dpif-netdev: Don't uninit emc on reload. lib/dpif-netdev.c | 129

[ovs-dev] [PATCH 2/4] dpif-netdev: Incremental addition/deletion of PMD threads.

Currently, change of 'pmd-cpu-mask' is very heavy operation. It requires destroying of all the PMD threads and creating them back. After that, all the threads will sleep until ports' redistribution finished. This patch adds ability to not stop the datapath while adjusting number/placement of PMD

[ovs-dev] [PATCH 3/4] dpif-netdev: Avoid port's reconfiguration on pmd-cpu-mask changes.

Reconfiguration of HW NICs may lead to packet drops. In current model all physical ports will be reconfigured each time number of PMD threads changed. Since we not stopping threads on pmd-cpu-mask changes, this patch will help to further decrease port's downtime by setting the maximum possible

Re: [ovs-dev] Memory pool issue on delete and re-add of DPDK ports with jumbo MTU >1894

Hi Mark, Thanks for the detailed analysis, i will wait for further updates. Btw on the mailing list, i seem to have clicked 'reply' instead of 'replyall' :) Regards Kapil. On Tue, Feb 21, 2017 at 3:26 PM, Kavanagh, Mark B wrote: > + dev - please keep the list

Re: [ovs-dev] [PATCH ovs V3 12/25] dpif-netlink: Use netdev flow put api to insert a flow

On 17/02/2017 19:38, Marcelo Ricardo Leitner wrote: On Wed, Feb 15, 2017 at 03:44:30PM +0200, Roi Dayan wrote: On 14/02/2017 01:55, Chandran, Sugesh wrote: Regards _Sugesh -Original Message- From: Roi Dayan [mailto:r...@mellanox.com] Sent: Wednesday, February 8, 2017 3:29 PM

[ovs-dev] [PATCH] ofproto: Remove 'ofproto/self-check' from ovs-vswitchd manpage

ofproto/self-check was removed in commit e79a6c83, so this patch removes the man page item. Signed-off-by: Timothy Redaelli --- ofproto/ofproto-unixctl.man | 7 --- 1 file changed, 7 deletions(-) diff --git a/ofproto/ofproto-unixctl.man b/ofproto/ofproto-unixctl.man

[ovs-dev] [PATCH] Documentation: Fix DPDK doc

OVS no longer expects DPDK devices to start with dpdk and end with a portid. Update docs to reflect this. Signed-off-by: Cian Ferriter --- Documentation/howto/dpdk.rst |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git

[ovs-dev] Intel's ONP User guide - test case description with config commands

Hi, Could someone help me to provide the Intel's ONP User guide which describes the complete configuration of OVS, test cases described ? That is, all the commands required to execute the test cases described in Intel

Re: [ovs-dev] [PATCH v3] dpif-netdev: Change definitions of 'idle' & 'processing' cycles

> -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Friday, 17 February, 2017 17:38 > > If there are multiple queues in a poll list and only one has packets, > the cycles polling the empty queues for packets will be counted in the > processing time - whereas

Re: [ovs-dev] Memory pool issue on delete and re-add of DPDK ports with jumbo MTU >1894

+ dev - please keep the list included in any discussions going forward folks :) Hi Kapil, I've managed to reproduce the issue of removing and re-adding ports with MTU > 1894, using the following configuration: DPDK: v16.07 OVS:f922f0f1c9