On 30/05/2017 11:10, Simon Horman wrote:
On Sun, May 28, 2017 at 02:59:49PM +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
On 02/06/2017 16:45, Flavio Leitner wrote:
On Sun, May 28, 2017 at 02:59:50PM +0300, Roi Dayan wrote:
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
On 02/06/2017 17:46, Flavio Leitner wrote:
On Sun, May 28, 2017 at 02:59:51PM +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
Acked-by
On 29/05/2017 10:30, Roi Dayan wrote:
On 28/05/2017 14:59, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev-tc-offloads.c | 184
---
1 file changed, 175 insertions
On 01/06/2017 20:53, Joe Stringer wrote:
On 1 June 2017 at 07:39, Roi Dayan wrote:
On 31/05/2017 03:50, Joe Stringer wrote:
On 28 May 2017 at 04:59, Roi Dayan wrote:
Add tc helper functions to query and manipulate the flower classifier.
Signed-off-by: Paul Blakey
Signed-off-by: Roi
On 30/05/2017 11:09, Simon Horman wrote:
On Sun, May 28, 2017 at 02:59:47PM +0300, Roi Dayan wrote:
From: Paul Blakey
Add a new API interface for offloading dpif flows to netdev.
The API consist on the following:
flow_put - offload a new flow
flow_get - query an offloaded flow
On 31/05/2017 03:20, Joe Stringer wrote:
On 28 May 2017 at 04:59, Roi Dayan wrote:
Add tc module to expose tc operations to be used by other modules.
Move some tc related functions from netdev-linux.c to tc.c
This patch doesn't change any functionality.
Signed-off-by: Paul Blakey
S
On 30/05/2017 11:07, Simon Horman wrote:
On Sun, May 28, 2017 at 02:59:44PM +0300, Roi Dayan wrote:
Add tc module to expose tc operations to be used by other modules.
Move some tc related functions from netdev-linux.c to tc.c
This patch doesn't change any functionality.
Signed-off-by:
On 31/05/2017 03:12, Joe Stringer wrote:
On 28 May 2017 at 04:59, Roi Dayan wrote:
Refactor tc_make_request and tc_add_del_ingress_qdisc to accept
ifindex instead of netdev struct.
We later want to move those outside netdev-linux module to be
used by other modules.
This patch doesn't c
On 31/05/2017 03:50, Joe Stringer wrote:
On 28 May 2017 at 04:59, Roi Dayan wrote:
Add tc helper functions to query and manipulate the flower classifier.
Signed-off-by: Paul Blakey
Signed-off-by: Roi Dayan
---
Again is this co-authored? utilities/checkpatch.py checks for stuff like this
On 28/05/2017 14:59, 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/netdev-tc-offloads.c | 400 +--
1 file changed, 390
On 28/05/2017 14:59, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev-tc-offloads.c | 184 ---
1 file changed, 175 insertions(+), 9 deletions(-)
diff --git a/lib
Signed-off-by: Roi Dayan
---
NEWS | 3 +++
1 file changed, 3 insertions(+)
diff --git a/NEWS b/NEWS
index 4d03ca5..4bfcf78 100644
--- a/NEWS
+++ b/NEWS
@@ -36,6 +36,9 @@ Post-v2.7.0
* The port status bit OFPPS_LIVE now reflects link aliveness.
- Fedora Packaging:
* OVN services
Remove redundant logging functions and reuse
the exposed dpif logging functions.
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/dpif-netlink.c | 39 ++-
1 file changed, 6 insertions(+), 33 deletions(-)
diff --git a/lib/dpif-netlink.c b/lib/dpif
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 4798d86..98c25f6
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
To be reused by other modules.
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/dpif.c | 87 +++---
lib/dpif.h | 28
2 files changed, 72 insertions(+), 43 deletions(-)
diff --git a/lib/dpif.c b/lib/dpif.c
This is for it to appear in bash completion.
Signed-off-by: Roi Dayan
---
lib/dpctl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dpctl.c b/lib/dpctl.c
index 11be857..d3d97d5 100644
--- a/lib/dpctl.c
+++ b/lib/dpctl.c
@@ -1554,7 +1554,7 @@ static const struct
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
From: Paul Blakey
When verbosity is requested on dump-flows (-m) indicate which flows
are offloaded.
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
---
lib/dpctl.c| 11 ---
lib/dpif-netlink.c | 4
lib/dpif.h | 1 +
3 files changed, 13 insertions(+), 3
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 | 218
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 | 50 ++--
1 file
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
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 | 13 -
lib/netdev.c | 15 +++
lib/netdev.h | 2 ++
3 files
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/dpctl.c | 44 --
lib/dpctl.man | 7 -
lib/dpif-netdev.c | 3 ++-
lib/dpif-netlink.c| 63 ++-
lib/dpif
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
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 f141784
From: Paul Blakey
Currently only tunnel offload is supported.
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev-tc-offloads.c | 400 +--
1 file changed, 390 insertions(+), 10 deletions(-)
diff --git a/lib
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 | 54
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev-tc-offloads.c | 184 ---
1 file changed, 175 insertions(+), 9 deletions(-)
diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc
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 | 168 +
lib/netdev.c
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
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 | 51 ++-
lib
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
Acked-by: Flavio Leitner
---
lib/dpif-netlink.c | 5 +
lib/netdev.c | 12
lib/netdev.h
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 46017d8..e766972 100644
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
: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev.c | 6 ++
lib/tc.c | 43 ++-
lib/tc.h | 3 +++
vswitchd/vswitch.xml | 17 +
4 files changed, 68 insertions(+), 1 deletion(-)
diff --git a/lib
-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/netdev.c | 35 +++
lib/netdev.h | 2 ++
vswitchd/bridge.c| 1 +
vswitchd/vswitch.xml | 15 +++
4 files changed, 53 insertions(+)
diff --git a/lib/netdev.c b/lib/netdev.c
index 9eb37ec
flows
In upcoming commits we will introduce an implementation of this
API for netdev-linux.
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
headers.
Signed-off-by: Paul Blakey
Signed-off-by: Simon Horman
Reviewed-by: Roi Dayan
Acked-by: Flavio Leitner
---
acinclude.m4 | 26 ++
configure.ac | 1 +
include/automake.mk | 1 +
include/linux/automake.mk
Add tc helper functions to query and manipulate the flower classifier.
Signed-off-by: Paul Blakey
Signed-off-by: Roi Dayan
---
lib/tc.c | 992 +++
lib/tc.h | 92 ++
2 files changed, 1084 insertions(+)
diff --git a/lib/tc.c b/lib
s: Add ingress on netdev flow api init
dpctl: Add an option to dump only certain kinds of flows
dpctl: Indicate if flow is offloaded when dumping flows of all types
tests: Add system-offloads-testsuite
netdev: Init flow api on already added ports on offload enable
Roi Dayan (7):
netdev
Add tc module to expose tc operations to be used by other modules.
Move some tc related functions from netdev-linux.c to tc.c
This patch doesn't change any functionality.
Signed-off-by: Paul Blakey
Signed-off-by: Roi Dayan
---
lib/automake.mk| 2 +
lib/netdev-linux.c
Refactor tc_make_request and tc_add_del_ingress_qdisc to accept
ifindex instead of netdev struct.
We later want to move those outside netdev-linux module to be
used by other modules.
This patch doesn't change any functionality.
Signed-off-by: Paul Blakey
Signed-off-by: Roi Dayan
---
lib/n
On 10/05/2017 17:29, Flavio Leitner wrote:
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:
On Wed, May 3, 2017 at 5:07 PM, Roi
On 10/05/2017 18:20, Simon Horman wrote:
On Wed, May 03, 2017 at 06:08:17PM +0300, Roi Dayan wrote:
Please add some changelog text here.
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/dpif-netlink.c | 55 +-
1 file changed
On 10/05/2017 18:21, Simon Horman wrote:
On Wed, May 10, 2017 at 05:19:26PM +0200, Simon Horman wrote:
On Wed, May 03, 2017 at 06:08:16PM +0300, Roi Dayan wrote:
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
Reviewed-by: Simon Horman
Sorry, I didn't notice before but could yo
On 10/05/2017 16:57, Simon Horman wrote:
On Wed, May 03, 2017 at 06:08:12PM +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
On 10/05/2017 16:48, Simon Horman wrote:
On Wed, May 03, 2017 at 06:08:12PM +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
On 10/05/2017 16:28, Simon Horman wrote:
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
On 09/05/2017 21:12, Flavio Leitner wrote:
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
On 09/05/2017 22:10, Flavio Leitner wrote:
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
On 08/05/2017 15:56, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:58PM +0300, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
This does not appear to address Flavio Leitner's review of v7 of this patch.
email i
On 08/05/2017 15:50, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:57PM +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
Flavio
On 08/05/2017 15:49, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:56PM +0300, Roi Dayan wrote:
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
On 08/05/2017 15:38, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:54PM +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
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
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
lib/automake.mk | 2 +
lib/netdev-bsd.c | 2 +
lib/netdev-dpdk.c
On 08/05/2017 15:28, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:53PM +0300, Roi Dayan wrote:
From: Paul Blakey
Please add some text to the changelog.
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
...
diff --git a/lib/netdev-tc-offloads.c b/lib
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:
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
On 04/05/2017 19:35, Simon Horman wrote:
On Wed, May 03, 2017 at 06:07:52PM +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
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 can specify whether to
enable rule offloading or not via OVS configuration
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/dpif-netlink.c | 55 +-
1 file changed, 17 insertions(+), 38 deletions(-)
diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c
index 048dae6..3638358 100644
--- a/lib/dpif-netlink.c
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/dpif.c | 87 +++---
lib/dpif.h | 28
2 files changed, 72 insertions(+), 43 deletions(-)
diff --git a/lib/dpif.c b/lib/dpif.c
index 658830b..bc1f034 100644
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
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
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
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
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| 63 ++-
lib/dpif-provider.h
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
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 c3eddf4..4fc0445
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 | 50 ++--
1 file
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
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
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 e3daf62
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 ++-
2 files changed, 385 insertions
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
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
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 | 216
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
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 | 168 +
lib/netdev.c
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
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 eb5e79a..b12a048 100644
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
: 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
-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 1677027..90c95cb
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
-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
ption to dump only certain kinds of flows
tests: Add system-offloads-testsuite
compat: Add tc compatibility headers for old kernels
netdev: Init flow api on already added ports on offload enable
Roi Dayan (2):
dpif: Refactor flow logging functions to be used by other modules
dpif-
On 24/04/2017 14:35, Simon Horman wrote:
On Tue, Apr 18, 2017 at 03:18:55PM +0300, Roi Dayan wrote:
On 14/04/2017 04:11, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
On 12/04/2017 20:04, 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 flows (default):
ovs-dpctl dump-flows
# to dump only flows that in kernel
On 14/04/2017 04:19, Joe Stringer wrote:
On 7 April 2017 at 06:13, Roi Dayan wrote:
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
On 14/04/2017 04:08, Joe Stringer wrote:
On 7 April 2017 at 06:12, 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.
Signed-off-by: Paul
On 14/04/2017 04:15, Joe Stringer wrote:
On 7 April 2017 at 06:13, Roi Dayan wrote:
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
On 14/04/2017 03:55, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
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
On 14/04/2017 03:51, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
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
On 14/04/2017 03:26, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
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
Missing the internals sub folder.
Signed-off-by: Roi Dayan
---
V2
- fix links but keep GitHub style as it's a top-level document
MAINTAINERS.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst
index 28831ab..60fa1f5 100644
On 18/04/2017 19:26, Stephen Finucane wrote:
On Tue, 2017-04-18 at 15:08 +0300, Roi Dayan wrote:
The links were pointing to static non-existent location instead
of internal doc link.
Signed-off-by: Roi Dayan
---
MAINTAINERS.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions
On 14/04/2017 04:11, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
diff --git a/lib/netdev.h b/lib/netdev.h
index d6c07c1..6d2db7d 100644
--- a/lib/netdev.h
+++ b/lib
On 14/04/2017 04:01, Joe Stringer wrote:
On 7 April 2017 at 06:12, Roi Dayan wrote:
From: Paul Blakey
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
@@ -769,6 +777,49 @@ struct netdev_class {
/* Discards all packets waiting to be received from
801 - 900 of 1063 matches
Mail list logo
