Re: [dpdk-dev] [PATCH v3] vhost: add unsafe API to check inflight packets

["Ding, Xuan" <xuan.ding@intel.com>]

Hi,

> -----Original Message-----
> From: Kevin Traynor <ktraynor@redhat.com>
> Sent: Wednesday, September 22, 2021 1:28 AM
> To: Ding, Xuan <xuan.ding@intel.com>; dev@dpdk.org;
> maxime.coquelin@redhat.com; Xia, Chenbo <chenbo.xia@intel.com>
> Cc: Hu, Jiayu <jiayu.hu@intel.com>; Jiang, Cheng1 <cheng1.jiang@intel.com>;
> Richardson, Bruce <bruce.richardson@intel.com>; Pai G, Sunil
> <sunil.pai.g@intel.com>; Yang, YvonneX <yvonnex.yang@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v3] vhost: add unsafe API to check inflight
> packets
> 
> On 16/09/2021 03:58, Xuan Ding wrote:
> > In async data path, when vring state changes, it is necessary to
> > know the number of inflight packets in DMA engine. This patch
> > provides a thread unsafe API to return the number of inflight
> > packets without using any lock.
> >
> > Signed-off-by: Xuan Ding <xuan.ding@intel.com>
> > ---
> > v3:
> > * Fixed one typo.
> > * Revised the doc to be more accuracy.
> >
> > v2:
> > * Fixed some format issues.
> > ---
> >   doc/guides/prog_guide/vhost_lib.rst    |  5 +++++
> >   doc/guides/rel_notes/release_21_11.rst |  5 +++++
> >   lib/vhost/rte_vhost_async.h            | 14 ++++++++++++++
> >   lib/vhost/version.map                  |  3 +++
> >   lib/vhost/vhost.c                      | 26 ++++++++++++++++++++++++++
> >   5 files changed, 53 insertions(+)
> >
> 
> <snip>
> 
> Should there be a change to the vring_state_changed() in vhost example
> app to go along with this patch? It would help to understand the
> operation if this API was used.

Thanks Kevin, will add it in vhost example in next version.

Regards,
Xuan