1. Patchew
  2. QEMU
  3. View series

[PATCH 0/3] Virtio dmabuf improvements

Albert Esteve posted 3 patches 1 year ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231107093744.388099-1-aesteve@redhat.com
Maintainers: Albert Esteve <aesteve@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>
There is a newer version of this series
hw/display/virtio-dmabuf.c        | 14 +++++-----
hw/virtio/vhost-user.c            | 33 ++++++++++++++++++-----
hw/virtio/vhost.c                 |  5 ++++
include/hw/virtio/vhost.h         |  6 +++++
include/hw/virtio/virtio-dmabuf.h | 33 ++++++++++++-----------
tests/unit/test-virtio-dmabuf.c   | 44 +++++++++++++++----------------
6 files changed, 83 insertions(+), 52 deletions(-)
[PATCH 0/3] Virtio dmabuf improvements
Posted by Albert Esteve 1 year ago 
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        | 14 +++++-----
 hw/virtio/vhost-user.c            | 33 ++++++++++++++++++-----
 hw/virtio/vhost.c                 |  5 ++++
 include/hw/virtio/vhost.h         |  6 +++++
 include/hw/virtio/virtio-dmabuf.h | 33 ++++++++++++-----------
 tests/unit/test-virtio-dmabuf.c   | 44 +++++++++++++++----------------
 6 files changed, 83 insertions(+), 52 deletions(-)

-- 
2.41.0
Re: [PATCH 0/3] Virtio dmabuf improvements
Posted by Albert Esteve 12 months ago 
On Tue, Nov 7, 2023 at 10:37 AM Albert Esteve <aesteve@redhat.com> wrote:

> 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        | 14 +++++-----
>  hw/virtio/vhost-user.c            | 33 ++++++++++++++++++-----
>  hw/virtio/vhost.c                 |  5 ++++
>  include/hw/virtio/vhost.h         |  6 +++++
>  include/hw/virtio/virtio-dmabuf.h | 33 ++++++++++++-----------
>  tests/unit/test-virtio-dmabuf.c   | 44 +++++++++++++++----------------
>  6 files changed, 83 insertions(+), 52 deletions(-)
>
> --
> 2.41.0
>
>
Bump :)

@Marc-André Lureau <marcandre.lureau@gmail.com> could you please take a
look? You suggested the API upgrades, so would be great if you could check
if it is what you had in mind.

Thanks!
Re: [PATCH 0/3] Virtio dmabuf improvements
Posted by Michael S. Tsirkin 11 months, 4 weeks ago 
On Thu, Nov 30, 2023 at 04:49:35PM +0100, Albert Esteve wrote:
> 
> 
> 
> On Tue, Nov 7, 2023 at 10:37 AM Albert Esteve <aesteve@redhat.com> wrote:
> 
>     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        | 14 +++++-----
>      hw/virtio/vhost-user.c            | 33 ++++++++++++++++++-----
>      hw/virtio/vhost.c                 |  5 ++++
>      include/hw/virtio/vhost.h         |  6 +++++
>      include/hw/virtio/virtio-dmabuf.h | 33 ++++++++++++-----------
>      tests/unit/test-virtio-dmabuf.c   | 44 +++++++++++++++----------------
>      6 files changed, 83 insertions(+), 52 deletions(-)
> 
>     --
>     2.41.0
> 
> 
> 
> Bump :)
> 
> @Marc-André Lureau could you please take a look? You suggested the API
> upgrades, so would be great if you could check if it is what you had in mind.
> 
> Thanks!

All this is post releas material, right?
Re: [PATCH 0/3] Virtio dmabuf improvements
Posted by Albert Esteve 11 months, 4 weeks ago 
On Mon, Dec 4, 2023 at 9:50 AM Michael S. Tsirkin <mst@redhat.com> wrote:

> On Thu, Nov 30, 2023 at 04:49:35PM +0100, Albert Esteve wrote:
> >
> >
> >
> > On Tue, Nov 7, 2023 at 10:37 AM Albert Esteve <aesteve@redhat.com>
> wrote:
> >
> >     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        | 14 +++++-----
> >      hw/virtio/vhost-user.c            | 33 ++++++++++++++++++-----
> >      hw/virtio/vhost.c                 |  5 ++++
> >      include/hw/virtio/vhost.h         |  6 +++++
> >      include/hw/virtio/virtio-dmabuf.h | 33 ++++++++++++-----------
> >      tests/unit/test-virtio-dmabuf.c   | 44
> +++++++++++++++----------------
> >      6 files changed, 83 insertions(+), 52 deletions(-)
> >
> >     --
> >     2.41.0
> >
> >
> >
> > Bump :)
> >
> > @Marc-André Lureau could you please take a look? You suggested the API
> > upgrades, so would be great if you could check if it is what you had in
> mind.
> >
> > Thanks!
>
> All this is post releas material, right?
>
>
Yes, it is.

Patchew 2.1-devel-b67e4c2

© 2016 - 2024 Red Hat, Inc.