[dpdk-dev] [PATCH 3/6] vhost: introduce last avail idx for Tx

Maxime Coquelin maxime.coquelin at redhat.com
Tue Aug 23 14:27:32 CEST 2016



On 08/23/2016 10:10 AM, Yuanhan Liu wrote:
> So far, we retrieve both the used ring avail ring idx by last_used_idx
> var; it won't be a problem because we used ring is updated immediately
> after those avail entries are consumed.
>
> But that's not true when Tx zero copy is enabled, that used ring is updated
> only when the mbuf is consumed. Thus, we need use another var to note
> the last avail ring idx we have consumed.
>
> Therefore, last_avail_idx is introduced.
>
> Signed-off-by: Yuanhan Liu <yuanhan.liu at linux.intel.com>
> ---
>  lib/librte_vhost/vhost.h      |  2 +-
>  lib/librte_vhost/virtio_net.c | 19 +++++++++++--------
>  2 files changed, 12 insertions(+), 9 deletions(-)

Looks good to me:
Reviewed-by: Maxime Coquelin <maxime.coquelin at redhat.com>

Thanks,
Maxime


More information about the dev mailing list