Re: [dpdk-dev] [PATCH RFC 0/1] vhost-add-DMADEV-support-for-async-datapath

[Maxime Coquelin <maxime.coquelin@redhat.com>]

Hi Sunil,

On 8/23/21 11:53 AM, Sunil Pai G wrote:
> Note to the reader:
> -------------------
> The intent of this patch is to explore possible different approaches
> of async implementations.
> Please consider this patch for discussions only and not for
> merge/upstream.

Thanks for sharing, that will indeed help to find the best solution.
I'm just wondering if you have the vhost example part of this rework, so
that we can understand what are the impacts on app side?

> This patch simplifies the vhost async datapath usability by
> utilizing the generic DMADEV API's to perform packet copy.
> 
> Previously, it was required by the application to implement its
> own DMA enabling logic making it difficult to adopt this feature.
> Having a common implementation in vhost library allows
> for easier adoption.
> 
> The usability of async datapath is enhanced by extending the
> API's to include a DMADEV ID to be passed by the application.
> This provides flexibility to applications to decide which DMADEV to be used.
> 
> 
> Sunil Pai G (1):
>   vhost: Add DMADEV support for async datapath.
> 
>  lib/vhost/meson.build       |   2 +-
>  lib/vhost/rte_vhost_async.h |  55 +------
>  lib/vhost/vhost.c           |  46 +++---
>  lib/vhost/vhost.h           |  24 ++-
>  lib/vhost/virtio_net.c      | 311 +++++++++++++++++++++++++++++++-----
>  5 files changed, 316 insertions(+), 122 deletions(-)
>