mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-08 02:03:56 -06:00
vhost: allocate SVQ device file descriptors at device start
The next patches will start control SVQ if possible. However, we don't know if that will be possible at qemu boot anymore. Delay device file descriptors until we know it at device start. This will avoid to create them if the device does not support SVQ. Signed-off-by: Eugenio Pérez <eperezma@redhat.com> Acked-by: Jason Wang <jasowang@redhat.com> Message-Id: <20221215113144.322011-4-eperezma@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
parent
20e7412bfd
commit
3cfb4d069c
2 changed files with 30 additions and 36 deletions
|
@ -428,15 +428,11 @@ static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
|
|||
|
||||
shadow_vqs = g_ptr_array_new_full(hdev->nvqs, vhost_svq_free);
|
||||
for (unsigned n = 0; n < hdev->nvqs; ++n) {
|
||||
g_autoptr(VhostShadowVirtqueue) svq;
|
||||
VhostShadowVirtqueue *svq;
|
||||
|
||||
svq = vhost_svq_new(v->iova_tree, v->shadow_vq_ops,
|
||||
v->shadow_vq_ops_opaque);
|
||||
if (unlikely(!svq)) {
|
||||
error_setg(errp, "Cannot create svq %u", n);
|
||||
return -1;
|
||||
}
|
||||
g_ptr_array_add(shadow_vqs, g_steal_pointer(&svq));
|
||||
g_ptr_array_add(shadow_vqs, svq);
|
||||
}
|
||||
|
||||
v->shadow_vqs = g_steal_pointer(&shadow_vqs);
|
||||
|
@ -864,11 +860,23 @@ static int vhost_vdpa_svq_set_fds(struct vhost_dev *dev,
|
|||
const EventNotifier *event_notifier = &svq->hdev_kick;
|
||||
int r;
|
||||
|
||||
r = event_notifier_init(&svq->hdev_kick, 0);
|
||||
if (r != 0) {
|
||||
error_setg_errno(errp, -r, "Couldn't create kick event notifier");
|
||||
goto err_init_hdev_kick;
|
||||
}
|
||||
|
||||
r = event_notifier_init(&svq->hdev_call, 0);
|
||||
if (r != 0) {
|
||||
error_setg_errno(errp, -r, "Couldn't create call event notifier");
|
||||
goto err_init_hdev_call;
|
||||
}
|
||||
|
||||
file.fd = event_notifier_get_fd(event_notifier);
|
||||
r = vhost_vdpa_set_vring_dev_kick(dev, &file);
|
||||
if (unlikely(r != 0)) {
|
||||
error_setg_errno(errp, -r, "Can't set device kick fd");
|
||||
return r;
|
||||
goto err_init_set_dev_fd;
|
||||
}
|
||||
|
||||
event_notifier = &svq->hdev_call;
|
||||
|
@ -876,8 +884,18 @@ static int vhost_vdpa_svq_set_fds(struct vhost_dev *dev,
|
|||
r = vhost_vdpa_set_vring_dev_call(dev, &file);
|
||||
if (unlikely(r != 0)) {
|
||||
error_setg_errno(errp, -r, "Can't set device call fd");
|
||||
goto err_init_set_dev_fd;
|
||||
}
|
||||
|
||||
return 0;
|
||||
|
||||
err_init_set_dev_fd:
|
||||
event_notifier_set_handler(&svq->hdev_call, NULL);
|
||||
|
||||
err_init_hdev_call:
|
||||
event_notifier_cleanup(&svq->hdev_kick);
|
||||
|
||||
err_init_hdev_kick:
|
||||
return r;
|
||||
}
|
||||
|
||||
|
@ -1089,6 +1107,9 @@ static void vhost_vdpa_svqs_stop(struct vhost_dev *dev)
|
|||
for (unsigned i = 0; i < v->shadow_vqs->len; ++i) {
|
||||
VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, i);
|
||||
vhost_vdpa_svq_unmap_rings(dev, svq);
|
||||
|
||||
event_notifier_cleanup(&svq->hdev_kick);
|
||||
event_notifier_cleanup(&svq->hdev_call);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue