mbox series

[0/2] virtgpu dummy events

Message ID 20211122232210.602-1-gurchetansingh@google.com (mailing list archive)
Headers show
Series virtgpu dummy events | expand

Message

Gurchetan Singh Nov. 22, 2021, 11:22 p.m. UTC
From: Gurchetan Singh <gurchetansingh@chromium.org>

There was a desire to not have a virtgpu-specific implementation of
poll(..), but there wasn't any real event to return either.

Solution: Dummy event with just event code

Context:

https://lists.freedesktop.org/archives/dri-devel/2021-November/330950.html

Userspace:

crrev.com/c/3296610

This series applies to drm-misc-fixes.

Gurchetan Singh (2):
  drm/virtgpu api: define a dummy fence signaled event
  drm/virtio: use drm_poll(..) instead of virtio_gpu_poll(..)

 drivers/gpu/drm/virtio/virtgpu_drv.c   | 42 +-------------------------
 drivers/gpu/drm/virtio/virtgpu_drv.h   |  1 -
 drivers/gpu/drm/virtio/virtgpu_ioctl.c |  2 +-
 include/uapi/drm/virtgpu_drm.h         |  7 +++++
 4 files changed, 9 insertions(+), 43 deletions(-)

Comments

Daniel Vetter Nov. 26, 2021, 7:16 a.m. UTC | #1
On Mon, Nov 22, 2021 at 03:22:08PM -0800, Gurchetan Singh wrote:
> From: Gurchetan Singh <gurchetansingh@chromium.org>
> 
> There was a desire to not have a virtgpu-specific implementation of
> poll(..), but there wasn't any real event to return either.
> 
> Solution: Dummy event with just event code
> 
> Context:
> 
> https://lists.freedesktop.org/archives/dri-devel/2021-November/330950.html
> 
> Userspace:
> 
> crrev.com/c/3296610
> 
> This series applies to drm-misc-fixes.

On the series:

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

I'm assuming someone from Google can push this to drm-misc-fixes for you?
-Daniel

> 
> Gurchetan Singh (2):
>   drm/virtgpu api: define a dummy fence signaled event
>   drm/virtio: use drm_poll(..) instead of virtio_gpu_poll(..)
> 
>  drivers/gpu/drm/virtio/virtgpu_drv.c   | 42 +-------------------------
>  drivers/gpu/drm/virtio/virtgpu_drv.h   |  1 -
>  drivers/gpu/drm/virtio/virtgpu_ioctl.c |  2 +-
>  include/uapi/drm/virtgpu_drm.h         |  7 +++++
>  4 files changed, 9 insertions(+), 43 deletions(-)
> 
> -- 
> 2.34.0.rc2.393.gf8c9666880-goog
>
Gerd Hoffmann Nov. 29, 2021, 11:09 a.m. UTC | #2
Hi,

> On the series:
> 
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 
> I'm assuming someone from Google can push this to drm-misc-fixes for you?

Thanks, pushed.

take care,
  Gerd
Daniel Vetter Nov. 30, 2021, 8:31 a.m. UTC | #3
On Mon, Nov 29, 2021 at 12:09:10PM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> > On the series:
> > 
> > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> > 
> > I'm assuming someone from Google can push this to drm-misc-fixes for you?
> 
> Thanks, pushed.

Thanks a lot to everyone for handling this so quickly, very much
appreciated.
-Daniel