On Thu, 7 Dec 2023 at 09:55, Albert Esteve <aest...@redhat.com> wrote: > > v1: https://www.mail-archive.com/qemu-devel@nongnu.org/msg1005257.html > v1 -> v2: > - Solved an unitialized uuid value on vhost-user source > - Changed cleanup strategy, and traverse all objects in the > table to remove them instead.
Please update the vhost-user specification (docs/interop/vhost-user.rst) so people implementing front-ends and back-ends are aware that only the back-end that added a shared resource can remove it. Acked-by: Stefan Hajnoczi <stefa...@redhat.com> > > Various improvements for the virtio-dmabuf module. > This patch includes: > > - Check for ownership before allowing a vhost device > to remove an object from the table. > - Properly cleanup shared resources if a vhost device > object gets cleaned up. > - Rename virtio dmabuf functions to `virtio_dmabuf_*` > > Albert Esteve (3): > hw/virtio: check owner for removing objects > hw/virtio: cleanup shared resources > hw/virtio: rename virtio dmabuf API > > hw/display/virtio-dmabuf.c | 36 ++++++++++++--- > hw/virtio/vhost-user.c | 31 ++++++++++--- > hw/virtio/vhost.c | 3 ++ > include/hw/virtio/virtio-dmabuf.h | 43 ++++++++++------- > tests/unit/test-virtio-dmabuf.c | 77 ++++++++++++++++++++++--------- > 5 files changed, 138 insertions(+), 52 deletions(-) > > -- > 2.43.0 >