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.

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?

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to