Re: [PATCH v2 0/2] drm/vboxvideo: Use struct drm_shadow_plane_state for cursor plane

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Am 08.02.21 um 18:43 schrieb Hans de Goede:
Hi,

On 2/8/21 2:50 PM, Thomas Zimmermann wrote:
(was: drm/vboxvideo: Vmap/vunmap cursor BO in prepare_fb and cleanup_fb)

Functions in the atomic commit tail are not allowed to acquire the
dmabuf's reservation lock. So we cannot legally call the GEM object's
vmap functionality in atomic_update.

But, much better, we can use drm_shadow_plane_state to do all the mapping
for us. Patch 1 exports the helpers for shadow-buffered planes from the
DRM KMS helper library and adds documentation on how to use them. Patch 2
replaces the vmap code in vbox' cursor update code with a the helpers for
shadow-buffered planes.

Thomas Zimmermann (2):
   drm/gem: Export helpers for shadow-buffered planes
   drm/vboxvideo: Implement cursor plane with struct
     drm_shadow_plane_state

I've given this a test spin in a virtualbox vm using VboxSVGA graphics
and I've not found any problems:

Tested-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Thanks a lot. Can I add your Acked-by as well?

Best regards
Thomas


Regards,

Hans




  drivers/gpu/drm/drm_gem_atomic_helper.c | 148 +++++++++++++++++++++++-
  drivers/gpu/drm/vboxvideo/vbox_mode.c   |  28 ++---
  include/drm/drm_gem_atomic_helper.h     |  32 +++++
  3 files changed, 181 insertions(+), 27 deletions(-)

--
2.30.0


_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel


--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux