On 1/17/22 18:44, Ilya Maximets wrote:
On 1/5/22 09:19, Maxime Coquelin wrote:
This series introduces a new hash-based Tx packets steering
mode alognside existing XPS and static modes. The goal is
to provide a mode where all the transmit queues are used,
whatever the number of PMD threads. This may be used with
Vhost-user ports, where the guest application driving the
Virtio device expects packets to be distributed on all the
queues.
As a preliminary step, in order to be able to validate the
feature at OVS level, the two first patches introduce
per-queue basic statistics for Vhost-user and dummy ports.
These patches are complementary to David's patch [0] adding
per-queue statistics to DPDK ports using xstats.
The series also introduces a dpif-netdev test for tx
steering, using dummy-pmd ports only, instead of the more
complex Vhost-user test in first revision.
Changes in v5:
==============
- Fix indents (David)
- Fix/remove comments (David)
- Fix grammar in documentation (David)
- Rebase and applied David's R-by's
Maxime Coquelin (5):
netdev-dpdk: Introduce per rxq/txq Vhost-user statistics.
netdev-dummy: Introduce per rxq/txq statistics.
dpif-netdev: Introduce Tx queue mode.
dpif-netdev: Introduce hash-based Tx packet steering mode.
dpif-netdev.at: Add test for Tx packets steering.
Documentation/automake.mk | 1 +
Documentation/topics/index.rst | 1 +
.../topics/userspace-tx-steering.rst | 73 +++++++++
lib/dpif-netdev.c | 142 ++++++++++++++---
lib/netdev-dpdk.c | 147 ++++++++++++++++--
lib/netdev-dummy.c | 87 +++++++++--
tests/dpif-netdev.at | 67 ++++++++
tests/ofproto.at | 3 +-
8 files changed, 477 insertions(+), 44 deletions(-)
create mode 100644 Documentation/topics/userspace-tx-steering.rst
Thanks, Maxime and David!
With some cosmetic changes I applied the series, except for the
first patch (vhost-user statistics), because it causes performance
degradation for me. I'll reply with details to that patch.
Thanks, I agree with skipping the Vhost-user stats patch for now.
And I just noticed that we're missing the documentation update
for vswitchd/vswitch.xml. Could you, please, update it with
a new tx-steering option and send as a separate patch?
I just sent the documentation patch adding the new PMD option.
Regards,
Maxime
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
