> -----Original Message----- > From: dev [mailto:[email protected]] On Behalf Of Tomasz Kulasek > Sent: Saturday, February 10, 2018 1:28 AM > To: [email protected] > Cc: [email protected]; Xie, Huawei; [email protected]; Wodkowski, PawelX > Subject: [dpdk-dev] [PATCH] vhost: fix return avail ring position in get vring > base > > According to the "Vhost-user Protocol" document, > VHOST_USER_GET_VRING_BASE should get the available vring base offset. > > Fixes: 8f972312b8f4 ("vhost: support vhost-user") > Cc: [email protected] > Cc: [email protected] > > Signed-off-by: Pawel Wodkowski <[email protected]> > Signed-off-by: Tomasz Kulasek <[email protected]>
Reviewed-by: Jianfeng Tan <[email protected]> Nice catch! > --- > lib/librte_vhost/vhost_user.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c > index 65ee33919..04eee3a3a 100644 > --- a/lib/librte_vhost/vhost_user.c > +++ b/lib/librte_vhost/vhost_user.c > @@ -881,8 +881,8 @@ vhost_user_get_vring_base(struct virtio_net *dev, > > dev->flags &= ~VIRTIO_DEV_READY; > > - /* Here we are safe to get the last used index */ > - msg->payload.state.num = vq->last_used_idx; > + /* Here we are safe to get the last avail index */ > + msg->payload.state.num = vq->last_avail_idx; > > RTE_LOG(INFO, VHOST_CONFIG, > "vring base idx:%d file:%d\n", msg->payload.state.index, > -- > 2.14.1
