summaryrefslogtreecommitdiffstats
path: root/hw
diff options
context:
space:
mode:
authorEugenio Pérez2021-11-04 09:56:24 +0100
committerMichael S. Tsirkin2021-11-11 09:13:05 +0100
commit245cf2c24eb0d5a4fe75151e1794aa5cd53b130d (patch)
treeea4a73d3b5fff36ab56f167309941663fe095aa0 /hw
parentsoftmmu/qdev-monitor: fix use-after-free in qdev_set_id() (diff)
downloadqemu-245cf2c24eb0d5a4fe75151e1794aa5cd53b130d.tar.gz
qemu-245cf2c24eb0d5a4fe75151e1794aa5cd53b130d.tar.xz
qemu-245cf2c24eb0d5a4fe75151e1794aa5cd53b130d.zip
vhost: Rename last_index to vq_index_end
The doc of this field pointed out that last_index is the last vq index. This is misleading, since it's actually one past the end of the vqs. Renaming and modifying comment. Signed-off-by: Eugenio Pérez <eperezma@redhat.com> Acked-by: Jason Wang <jasowang@redhat.com> Message-Id: <20211104085625.2054959-2-eperezma@redhat.com> Reviewed-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw')
-rw-r--r--hw/net/vhost_net.c4
-rw-r--r--hw/virtio/vhost-vdpa.c2
2 files changed, 3 insertions, 3 deletions
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index 0d888f29a6..29f2c4212f 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -232,10 +232,10 @@ fail:
}
static void vhost_net_set_vq_index(struct vhost_net *net, int vq_index,
- int last_index)
+ int vq_index_end)
{
net->dev.vq_index = vq_index;
- net->dev.last_index = last_index;
+ net->dev.vq_index_end = vq_index_end;
}
static int vhost_net_start_one(struct vhost_net *net,
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
index 0d8051426c..bcaf00e09f 100644
--- a/hw/virtio/vhost-vdpa.c
+++ b/hw/virtio/vhost-vdpa.c
@@ -645,7 +645,7 @@ static int vhost_vdpa_dev_start(struct vhost_dev *dev, bool started)
vhost_vdpa_host_notifiers_uninit(dev, dev->nvqs);
}
- if (dev->vq_index + dev->nvqs != dev->last_index) {
+ if (dev->vq_index + dev->nvqs != dev->vq_index_end) {
return 0;
}