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

2017-04-07 Thread Roi Dayan
-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev.c | 30 ++ lib/netdev.h | 2 ++ vswitchd/bridge.c| 1 + vswitchd/vswitch.xml | 11 +++ 4 files changed, 44 insertions(+) diff --git a/lib/netdev.c b/lib/netdev.c index f7b80b2..977844a

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

2017-04-07 Thread Roi Dayan
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 --- lib/dpif-netlink.c | 5 + lib/netdev.c | 12 lib/netdev.h | 1 + 3 files

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

2017-04-07 Thread Roi Dayan
: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev.c | 6 ++ lib/tc.c | 43 ++- lib/tc.h | 1 + vswitchd/vswitch.xml | 17 + 4 files changed, 66 insertions(+), 1 deletion(-) diff --git a/lib/netdev.c

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

2017-04-06 Thread Roi Dayan
On 05/04/2017 10:08, Joe Stringer wrote: On 4 April 2017 at 21:18, Roi Dayan wrote: On 04/04/2017 23:53, Joe Stringer wrote: On 3 April 2017 at 10:53, Joe Stringer wrote: On 3 April 2017 at 03:27, Roi Dayan wrote: On 29/03/2017 20:13, Joe Stringer wrote: On 29 March 2017 at 04

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

2017-04-05 Thread Roi Dayan
On 05/04/2017 07:18, Roi Dayan wrote: On 04/04/2017 23:53, Joe Stringer wrote: On 3 April 2017 at 10:53, Joe Stringer wrote: On 3 April 2017 at 03:27, Roi Dayan wrote: On 29/03/2017 20:13, Joe Stringer wrote: On 29 March 2017 at 04:50, Roi Dayan wrote: On 23/03/2017 09:01, Joe

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

2017-04-04 Thread Roi Dayan
On 04/04/2017 21:40, Joe Stringer wrote: On 4 April 2017 at 06:21, Paul Blakey wrote: On 03/04/2017 21:00, Joe Stringer wrote: On 1 April 2017 at 20:50, Roi Dayan wrote: On 31/03/2017 01:11, Marcelo Ricardo Leitner wrote: On Thu, Mar 30, 2017 at 03:43:36PM -0300, Marcelo Ricardo

Re: [ovs-dev] [PATCH ovs V6 22/24] tests: Add system-offloads-testsuite

2017-04-04 Thread Roi Dayan
On 04/04/2017 23:47, Joe Stringer wrote: On 29 March 2017 at 05:43, Roi Dayan wrote: From: Paul Blakey The new system-offloads-testsuite, which can be launched via `make check-offloads`, tests offloading capabilities to makes sure that certian flows are actually offloaded. The tests run

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

2017-04-04 Thread Roi Dayan
On 04/04/2017 23:53, Joe Stringer wrote: On 3 April 2017 at 10:53, Joe Stringer wrote: On 3 April 2017 at 03:27, Roi Dayan wrote: On 29/03/2017 20:13, Joe Stringer wrote: On 29 March 2017 at 04:50, Roi Dayan wrote: On 23/03/2017 09:01, Joe Stringer wrote: I ran the make check

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

2017-04-03 Thread Roi Dayan
On 29/03/2017 20:13, Joe Stringer wrote: On 29 March 2017 at 04:50, Roi Dayan wrote: On 23/03/2017 09:01, Joe Stringer wrote: I ran the make check-offloads tests on a recent net-next kernel and it failed, output was not as expected: ../../tests/system-offloaded-traffic.at:54 <h

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

2017-04-02 Thread Roi Dayan
On 02/04/2017 06:50, Roi Dayan wrote: On 31/03/2017 01:11, Marcelo Ricardo Leitner wrote: On Thu, Mar 30, 2017 at 03:43:36PM -0300, Marcelo Ricardo Leitner wrote: On Wed, Mar 29, 2017 at 03:43:10PM +0300, Roi Dayan wrote: This patch series introduces rule offload functionality to dpif

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

2017-04-01 Thread Roi Dayan
On 31/03/2017 01:11, Marcelo Ricardo Leitner wrote: On Thu, Mar 30, 2017 at 03:43:36PM -0300, Marcelo Ricardo Leitner wrote: On Wed, Mar 29, 2017 at 03:43:10PM +0300, Roi Dayan wrote: This patch series introduces rule offload functionality to dpif-netlink via netdev ports new flow offloading

[ovs-dev] [PATCH ovs V6 17/24] netdev-tc-offloads: Implement flow get using tc interface

2017-03-29 Thread Roi Dayan
From: Paul Blakey Search the requested ufid for a offloaded flow, and if found, dump and parse it back to required format. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev-tc-offloads.c | 49 ++-- 1 file

[ovs-dev] [PATCH ovs V6 16/24] dpif-netlink: Use netdev flow get api to query a flow

2017-03-29 Thread Roi Dayan
From: Paul Blakey Search all datapath added netdevs for a given flow using netdev flow api and parse it back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/dpif-netlink.c | 50 +- lib/netdev.c

[ovs-dev] [PATCH ovs V6 15/24] netdev-tc-offloads: Implement netdev flow del using tc interface

2017-03-29 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev-tc-offloads.c | 33 ++--- 1 file changed, 30 insertions(+), 3 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 48a98eb

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

2017-03-29 Thread Roi Dayan
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 | 2 +- lib/netdev-tc-offloads.c | 324 ++- 2 files changed, 318 insertions

[ovs-dev] [PATCH ovs V6 14/24] dpif-netlink: Use netdev flow del api to delete a flow

2017-03-29 Thread Roi Dayan
From: Paul Blakey If a flow was offloaded to a netdev we delete it using netdev flow api. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/dpif-netlink.c | 12 +++- lib/netdev.c | 15 +++ lib/netdev.h | 2 ++ 3 files

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

2017-03-29 Thread Roi Dayan
headers. Signed-off-by: Paul Blakey Signed-off-by: Simon Horman Reviewed-by: Roi Dayan --- acinclude.m4 | 26 ++ configure.ac | 1 + include/automake.mk | 1 + include/linux/automake.mk| 4 + include/linux

[ovs-dev] [PATCH ovs V6 18/24] netdev-linux: Disallow setting policing when configured with hw offload

2017-03-29 Thread Roi Dayan
From: Paul Blakey Notify as not supported. Otherwise the ingress qdisc is being removed and offload rules will be removed. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev-linux.c | 8 1 file changed, 8 insertions(+) diff --git a/lib

[ovs-dev] [PATCH ovs V6 24/24] netdev: Init flow api on already added ports on offload enable

2017-03-29 Thread Roi Dayan
From: Paul Blakey Ports already added to a switch are not being initialized for offloading so when enabling offload we need to go over those ports. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev.c | 14 ++ 1 file changed, 14 insertions

[ovs-dev] [PATCH ovs V6 12/24] netdev-tc-offloads: Add flower mask to priority map

2017-03-29 Thread Roi Dayan
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 Horman --- lib/netdev-tc-offloads.c | 38

[ovs-dev] [PATCH ovs V6 22/24] tests: Add system-offloads-testsuite

2017-03-29 Thread Roi Dayan
From: Paul Blakey The new system-offloads-testsuite, which can be launched via `make check-offloads`, tests offloading capabilities to makes sure that certian flows are actually offloaded. The tests run on virtual netdevices (VETH). Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed

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

2017-03-29 Thread Roi Dayan
Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/dpctl.c | 21 ++- lib/dpctl.man | 7 - lib/dpif-netdev.c | 3 ++- lib/dpif-netlink.c| 62 ++- lib/dpif-provider.h

[ovs-dev] [PATCH ovs V6 20/24] netdev-tc-offloads: Add ingress on netdev flow api init

2017-03-29 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev-tc-offloads.c | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 19aceba..2d0fe38

[ovs-dev] [PATCH ovs V6 19/24] netdev-vport: Use common offloads interface

2017-03-29 Thread Roi Dayan
From: Paul Blakey netdev vports are backed by actualy netdev at the kernel level, so they can use the common netdev-tc offloads interface for flow offloading (if enabled). Signed-off-by: Paul Blakey Signed-off-by: Simon Horman Reviewed-by: Roi Dayan --- lib/netdev-linux.c | 7 +++--- lib

[ovs-dev] [PATCH ovs V6 10/24] netdev-tc-offloads: Implement netdev flow dump api using tc interface

2017-03-29 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev-tc-offloads.c | 182 --- 1 file changed, 174 insertions(+), 8 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc

[ovs-dev] [PATCH ovs V6 11/24] dpif-netlink: Use netdev flow put api to insert a flow

2017-03-29 Thread Roi Dayan
From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in later patches. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/dpif-netlink.c | 213

[ovs-dev] [PATCH ovs V6 08/24] dpif-netlink: Dump netdevs flows on flow dump

2017-03-29 Thread Roi Dayan
From: Paul Blakey While dumping flows, dump flows that were offloaded to netdev and parse them back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/dpif-netlink.c | 179 - lib/netdev.c

[ovs-dev] [PATCH ovs V6 09/24] netdev-tc-offloads: Add ufid to tc/netdev map

2017-03-29 Thread Roi Dayan
From: Paul Blakey Flows offloaded to tc are identified by priority and handle pair while OVS flows are identified by ufid. Added a hash map to convert between the two for later retrieval and deleting of offloaded flows. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon

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

2017-03-29 Thread Roi Dayan
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/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 946c2a6..2ea7fe3 100644

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

2017-03-29 Thread Roi Dayan
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 --- lib/dpif-netlink.c | 5 + lib/netdev.c | 12 lib/netdev.h | 1 + 3 files

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

2017-03-29 Thread Roi Dayan
-by: Shahar Klein Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman --- lib/automake.mk|2 + lib/netdev-linux.c | 164 ++-- lib/tc.c | 1109 lib/tc.h | 128 ++ 4 files changed

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

2017-03-29 Thread Roi Dayan
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-dummy.c | 2 + lib/netdev-linux.c | 15 +++-- lib/netdev-linux.h

[ovs-dev] [PATCH ovs V6 00/24] Introducing HW offload support for openvswitch

2017-03-29 Thread Roi Dayan
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 configuration. Netdev providers are able to implement netdev flow offload API in order to offload rules. This patc

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

2017-03-29 Thread Roi Dayan
From: Paul Blakey To use netdev flow offloading api, dpifs needs to iterate over added ports. This addition inserts the added dpif ports in a hash map, The map will also be used to translate dpif ports to netdevs. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan Reviewed-by: Simon Horman

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

2017-03-29 Thread Roi Dayan
-by: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev.c | 30 ++ lib/netdev.h | 2 ++ vswitchd/bridge.c| 1 + vswitchd/vswitch.xml | 11 +++ 4 files changed, 44 insertions(+) diff --git a/lib/netdev.c b/lib/netdev.c index f7b80b2..977844a

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

2017-03-29 Thread Roi Dayan
: Roi Dayan Reviewed-by: Simon Horman --- lib/netdev.c | 6 ++ lib/tc.c | 43 ++- lib/tc.h | 1 + vswitchd/vswitch.xml | 17 + 4 files changed, 66 insertions(+), 1 deletion(-) diff --git a/lib/netdev.c

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

2017-03-29 Thread Roi Dayan
On 23/03/2017 09:01, Joe Stringer wrote: I ran the make check-offloads tests on a recent net-next kernel and it failed, output was not as expected: ../../tests/system-offloaded-traffic.at:54 : ovs-appctl dpctl/dump-flows | grep "eth_type(0x0800)" | sed -e

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

2017-03-23 Thread Roi Dayan
On 23/03/2017 09:01, Joe Stringer wrote: On 22 March 2017 at 04:10, Roi Dayan mailto:r...@mellanox.com>> 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 n

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

2017-03-22 Thread Roi Dayan
Reviewed-by: Roi Dayan --- lib/dpctl.c | 21 ++- lib/dpctl.man | 7 - lib/dpif-netdev.c | 3 ++- lib/dpif-netlink.c| 62 ++- lib/dpif-provider.h | 6 +++-- lib/dpif.c

[ovs-dev] [PATCH ovs V5 15/24] netdev-tc-offloads: Implement netdev flow del using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 33 ++--- 1 file changed, 30 insertions(+), 3 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 0eb3e01..59af267 100644 --- a/lib

[ovs-dev] [PATCH ovs V5 14/24] dpif-netlink: Use netdev flow del api to delete a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey If a flow was offloaded to a netdev we delete it using netdev flow api. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 12 +++- lib/netdev.c | 15 +++ lib/netdev.h | 2 ++ 3 files changed, 28 insertions(+), 1

[ovs-dev] [PATCH ovs V5 16/24] dpif-netlink: Use netdev flow get api to query a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey Search all datapath added netdevs for a given flow using netdev flow api and parse it back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 50 +- lib/netdev.c | 16

[ovs-dev] [PATCH ovs V5 18/24] netdev-linux: Disallow setting policing when configured with hw offload

2017-03-22 Thread Roi Dayan
From: Paul Blakey Notify as not supported. Otherwise the ingress qdisc is being removed and offload rules will be removed. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-linux.c | 8 1 file changed, 8 insertions(+) diff --git a/lib/netdev-linux.c b/lib/netdev

[ovs-dev] [PATCH ovs V5 20/24] netdev-tc-offloads: Add ingress on netdev flow api init

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 0164efd..daa4f1f 100644 --- a/lib/netdev-tc

[ovs-dev] [PATCH ovs V5 19/24] netdev-vport: Use common offloads interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey netdev vports are backed by actualy netdev at the kernel level, so they can use the common netdev-tc offloads interface for flow offloading (if enabled). Signed-off-by: Paul Blakey Signed-off-by: Simon Horman Reviewed-by: Roi Dayan --- lib/netdev-linux.c | 7 +++--- lib

[ovs-dev] [PATCH ovs V5 17/24] netdev-tc-offloads: Implement flow get using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Search the requested ufid for a offloaded flow, and if found, dump and parse it back to required format. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 49 ++-- 1 file changed, 43 insertions(+), 6

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

2017-03-22 Thread Roi Dayan
headers. Signed-off-by: Paul Blakey Signed-off-by: Simon Horman Reviewed-by: Roi Dayan --- acinclude.m4 | 26 ++ configure.ac | 1 + include/automake.mk | 1 + include/linux/automake.mk| 4 + include/linux

[ovs-dev] [PATCH ovs V5 22/24] tests: Add system-offloads-testsuite

2017-03-22 Thread Roi Dayan
From: Paul Blakey The new system-offloads-testsuite, which can be launched via `make check-offloads`, tests offloading capabilities to makes sure that certian flows are actually offloaded. The tests run on virtual netdevices (VETH). Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan

[ovs-dev] [PATCH ovs V5 24/24] netdev: Init flow api on already added ports on offload enable

2017-03-22 Thread Roi Dayan
From: Paul Blakey Ports already added to a switch are not being initialized for offloading so when enabling offload we need to go over those ports. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/lib

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

2017-03-22 Thread Roi Dayan
From: Paul Blakey Currently only tunnel offload is supported. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 324 ++- 1 file changed, 317 insertions(+), 7 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib

[ovs-dev] [PATCH ovs V5 12/24] netdev-tc-offloads: Add flower mask to priority map

2017-03-22 Thread Roi Dayan
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 --- lib/netdev-tc-offloads.c | 38 ++ 1 file changed, 38

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

2017-03-22 Thread Roi Dayan
-by: Shahar Klein Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/automake.mk|2 + lib/netdev-linux.c | 164 ++-- lib/tc.c | 1108 lib/tc.h | 127 ++ 4 files changed, 1277 insertions(+), 124

[ovs-dev] [PATCH ovs V5 08/24] dpif-netlink: Dump netdevs flows on flow dump

2017-03-22 Thread Roi Dayan
From: Paul Blakey While dumping flows, dump flows that were offloaded to netdev and parse them back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 179 - lib/netdev.c | 32 ++ 2 files

[ovs-dev] [PATCH ovs V5 10/24] netdev-tc-offloads: Implement netdev flow dump api using tc interface

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 182 --- 1 file changed, 174 insertions(+), 8 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index bdde965..1091b4e

[ovs-dev] [PATCH ovs V5 00/24] Introducing HW offload support for openvswitch

2017-03-22 Thread Roi Dayan
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 configuration. Netdev providers are able to implement netdev flow offload API in order to offload rules. This patc

[ovs-dev] [PATCH ovs V5 09/24] netdev-tc-offloads: Add ufid to tc/netdev map

2017-03-22 Thread Roi Dayan
From: Paul Blakey Flows offloaded to tc are identified by priority and handle pair while OVS flows are identified by ufid. Added a hash map to convert between the two for later retrieval and deleting of offloaded flows. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc

[ovs-dev] [PATCH ovs V5 11/24] dpif-netlink: Use netdev flow put api to insert a flow

2017-03-22 Thread Roi Dayan
From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in later patches. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 213

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

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/automake.mk | 2 + lib/netdev-bsd.c | 2 + lib/netdev-dpdk.c| 1 + lib/netdev-dummy.c | 2 + lib/netdev-linux.c | 15 +++-- lib/netdev-linux.h | 9 +++ lib/netdev

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

2017-03-22 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 946c2a6..2ea7fe3 100644 --- a/lib/netdev-tc-offloads.c

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

2017-03-22 Thread Roi Dayan
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 --- lib/dpif-netlink.c | 5 + lib/netdev.c | 12 lib/netdev.h | 1 + 3 files changed, 18 insertions(+) diff

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

2017-03-22 Thread Roi Dayan
From: Paul Blakey To use netdev flow offloading api, dpifs needs to iterate over added ports. This addition inserts the added dpif ports in a hash map, The map will also be used to translate dpif ports to netdevs. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif.c | 25

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

2017-03-22 Thread Roi Dayan
: Roi Dayan --- lib/netdev.c | 6 ++ lib/tc.c | 43 ++- lib/tc.h | 1 + vswitchd/vswitch.xml | 17 + 4 files changed, 66 insertions(+), 1 deletion(-) diff --git a/lib/netdev.c b/lib/netdev.c index

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

2017-03-22 Thread Roi Dayan
-by: Roi Dayan --- lib/netdev.c | 30 ++ lib/netdev.h | 2 ++ vswitchd/bridge.c| 1 + vswitchd/vswitch.xml | 11 +++ 4 files changed, 44 insertions(+) diff --git a/lib/netdev.c b/lib/netdev.c index f7b80b2..977844a 100644 --- a/lib/netdev.c

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

2017-03-20 Thread Roi Dayan
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 via netdev ports new flow offloading API. The user can specify whether to enable

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 ovs V4 00/24] Introducing HW offload support for openvswitch

2017-03-14 Thread Roi Dayan
On 13/03/2017 20:38, 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

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

2017-03-13 Thread Roi Dayan
-by: Shahar Klein Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/automake.mk|2 + lib/netdev-linux.c | 164 ++--- lib/tc.c | 1042 lib/tc.h | 119 ++ 4 files changed, 1203 insertions(+), 124

[ovs-dev] [PATCH ovs V4 22/24] tests: Add system-offloads-testsuite

2017-03-13 Thread Roi Dayan
From: Paul Blakey The new system-offloads-testsuite, which can be launched via `make check-offloads`, tests offloading capabilities to makes sure that certian flows are actually offloaded. The tests run on virtual netdevices (VETH). Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan

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

2017-03-13 Thread Roi Dayan
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 --- lib/dpif-netlink.c |5 + lib/netdev.c | 12 lib/netdev.h |1 + 3 files changed, 18 insertions

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

2017-03-13 Thread Roi Dayan
From: Paul Blakey To use netdev flow offloading api, dpifs needs to iterate over added ports. This addition inserts the added dpif ports in a hash map, The map will also be used to translate dpif ports to netdevs. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif.c | 25

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

2017-03-13 Thread Roi Dayan
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 configuration. Netdev providers are able to implement netdev flow offload API in order to offload rules. This patc

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

2017-03-13 Thread Roi Dayan
Reviewed-by: Roi Dayan --- lib/dpctl.c | 21 ++ lib/dpif-netdev.c |3 +- lib/dpif-netlink.c| 62 + lib/dpif-provider.h |6 ++- lib/dpif.c|4 +- lib/dpif.h

[ovs-dev] [PATCH ovs V4 16/24] dpif-netlink: Use netdev flow get api to query a flow

2017-03-13 Thread Roi Dayan
From: Paul Blakey Search all datapath added netdevs for a given flow using netdev flow api and parse it back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 50 +- lib/netdev.c | 16

[ovs-dev] [PATCH ovs V4 17/24] netdev-tc-offloads: Implement flow get using tc interface

2017-03-13 Thread Roi Dayan
From: Paul Blakey Search the requested ufid for a offloaded flow, and if found, dump and parse it back to required format. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 49 - 1 files changed, 43 insertions(+), 6

[ovs-dev] [PATCH ovs V4 11/24] dpif-netlink: Use netdev flow put api to insert a flow

2017-03-13 Thread Roi Dayan
From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in later patches. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 213

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

2017-03-13 Thread Roi Dayan
From: Paul Blakey Added compatibility headers for actions vlan and tunnel key. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- acinclude.m4 | 26 ++ configure.ac |1 + include/automake.mk |1 + include/linux

[ovs-dev] [PATCH ovs V4 15/24] netdev-tc-offloads: Implement netdev flow del using tc interface

2017-03-13 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 33 ++--- 1 files changed, 30 insertions(+), 3 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index cf21c3a..7b21b73 100644 --- a

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

2017-03-13 Thread Roi Dayan
From: Paul Blakey Currently only tunnel offload is supported. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 309 - 1 files changed, 302 insertions(+), 7 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib

[ovs-dev] [PATCH ovs V4 24/24] netdev: Init flow api on already added ports on offload enable

2017-03-13 Thread Roi Dayan
From: Paul Blakey Ports already added to a switch are not being initialized for offloading so when enabling offload we need to go over those ports. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions

[ovs-dev] [PATCH ovs V4 20/24] netdev-tc-offloads: Add ingress on netdev flow api init

2017-03-13 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 23 ++- 1 files changed, 22 insertions(+), 1 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index b822777..e06201f 100644 --- a/lib/netdev

[ovs-dev] [PATCH ovs V4 19/24] netdev-vport: Use common offloads interface

2017-03-13 Thread Roi Dayan
From: Paul Blakey netdev vports are backed by actualy netdev at the kernel level, so they can use the common netdev-tc offloads interface for flow offloading (if enabled). Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-linux.c |7 ++--- lib/netdev-linux.h |2 + lib

[ovs-dev] [PATCH ovs V4 14/24] dpif-netlink: Use netdev flow del api to delete a flow

2017-03-13 Thread Roi Dayan
From: Paul Blakey If a flow was offloaded to a netdev we delete it using netdev flow api. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 12 +++- lib/netdev.c | 15 +++ lib/netdev.h |2 ++ 3 files changed, 28 insertions

[ovs-dev] [PATCH ovs V4 18/24] netdev-linux: Disallow setting policing when configured with hw offload

2017-03-13 Thread Roi Dayan
From: Paul Blakey Notify as not supported. Otherwise the ingress qdisc is being removed and offload rules will be removed. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-linux.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/lib/netdev

[ovs-dev] [PATCH ovs V4 12/24] netdev-tc-offloads: Add flower mask to priority map

2017-03-13 Thread Roi Dayan
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 --- lib/netdev-tc-offloads.c | 38 ++ 1 files changed, 38

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

2017-03-13 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/automake.mk |2 + lib/netdev-bsd.c |2 + lib/netdev-dpdk.c|1 + lib/netdev-dummy.c |2 + lib/netdev-linux.c | 15 +++- lib/netdev-linux.h |9 +++ lib

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

2017-03-13 Thread Roi Dayan
: Roi Dayan --- lib/netdev.c |6 ++ lib/tc.c | 43 ++- lib/tc.h |1 + vswitchd/vswitch.xml | 17 + 4 files changed, 66 insertions(+), 1 deletions(-) diff --git a/lib/netdev.c b/lib/netdev.c

[ovs-dev] [PATCH ovs V4 10/24] netdev-tc-offloads: Implement netdev flow dump api using tc interface

2017-03-13 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 161 +++-- 1 files changed, 153 insertions(+), 8 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index bdde965..4c4e43f

[ovs-dev] [PATCH ovs V4 08/24] dpif-netlink: Dump netdevs flows on flow dump

2017-03-13 Thread Roi Dayan
From: Paul Blakey While dumping flows, dump flows that were offloaded to netdev and parse them back to dpif flow. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/dpif-netlink.c | 179 +++- lib/netdev.c | 32 + 2 files

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

2017-03-13 Thread Roi Dayan
-by: Roi Dayan --- lib/netdev.c | 30 ++ lib/netdev.h |2 ++ vswitchd/bridge.c|1 + vswitchd/vswitch.xml | 11 +++ 4 files changed, 44 insertions(+), 0 deletions(-) diff --git a/lib/netdev.c b/lib/netdev.c index f7b80b2..977844a

[ovs-dev] [PATCH ovs V4 09/24] netdev-tc-offloads: Add ufid to tc/netdev map

2017-03-13 Thread Roi Dayan
From: Paul Blakey Flows offloaded to tc are identified by priority and handle pair while OVS flows are identified by ufid. Added a hash map to convert between the two for later retrieval and deleting of offloaded flows. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc

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

2017-03-13 Thread Roi Dayan
From: Paul Blakey Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan --- lib/netdev-tc-offloads.c | 14 -- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 946c2a6..2ea7fe3 100644 --- a/lib/netdev-tc

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

2017-03-12 Thread Roi Dayan
On 10/03/2017 10:46, Simon Horman wrote: On Wed, Feb 08, 2017 at 05:29:13PM +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 ovs V3 12/25] dpif-netlink: Use netdev flow put api to insert a flow

2017-02-21 Thread Roi Dayan
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

Re: [ovs-dev] [PATCH ovs V3 05/25] other-config: Add tc-policy switch to control tc flower flag

2017-02-19 Thread Roi Dayan
On 17/02/2017 11:07, Simon Horman wrote: On Wed, Feb 08, 2017 at 05:29:18PM +0200, 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

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

2017-02-15 Thread Roi Dayan
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 To: d...@openvswitch.org Cc: Paul Blakey ; Or Gerlitz ; Hadar Hen Zion ; Shahar Klein ; Mark Bloch ; Rony Efraim

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

2017-02-15 Thread Roi Dayan
On 14/02/2017 17:54, Simon Horman wrote: On Wed, Feb 08, 2017 at 05:29:25PM +0200, Roi Dayan wrote: From: Paul Blakey Using the new netdev flow api operate will now try and offload flows to the relevant netdev of the input port. Other operate methods flows will come in later patches

Re: [ovs-dev] [PATCH ovs V3 23/25] dpctl: Add an option to dump only certain kinds of flows

2017-02-15 Thread Roi Dayan
On 14/02/2017 17:55, Simon Horman wrote: On Wed, Feb 08, 2017 at 05:29:36PM +0200, 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 ovs does

Re: [ovs-dev] [PATCH ovs V3 14/25] netdev-tc-offloads: Implement netdev flow put using tc interface

2017-02-15 Thread Roi Dayan
On 14/02/2017 01:56, Chandran, Sugesh wrote: Regards _Sugesh -Original Message- From: Roi Dayan [mailto:r...@mellanox.com] Sent: Wednesday, February 8, 2017 3:29 PM To: d...@openvswitch.org Cc: Paul Blakey ; Or Gerlitz ; Hadar Hen Zion ; Shahar Klein ; Mark Bloch ; Rony Efraim

Re: [ovs-dev] [PATCH ovs V3 05/25] other-config: Add tc-policy switch to control tc flower flag

2017-02-15 Thread Roi Dayan
On 15/02/2017 13:16, Chandran, Sugesh wrote: Regards _Sugesh -Original Message- From: Roi Dayan [mailto:r...@mellanox.com] Sent: Wednesday, February 15, 2017 7:42 AM To: Chandran, Sugesh ; d...@openvswitch.org Cc: r...@mellanox.com; Paul Blakey ; Or Gerlitz ; Hadar Hen Zion

Re: [ovs-dev] [PATCH ovs V3 13/25] netdev-tc-offloads: Add flower mask to priority map

2017-02-15 Thread Roi Dayan
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 To: d...@openvswitch.org Cc: Paul Blakey ; Or Gerlitz ; Hadar Hen Zion ; Shahar Klein ; Mark Bloch ; Rony Efraim

<    5   6   7   8   9   10   11   >