Re: [ovs-dev] [PATCH ovs V8 02/26] netdev: Adding a new netdev api to be used for offloading flows

2017-05-16 Thread Flavio Leitner
On Mon, May 15, 2017 at 09:34:35AM +0300, Roi Dayan wrote: > > > On 09/05/2017 21:12, Flavio Leitner wrote: > > On Wed, May 03, 2017 at 06:07:53PM +0300, Roi Dayan wrote: > > > From: Paul Blakey > > > > > > Signed-off-by: Paul Blakey > > &g

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

2017-05-10 Thread Flavio Leitner
On Wed, May 10, 2017 at 10:44:46AM +0300, Roi Dayan wrote: > > > On 09/05/2017 22:05, Flavio Leitner wrote: > > On Sun, May 07, 2017 at 10:55:32AM +0300, Roi Dayan wrote: > > > > > > > > > On 03/05/2017 18:58, Federico Iezzi wrote: > > >

Re: [ovs-dev] [PATCH ovs V8 12/26] netdev-tc-offloads: Add flower mask to priority map

2017-05-09 Thread Flavio Leitner
On Tue, May 09, 2017 at 03:23:12PM -0300, Flavio Leitner wrote: > On Wed, May 03, 2017 at 06:08:03PM +0300, Roi Dayan wrote: > > From: Paul Blakey > > > > Flower classifer requires a different priority per mask, > > so we hash the mask and generate a new priority

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

2017-05-09 Thread Flavio Leitner
On Sun, May 07, 2017 at 10:55:32AM +0300, Roi Dayan wrote: > > > On 03/05/2017 18:58, Federico Iezzi wrote: > > On Wed, May 3, 2017 at 5:07 PM, Roi Dayan wrote: > > > This patch series introduces rule offload functionality to dpif-netlink > > > via netdev ports new flow offloading API. The user

Re: [ovs-dev] [PATCH ovs V8 12/26] netdev-tc-offloads: Add flower mask to priority map

2017-05-09 Thread Flavio Leitner
On Wed, May 03, 2017 at 06:08:03PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Flower classifer requires a different priority per mask, > so we hash the mask and generate a new priority for > each new mask used. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon H

Re: [ovs-dev] [PATCH ovs V8 13/26] netdev-tc-offloads: Implement netdev flow put using tc interface

2017-05-09 Thread Flavio Leitner
On Wed, May 03, 2017 at 06:08:04PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Currently only tunnel offload is supported. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --- > lib/dpif-netlink.c | 4 +- > lib/netdev-tc-offloads.c | 392 > ++

Re: [ovs-dev] [PATCH ovs V8 02/26] netdev: Adding a new netdev api to be used for offloading flows

2017-05-09 Thread Flavio Leitner
On Wed, May 03, 2017 at 06:07:53PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --- > lib/automake.mk | 2 + > lib/netdev-bsd.c | 2 + > lib/netdev-dpdk.c| 1 + > lib/netdev-dum

Re: [ovs-dev] [PATCH] build: Don't run tests in rpm makefile targets.

2017-04-27 Thread Flavio Leitner
On Thu, Apr 27, 2017 at 03:04:43PM -0400, Russell Bryant wrote: > On Fri, Apr 21, 2017 at 7:01 PM, Ben Pfaff wrote: > > On Fri, Mar 31, 2017 at 11:27:23AM -0400, Russell Bryant wrote: > >> The RPM build makefile targets are helpful during development and testing, > >> but I personally almost never

Re: [ovs-dev] [PATCH ovs V7 21/24] dpctl: Add an option to dump only certain kinds of flows

2017-04-12 Thread Flavio Leitner
On Wed, Apr 12, 2017 at 11:01:50AM -0700, Joe Stringer wrote: > On 12 April 2017 at 10:53, Flavio Leitner wrote: > > On Wed, Apr 12, 2017 at 10:04:50AM -0700, Joe Stringer wrote: > >> On 12 April 2017 at 08:22, Flavio Leitner wrote: > >> > On Fri, Apr 07, 2017

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

2017-04-12 Thread Flavio Leitner
On Wed, Apr 12, 2017 at 02:53:13PM -0300, Marcelo Ricardo Leitner wrote: > Hi, > > On Wed, Apr 12, 2017 at 01:13:36PM -0300, Flavio Leitner wrote: > > > > Hi Marcelo, > > > > Could you please confirm if this patch series fixes the aggregation > > issue

Re: [ovs-dev] Minutes: OvS Offload Discussion at Netdev 2.1

2017-04-12 Thread Flavio Leitner
On Sat, Apr 08, 2017 at 04:47:57PM -0400, Simon Horman wrote: > At Netdev 2.1 a meeting was held to discuss OvS offload. Minutes of the > discussion follow. I apologise in advance for any errors or omissions; > doubly for any errors in the attendee list. > > Topic: OVS Hardware Offload Using TC >

Re: [ovs-dev] [PATCH ovs V7 21/24] dpctl: Add an option to dump only certain kinds of flows

2017-04-12 Thread Flavio Leitner
On Wed, Apr 12, 2017 at 10:04:50AM -0700, Joe Stringer wrote: > On 12 April 2017 at 08:22, Flavio Leitner wrote: > > On Fri, Apr 07, 2017 at 04:13:08PM +0300, Roi Dayan wrote: > >> From: Paul Blakey > >> > >> Usage: > >> # to dump all datapath f

Re: [ovs-dev] [PATCH ovs V7 13/24] netdev-tc-offloads: Implement netdev flow put using tc interface

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:13:00PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Currently only tunnel offload is supported. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --- > lib/dpif-netlink.c | 4 +- > lib/netdev-tc-offloads.c | 390 > ++

Re: [ovs-dev] [PATCH ovs V7 23/24] compat: Add tc compatibility headers for old kernels

2017-04-12 Thread Flavio Leitner
rs to not break bisection in the future. Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

2017-04-12 Thread Flavio Leitner
Hi Marcelo, Could you please confirm if this patch series fixes the aggregation issue in your environment? Thanks, fbl On Fri, Apr 07, 2017 at 04:12:47PM +0300, Roi Dayan wrote: > This patch series introduces rule offload functionality to dpif-netlink > via netdev ports new flow offloading API

Re: [ovs-dev] [PATCH ovs V7 21/24] dpctl: Add an option to dump only certain kinds of flows

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:13:08PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Usage: > # to dump all datapath flows (default): > ovs-dpctl dump-flows > > # to dump only flows that in kernel datapath: > ovs-dpctl dump-flows type=ovs > > # to dump only flows that are offl

Re: [ovs-dev] [PATCH ovs V7 07/24] netdev-tc-offloads: Implement netdev flow flush using tc interface

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:12:54PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --- > lib/netdev-tc-offloads.c | 14 -- > 1 file changed, 12 insertions(+), 2 deletions(-) > > diff --git a/lib/net

Re: [ovs-dev] [PATCH ovs V7 06/24] dpif-netlink: Flush added ports using netdev flow api

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:12:53PM +0300, Roi Dayan wrote: > From: Paul Blakey > > If netdev flow offloading is enabled, flush all > added ports using netdev flow api. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --

Re: [ovs-dev] [PATCH ovs V7 05/24] dpif: Save added ports in a port map for netdev flow api use

2017-04-12 Thread Flavio Leitner
vs. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman > --- Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovs V7 04/24] other-config: Add tc-policy switch to control tc flower flag

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:12:51PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add a new configuration tc-policy option that controls tc > flower flag. Possible options are none, skip_sw, skip_hw. > The default is none which is to insert the rule both to sw and hw. > This option is only relev

Re: [ovs-dev] [PATCH ovs V7 03/24] other-config: Add hw-offload switch to control netdev flow offloading

2017-04-12 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:12:50PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add a new configuration option - hw-offload that enables netdev > flow api. Enabling this option will allow offloading flows > using netdev implementation instead of the kernel datapath. > This configuration option

Re: [ovs-dev] [PATCH ovs V7 01/24] tc: Add tc flower interface

2017-04-07 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 05:56:40PM -0300, Flavio Leitner wrote: > On Fri, Apr 07, 2017 at 04:12:48PM +0300, Roi Dayan wrote: > > From: Paul Blakey [...] > > > > +static const struct nl_policy act_policy[] = { > > +[TCA_ACT_KIND] = { .type = NL_

Re: [ovs-dev] [PATCH ovs V7 01/24] tc: Add tc flower interface

2017-04-07 Thread Flavio Leitner
On Fri, Apr 07, 2017 at 04:12:48PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add tc flower interface that will be used to offload flows via tc > flower classifier. Depending on the flag used (skip_sw/hw) flower > will pass those to HW or handle them itself. > Move some tc related functions

Re: [ovs-dev] [RFC] lib/automake.mk: remove runtime directories

2017-03-23 Thread Flavio Leitner
On Wed, Mar 22, 2017 at 12:43:29PM -0400, Eric Garver wrote: > On Tue, Mar 21, 2017 at 02:20:30PM -0400, Aaron Conole wrote: > > Aaron Conole writes: > > > > > The Open vSwitch run, log, and DB directories are installed as part of the > > > normal `make install` process. However, this means they

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

2017-03-03 Thread Flavio Leitner
On Thu, Mar 02, 2017 at 04:13:17PM -0500, Leif Madsen wrote: > On Thu, Mar 2, 2017 at 3:44 PM, Russell Bryant wrote: > > > > > On Thu, Mar 2, 2017 at 3:42 PM, Leif Madsen wrote: > > > >> Just an FYI that this patch breaks CentOS 7 builds because of the > >> python3-devel dependency. Might need t

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

2017-02-24 Thread Flavio Leitner
Hi, We pushed 2.7 through our testing and found no issues. Thanks! Flavio On Tue, Feb 21, 2017 at 01:59:23PM -0800, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > NEWS | 2 +- > debian/changelog | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --g

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

2017-02-22 Thread Flavio Leitner
On Wed, Feb 22, 2017 at 04:29:09AM +, Darrell Ball wrote: > > > 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 int

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

2017-02-21 Thread Flavio Leitner
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 tha

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

2017-02-21 Thread Flavio Leitner
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 https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

2017-02-21 Thread Flavio Leitner
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 https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

2017-02-21 Thread Flavio Leitner
e unrelated changes like cleanups or code movement in process_one(), but that's fine I guess. Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [patch_v6 3/8] dpdk: Remove batch sorting in userspace conntrack.

2017-02-21 Thread Flavio Leitner
;or gateways, the batch sorting won't provide benefit >and will just be a tax. > > 3) Code clarity. > > Signed-off-by: Darrell Ball > --- I can't tell about the real performance impact but I'd agree with the second point. Acked-by: Flavio Leitner _

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

2017-02-21 Thread Flavio Leitner
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 ___ dev maili

Re: [ovs-dev] [PATCH 2/2] rhel-systemd: Document systemd behavior

2017-02-20 Thread Flavio Leitner
ed-off-by: Eelco Chaudron > --- Glad to see this being documented, Thanks Eelco. Acked-by: Flavio Leitner Russell, FYI this is the second patch documenting the change of another patch as noted in the commit message. So, either apply both or none :-). Thanks Flavio

Re: [ovs-dev] [PATCH] rhel-systemd: Restart openvswitch service if a daemon crashes

2017-02-20 Thread Flavio Leitner
temctl start ovs-vswitchd* | started | started |inactive, > dead|active,running|active,running| > systemctl start ovsdb-server* | not started | started |inactive, > dead|inactive, dead|active,running| > > * These commands whe

Re: [ovs-dev] [DPDK:patch_v5 0/8] Userspace Datapath: Introduce NAT support.

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:17:53AM -0800, Darrell Ball wrote: > This patch series introduces NAT support for the userspace datapath. Thanks a lot for this work, nice. I did a quick look at the patches and provided feedback on the easier ones, but I plan to go over the long ones too. > The per p

Re: [ovs-dev] [DPDK:patch_v5 8/8] Update NEWS for userspace datapath NAT

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:18:01AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball That's an important one, but I think you missed the table in Documentation/faq/releases.rst Feature Linux upstream Linux OVS tree Userspace Hyper-V =

Re: [ovs-dev] [DPDK:patch_v5 7/8] Enable NAT tests for userspace datapath.

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:18:00AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball > --- Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [DPDK:patch_v5 6/8] Add missing CHECK_CONNTRACK_ALG guards.

2017-02-15 Thread Flavio Leitner
-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [DPDK:patch_v5 5/8] Enhance V6 NAT test.

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:17:58AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball > --- This is an interesting improvement but not much related to the patchset, so maybe if the reviews are okay it can be individually applied. Acked-by: Flavio L

Re: [ovs-dev] [DPDK:patch_v5 3/8] Remove batch sorting in userspace conntrack.

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:17:56AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball I miss a comment to why this patch is needed. Thanks Flavio > --- > lib/conntrack.c | 49 ++--- > 1 file changed, 10 insertions(+), 39 deletions(-) > > diff

Re: [ovs-dev] [DPDK:patch_v5 1/8] Export packet_set_ipv6_addr() for DPDK.

2017-02-15 Thread Flavio Leitner
On Wed, Feb 08, 2017 at 12:17:54AM -0800, Darrell Ball wrote: > Signed-off-by: Darrell Ball > --- IPv4 version is exported already (for another reason) and both are needed by nat_packet() in a later patch. Acked-by: Flavio Leitner ___ dev m

Re: [ovs-dev] [PATCH 2/2] doc: Add info on distributions shipping openvswitch package.

2017-02-08 Thread Flavio Leitner
On Tue, Feb 07, 2017 at 09:18:23PM +, Bhanuprakash Bodireddy wrote: > List details of various popular distributions shipping Open vSwitch > packages. Also include the information of the distros supporting DPDK > accelerated datapath. > > Signed-off-by: Bhanuprakash Bodireddy > --- > Document

Re: [ovs-dev] [RFC] ofproto/bond: operational vs administratively disabled bond interface

2017-01-17 Thread Flavio Leitner
Hi, On Tue, 17 Jan 2017 09:34:19 +0100 Eelco Chaudron wrote: > Currently OVS does not distinguish between a bond slave being operational > disabled, i.e. link being down, and administratively disabled. > > Take the example where the administrator disabled a link in a bond, > "ovs-appctl bond/d

Re: [ovs-dev] [PATCH] rhel: Support building python ovs package with native json impl

2016-12-30 Thread Flavio Leitner
On Tue, 13 Dec 2016 13:45:05 -0500 Aaron Conole wrote: > Although I'm not Flavio, I do have some quick thoughts on this. > > Russell Bryant writes: > > > My main concern with this was that it implements a new python-ovs package > > and leaves the existing one alone. I'd like to figure out how

Re: [ovs-dev] [Openvswitch support NSH]

2016-11-21 Thread Flavio Leitner
On Fri, Nov 18, 2016 at 08:02:30PM +0700, võ phúc wrote: > Dear OVS team, > I want install openstack with sfc, so i want to find ovs support with NSH , > you can show me how to build ovs with NSH or version OVS support NSH? > Thank you. This is a work in progress to add NSH support to OVS which re

<    6   7   8   9   10   11