summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2015-11-25 13:39:57 +0200
committerMichael S. Tsirkin <mst@redhat.com>2015-11-26 12:02:11 +0200
commit449e3578107799817a81249b189f6f82aa9e787d (patch)
treec660866e2c1e52bc2cf4efc5fe3f81d40e76aee7
parent7cf32491eac9dc32fd8ca7933f3edc9d42f884ee (diff)
downloadhqemu-449e3578107799817a81249b189f6f82aa9e787d.zip
hqemu-449e3578107799817a81249b189f6f82aa9e787d.tar.gz
Revert "vhost: send SET_VRING_ENABLE at start/stop"
This reverts commit 3a12f32229a046f4d4ab0a3a52fb01d2d5a1ab76. In case of live migration several queues can be enabled and not only the first one. So informing backend that only the first queue is enabled is wrong. Reported-by: Thibaut Collet <thibaut.collet@6wind.com> Cc: Yuanhan Liu <yuanhan.liu@linux.intel.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
-rw-r--r--hw/virtio/vhost.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index 1794f0d..de29968 100644
--- a/hw/virtio/vhost.c
+++ b/hw/virtio/vhost.c
@@ -1226,11 +1226,6 @@ int vhost_dev_start(struct vhost_dev *hdev, VirtIODevice *vdev)
}
}
- if (hdev->vhost_ops->vhost_set_vring_enable) {
- /* only enable first vq pair by default */
- hdev->vhost_ops->vhost_set_vring_enable(hdev, hdev->vq_index == 0);
- }
-
return 0;
fail_log:
vhost_log_put(hdev, false);
@@ -1261,10 +1256,6 @@ void vhost_dev_stop(struct vhost_dev *hdev, VirtIODevice *vdev)
hdev->vq_index + i);
}
- if (hdev->vhost_ops->vhost_set_vring_enable) {
- hdev->vhost_ops->vhost_set_vring_enable(hdev, 0);
- }
-
vhost_log_put(hdev, true);
hdev->started = false;
hdev->log = NULL;
OpenPOWER on IntegriCloud