diff options
| author | Eugenio Pérez <eperezma@redhat.com> | 2022-08-23 20:20:05 +0200 |
|---|---|---|
| committer | Jason Wang <jasowang@redhat.com> | 2022-09-02 10:22:39 +0800 |
| commit | b37c12be962f95fd1e93b470a5ff05f6e2035d46 (patch) | |
| tree | 2e6a9c142b75891e0d2f61869fc2be2439467c0e /hw/virtio | |
| parent | 69292a8e40f4dae8af5f04724e06392cdf03c09e (diff) | |
| download | qemu-b37c12be962f95fd1e93b470a5ff05f6e2035d46.zip qemu-b37c12be962f95fd1e93b470a5ff05f6e2035d46.tar.gz qemu-b37c12be962f95fd1e93b470a5ff05f6e2035d46.tar.bz2 | |
vdpa: Remove SVQ vring from iova_tree at shutdown
Although the device will be reset before usage, the right thing to do is
to clean it.
Reported-by: Lei Yang <leiyang@redhat.com>
Fixes: 34e3c94eda ("vdpa: Add custom IOTLB translations to SVQ")
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'hw/virtio')
| -rw-r--r-- | hw/virtio/vhost-vdpa.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c index 87e0ad3..e16e0e2 100644 --- a/hw/virtio/vhost-vdpa.c +++ b/hw/virtio/vhost-vdpa.c @@ -898,6 +898,12 @@ static bool vhost_vdpa_svq_unmap_ring(struct vhost_vdpa *v, size = ROUND_UP(result->size, qemu_real_host_page_size()); r = vhost_vdpa_dma_unmap(v, result->iova, size); + if (unlikely(r < 0)) { + error_report("Unable to unmap SVQ vring: %s (%d)", g_strerror(-r), -r); + return false; + } + + vhost_iova_tree_remove(v->iova_tree, *result); return r == 0; } |