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
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
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
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
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
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
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
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
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
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 -
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/
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 +++
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
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/
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
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
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
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
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
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
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
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
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
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
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
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
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
|*
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
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
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
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 +
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
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
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
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
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
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
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
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
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 +
-*-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
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
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
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
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
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
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
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
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
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
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.
51 matches
Mail list logo