[ovs-dev] [PATCH per-port ingress scheduling 2/2] ingress scheduling: Provide per interface ingress priority

2018-08-28 Thread Billy O'Mahony
interfaces from packet loss as PMDs get overloaded. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 + lib/dpif-netdev.c | 188 +--- lib/netdev-dpdk.c | 10 +++ 3 files changed, 170 insertions(+), 31 dele

[ovs-dev] [PATCH per-port ingress scheduling 1/2] ingress scheduling: documentation

2018-08-28 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 15 +++ vswitchd/vswitch.xml | 15 +++ 2 files changed, 30 insertions(+) diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst index ab3d576..83284e7 100644 --- a/Document

[ovs-dev] [PATCH per-port ingress scheduling 0/2]

2018-08-28 Thread Billy O'Mahony
configuration in Interfaces other_config * applies cleanly on 9b4f08c RFCv2: * Keep ingress prio config in netdev base rather than in each netdev type. * Account for differing rxq lengths * Applies cleanly to 4299145 RFCv1: Initial version. Billy O'Mahony (2): ingress scheduling: documentatio

[ovs-dev] [PATCH v3] OVS-DPDK: Change "dpdk-socket-mem" default value.

2018-05-09 Thread Billy O'Mahony
ntation update. Signed-off-by: Marcin Rybka Co-authored-by: Billy O'Mahony Signed-off-by: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 3 ++- lib/dpdk.c | 28 +++- vswitchd/vswitch.xml | 7 --- 3 file

[ovs-dev] [PATCH v3] OVS-DPDK: Change "dpdk-socket-mem" default value

2018-05-09 Thread Billy O'Mahony
Hi All, This change is the outcome of this discussion: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-April/044110.html. I am posting this on behalf of Marcin who is the author but no longer works for Intel. I made a few changes hence the double sign-off but please remove my sig if requ

[ovs-dev] [PATCH v2] OVS-DPDK: Change "dpdk-socket-mem" default value.

2018-05-08 Thread Billy O'Mahony
ntation update. Signed-off-by: Marcin Rybka Co-authored-by:: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 3 ++- lib/dpdk.c | 29 - vswitchd/vswitch.xml | 7 --- 3 files changed, 34 insertions(+), 5 delet

[ovs-dev] [PATCH v2] OVS-DPDK: Change "dpdk-socket-mem" default value

2018-05-08 Thread Billy O'Mahony
Hi All, This change is teh outcome of this discussion: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-April/044110.html. I am posting this on behalf of Marcin who is the author but no longer works for Intel. I made a few changes hence the double sign-off but please remove my sig if r

[ovs-dev] [PATCH] OVS-DPDK: Change "dpdk-socket-mem" default value.

2018-05-08 Thread Billy O'Mahony
ntation update. Signed-off-by: Marcin Rybka Signed-off-by: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 3 ++- lib/dpdk.c | 35 ++- vswitchd/vswitch.xml | 7 --- 3 files changed, 40 insertio

[ovs-dev] [PATCH] Change "dpdk-socket-mem" default value.

2018-05-08 Thread Billy O'Mahony
Hi All, This change is teh outcome of this discussion: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-April/044110.html. I am posting this on behalf of Marcin who is the author but no longer works for Intel. I made a few changes hence the double sign-off but please remove my sig if r

[ovs-dev] [RFC v2 2/2] ingress scheduling: Provide per interface ingress priority

2018-03-28 Thread Billy O'Mahony
from loss and latency as PMDs get overloaded. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 ++ lib/dpif-netdev.c | 103 +++- lib/netdev-bsd.c| 1 + lib/netdev-dpdk.c | 13 -

[ovs-dev] [RFC v2 1/2] ingress scheduling: schema and docs

2018-03-28 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 18 ++ vswitchd/vswitch.ovsschema | 9 +++-- vswitchd/vswitch.xml | 40 3 files changed, 65 insertions(+), 2 deletions(-) diff --git a/Documentation/

[ovs-dev] [RFC v2 0/2] Ingress Scheduling

2018-03-28 Thread Billy O'Mahony
rio config in netdev base rather than in each netdev type. * Account for differing rxq lengths * Applies clean to 4299145 Billy O'Mahony (2): ingress scheduling: schema and docs ingress scheduling: Provide per interface ingress priority Documentation/howto/dpdk.rst| 18 +++

[ovs-dev] [RFC 2/2] ingress scheduling: Provide per interface ingress priority

2018-02-16 Thread Billy O'Mahony
from loss and latency as PMDs get overloaded. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 ++ lib/dpif-netdev.c | 47 +- lib/netdev-bsd.c| 1 + lib/netdev-dpdk.c

[ovs-dev] [RFC 1/2] ingress scheduling: schema and docs

2018-02-16 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 18 ++ vswitchd/vswitch.ovsschema | 9 +++-- vswitchd/vswitch.xml | 40 3 files changed, 65 insertions(+), 2 deletions(-) diff --git a/Documentation/

[ovs-dev] [RFC 0/2] Ingress Scheduling

2018-02-16 Thread Billy O'Mahony
f 1.1747.0) Intel(R) Xeon(R) CPU E5-2695 v3 @ 2.30GHz One pmd Port fwding port 0<->1, 2 <-> 3 Frame 64B, UDP 221 streams per port. OvS base - 4c80644 http://github.com/istokes/ovs dpdk_merge_2_9. Added 600cycles approx pkt processing in order to bring per packet cost to ~1000 cycles

[ovs-dev] [RFC 1/2] dpif-netdev: Fix per packet cycles statistics.

2017-09-22 Thread Billy O'Mahony
From: Ilya Maximets DP_STAT_LOOKUP_HIT statistics used mistakenly for calculation of total number of packets. This leads to completely wrong per packet cycles statistics. For example: emc hits:0 megaflow hits:253702308 avg. subtable lookups per hit:1.50 miss:0

[ovs-dev] [RFC 2/2] dpif-netdev: RFC EMC load-shedding

2017-09-22 Thread Billy O'Mahony
When EMC hit rate goes down start start shedding load from the EMC. --- lib/dpif-netdev.c | 107 -- 1 file changed, 103 insertions(+), 4 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index e3a5590..f77e79a 100644 --- a/lib/dpif

[ovs-dev] [RFC 0/2] EMC load-shedding

2017-09-22 Thread Billy O'Mahony
out how to produce it for testing :). [1] https://mail.openvswitch.org/pipermail/ovs-dev/2017-August/336509.html [2] https://mail.openvswitch.org/pipermail/ovs-dev/2017-September/338380.html [3] https://mail.openvswitch.org/pipermail/ovs-dev/2017-August/337309.html Billy O'Mahony (1): dpif-netdev: R

[ovs-dev] [PATCH v2 4/4] docs: Document ingress scheduling feature

2017-08-17 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 31 +++ vswitchd/vswitch.xml | 31 +++ 2 files changed, 62 insertions(+) diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst index d7

[ovs-dev] [PATCH v2 3/4] dpif-netdev: Add rxq prioritization

2017-08-17 Thread Billy O'Mahony
If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) d

[ovs-dev] [PATCH v2 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

2017-08-17 Thread Billy O'Mahony
Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 + lib/dpif-netdev.c | 1 + lib/netdev-d

[ovs-dev] [PATCH v2 1/4] netdev: Add set_ingress_sched to netdev api

2017-08-17 Thread Billy O'Mahony
Passes ingress_sched config item from other_config column of Interface table to the netdev. Signed-off-by: Billy O'Mahony --- lib/netdev-bsd.c | 1 + lib/netdev-dpdk.c | 21 + lib/netdev-dummy.c| 1 + lib/netdev-linux.c| 1 + lib/netdev-provider.h

[ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

2017-08-17 Thread Billy O'Mahony
le) Regards, Billy. billy O'Mahony (4): netdev: Add set_ingress_sched to netdev api netdev-dpdk: Apply ingress_sched config to dpdk phy ports dpif-netdev: Add rxq prioritization docs: Document ingress scheduling feature Documentation/howto/dpdk.rst| 31 +++ include/openv

[ovs-dev] [PATCH 3/4] dpif-netdev: Add rxq prioritization

2017-07-20 Thread Billy O'Mahony
If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) d

[ovs-dev] [PATCH 4/4] docs: Document ingress scheduling feature

2017-07-20 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 31 +++ vswitchd/vswitch.xml | 31 +++ 2 files changed, 62 insertions(+) diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst index af

[ovs-dev] [PATCH 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

2017-07-20 Thread Billy O'Mahony
Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 + lib/dpif-netdev.c | 1 + lib/netdev-d

[ovs-dev] [PATCH 1/4] netdev: Add set_ingress_sched to netdev api

2017-07-20 Thread Billy O'Mahony
Passes ingress_sched config item from other_config column of Interface table to the netdev. Signed-off-by: Billy O'Mahony --- lib/netdev-bsd.c | 1 + lib/netdev-dpdk.c | 19 +++ lib/netdev-dummy.c| 1 + lib/netdev-linux.c| 1 + lib/netdev-provider.h

[ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

2017-07-20 Thread Billy O'Mahony
|* 1e-8|* |*|*|*|* |* 0-1 1-20 20-40 40-50 50-60 60-70 ... 120-400 Latency (us) Proportion of packets per latency bin @ 80% Max Throughput (Log scale) Regards, Billy. Billy O'Mahony (4): netdev: Add set_ingre

[ovs-dev] [RFC v2 4/4] docs: Document ingress scheduling feature

2017-07-11 Thread Billy O'Mahony
Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 31 +++ vswitchd/vswitch.xml | 31 +++ 2 files changed, 62 insertions(+) diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst index 93

[ovs-dev] [RFC v2 3/4] dpif-netdev: Add rxq prioritization

2017-07-11 Thread Billy O'Mahony
If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-) d

[ovs-dev] [RFC v2 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

2017-07-11 Thread Billy O'Mahony
Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the possible ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 ++ lib/dpif-netdev.c | 1 +

[ovs-dev] [RFC v2 1/4] netdev: Add set_ingress_sched to netdev api

2017-07-11 Thread Billy O'Mahony
Passes ingress_sched config item from other_config column of Interface table to the netdev. Signed-off-by: Billy O'Mahony --- lib/netdev-bsd.c | 1 + lib/netdev-dpdk.c | 19 +++ lib/netdev-dummy.c| 1 + lib/netdev-linux.c| 1 + lib/netdev-provider.h

[ovs-dev] [RFC v2 0/4] prioritizing latency sensitive traffic

2017-07-11 Thread Billy O'Mahony
hings it does not take into account: * reconfiguration - of the ingress_sched config of pmd masks and so on. * ovs-appctl dpif-netdev/pmd-*-show o/p should indicate rxq prioritization. * any requirements for a more granular prioritization mechanism * ... Regards, Billy. Billy O'Mahony

[ovs-dev] [PATCH v10] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-07-10 Thread Billy O'Mahony
with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony Signed-off-by: Ilya Maximets Co-authored-by: Ilya Maximets Tested-by: Ian Stokes Acked-by: Ian Stokes --- v10: tweak code comments, docs based on reviews v9: v8 missed some comments on v7

[ovs-dev] [PATCH v9] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-06-29 Thread Billy O'Mahony
with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony Signed-off-by: Ilya Maximets Co-authored-by: Ilya Maximets --- v9: v8 missed some comments on v7 v8: Some coding style issues; doc tweak v7: Incorporate review comments on docs and implementa

[ovs-dev] [PATCH v8] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-06-28 Thread Billy O'Mahony
with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony Signed-off-by: Ilya Maximets Co-authored-by: Ilya Maximets --- v8: Some coding style issues; doc tweak v7: Incorporate review comments on docs and implementation v6: Change 'port' to

[ovs-dev] [PATCH v7] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-06-27 Thread Billy O'Mahony
with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony Signed-off-by: Ilya Maximets Co-authored-by: Ilya Maximets --- v7: Incorporate review comments on docs and implementation v6: Change 'port' to 'queue' in a warning msg v5: F

[ovs-dev] [PATCH v7] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-06-27 Thread Billy O'Mahony
I have incorporated Darrell's doc comments and Ilya's implementation and tested with various scenarios. I had to make a few small changes to ensure that the warning messages were issued for each queue as per previous patch comments. Billy O'Mahony (1): dpif-netdev: Assign ports

[ovs-dev] [RFC PATCH 3/3] dpif-netdev: Add rxq prioritization

2017-06-16 Thread Billy O'Mahony
If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-) d

[ovs-dev] [RFC PATCH 2/3] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

2017-06-16 Thread Billy O'Mahony
Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the possible ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 ++ lib/dpif-netdev.c | 1 +

[ovs-dev] [RFC PATCH 0/3] prioritizing latency sensitive traffic

2017-06-16 Thread Billy O'Mahony
-*-show o/p should indicate rxq prioritization. * any requirements for a more granular prioritization mechanism * ... * Regards, Billy. Billy O'Mahony (3): netdev: Add set_ingress_sched to netdev api netdev-dpdk: Apply ingress_sched config to dpdk phy ports dpif-netdev: Add rxq

[ovs-dev] [RFC PATCH 1/3] netdev: Add set_ingress_sched to netdev api

2017-06-16 Thread Billy O'Mahony
Passes ingress_sched config item from other_config column of Interface table to the netdev. Signed-off-by: Billy O'Mahony --- lib/netdev-bsd.c | 1 + lib/netdev-dpdk.c | 19 +++ lib/netdev-dummy.c| 1 + lib/netdev-linux.c| 1 + lib/netdev-provider.h

[ovs-dev] [PATCH v6] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-05-10 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- v6: Change 'port' to 'queue' in a warning msg v5: Fix warning msg; Update same in docs v4: Fix a checkpatch error v3: Fix warning messages not appearing wh

[ovs-dev] [PATCH v5] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-05-10 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 10 + lib/dpif-netdev.c| 43 +++- 2 files changed, 48 insertions(+), 5 del

[ovs-dev] [PATCH v4] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-05-05 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- v4: Fix a checkpatch error v3: Fix warning messages not appearing when using multiqueue v2: Add details of warning messages into docs Documentation/intro/instal

[ovs-dev] [PATCH v3] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-05-04 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 10 + lib/dpif-netdev.c| 39 2 files changed, 45 insertions(+), 4 del

[ovs-dev] [PATCH v2] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-03-30 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- Documentation/intro/install/dpdk.rst | 10 ++ lib/dpif-netdev.c| 36 2 files changed, 42 insertions(+), 4 del

[ovs-dev] [PATCH] dpif-netdev: emc comments

2017-03-30 Thread Billy O'Mahony
Add a concrete example of how a flow's hash determines the set of possible storage locations in the EMC. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netd

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

2017-03-01 Thread Billy O'Mahony
This gives much better performance for linux apps in the guest without affecting dpdk applications in the guest. Test details in cover letter. Signed-off-by: Billy O'Mahony --- lib/netdev-dpdk.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/netdev-dpdk.c b/lib/n

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

2017-03-01 Thread Billy O'Mahony
ranch but before release), other versions not tested. DPDK v16.11 QEMU v2.5.0 (also seen with v2.7.1) Results: INDIRECT_DESC enabled2.75 Mpps @64B pkts (0.176 Gbit/s) INDIRECT_DESC disabled 2.75 Mpps @64B pkts (0.176 Gbit/s) Billy O'Mahony (1): netdev-dpdk: Enable INDIRE

[ovs-dev] [PATCH] dpif-netdev: Assign ports to pmds on non-local numa node.

2017-02-28 Thread Billy O'Mahony
numa node albeit with lower performance. Local pmds are still chosen when available. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index 30907b7.