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
