On Mon, Dec 11, 2023 at 10:48 PM Stefan Hajnoczi <stefa...@gmail.com> wrote:

> 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>
>
>
Will do :)

Should I add the `Acked-by` manually to the commits or does it
get automatically picked up later?

BR,
Albert

PS: Happy New Year!


> >
> > 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
> >
>
>

Reply via email to