[ovs-dev] [PATCH v2 0/2] vHost Dequeue Zero Copy

Jan Scheurich jan.scheurich at ericsson.com
Wed Oct 11 16:23:09 UTC 2017


Hi Ciara,

These improvements look very good. I would expect even bigger improvements for big packets, as long as we don't hit some link bandwidth limitations. But at least the vhost-vhost cases should benefit.

Have you also tested larger packet sizes?

I plan to review your patches.

Thanks, Jan

> -----Original Message-----
> From: ovs-dev-bounces at openvswitch.org [mailto:ovs-dev-bounces at openvswitch.org] On Behalf Of Ciara Loftus
> Sent: Wednesday, 11 October, 2017 16:22
> To: dev at openvswitch.org
> Subject: [ovs-dev] [PATCH v2 0/2] vHost Dequeue Zero Copy
> 
> This patch enables optional dequeue zero copy for vHost ports.
> This gives a performance increase for some use cases. I'm using
> the cover letter to report my results.
> 
> vhost (vm1) -> vhost (vm2)
> Using testpmd to source (txonly) in vm1 and sink (rxonly) in vm2.
> 4C1Q 64B packets: 5.05Mpps -> 5.52Mpps = 9.2% improvement
> 
> vhost (virtio_user backend 1) -> vhost (virtio_user backend 2)
> Using 2 instances of testpmd, each with a virtio_user backend
> connected to one of the two vhost ports created in OVS.
> 2C1Q 1518B packets: 2.59Mpps -> 3.09Mpps = 19.3% improvement
> 
> vhost -> phy
> Using testpmd to source (txonly) and sink in the NIC
> 1C1Q 64B packets: 6.81Mpps -> 7.76Mpps = 13.9% improvement
> 
> phy -> vhost -> phy
> No improvement measured
> 
> This patch is dependent on the series below which fixes issues with
> mempool management:
> https://patchwork.ozlabs.org/patch/822590/
> 
> v2 changes:
> * Mention feature is disabled by default in the documentation
> * Add PHY-VM-PHY with vHost dequeue zero copy documentation guide
> * Line wrap link to DPDK documentation
> * Rename zc_enabled to dq_zc_enabled for future-proofing
> * Mention feature is available for both vHost port types in the docs
> * In practise, rebooting the VM doesn't always enable the feature if
> enabled post-boot, so update the documentation to suggest a shutdown
> rather than a reboot. The reason why this doesn't work is probably
> because the total downtime during reboot isn't enough to allow a vhost
> device unregister & re-register with the new feature, so when the VM
> starts again it doesn't pick up the new device as it hasn't been
> re-registered in time.
> 
> Ciara Loftus (2):
>   netdev-dpdk: Helper function for vHost device setup
>   netdev-dpdk: Enable optional dequeue zero copy for vHost User
> 
>  Documentation/howto/dpdk.rst             |  29 +++++
>  Documentation/topics/dpdk/vhost-user.rst |  35 ++++++
>  NEWS                                     |   3 +
>  lib/netdev-dpdk.c                        | 202 +++++++++++++++++++++----------
>  vswitchd/vswitch.xml                     |  11 ++
>  5 files changed, 218 insertions(+), 62 deletions(-)
> 
> --
> 2.7.5
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list