mbox series

[0/3] drm/virtio: fix stale mm entries on driver shutdown

Message ID 20200908070723.6394-1-kraxel@redhat.com (mailing list archive)
Headers show
Series drm/virtio: fix stale mm entries on driver shutdown | expand

Message

Gerd Hoffmann Sept. 8, 2020, 7:07 a.m. UTC 
Gerd Hoffmann (3):
  drm/virtio: use drmm_mode_config_init
  drm/virtio: return virtio_gpu_queue errors
  drm/virtio: add virtio_gpu_cmd_unref_resource error handling

 drivers/gpu/drm/virtio/virtgpu_drv.h     |  2 +-
 drivers/gpu/drm/virtio/virtgpu_display.c | 11 ++++---
 drivers/gpu/drm/virtio/virtgpu_kms.c     |  6 +++-
 drivers/gpu/drm/virtio/virtgpu_vq.c      | 41 +++++++++++++-----------
 4 files changed, 36 insertions(+), 24 deletions(-)

Comments

Gurchetan Singh Sept. 10, 2020, 12:37 a.m. UTC | #1 
On Tue, Sep 8, 2020 at 12:07 AM Gerd Hoffmann <kraxel@redhat.com> wrote:

>
>
> Gerd Hoffmann (3):
>   drm/virtio: use drmm_mode_config_init
>   drm/virtio: return virtio_gpu_queue errors
>   drm/virtio: add virtio_gpu_cmd_unref_resource error handling
>

lgtm +/- nits:
- add a simple "why" in the first commit message
- checkpatch --strict reports:

CHECK: Alignment should match open parenthesis
#101: FILE: drivers/gpu/drm/virtio/virtgpu_vq.c:450:
+static int virtio_gpu_queue_ctrl_buffer(struct virtio_gpu_device *vgdev,
                                         struct virtio_gpu_vbuffer *vbuf)

Either way, series is:

Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>


>
>  drivers/gpu/drm/virtio/virtgpu_drv.h     |  2 +-
>  drivers/gpu/drm/virtio/virtgpu_display.c | 11 ++++---
>  drivers/gpu/drm/virtio/virtgpu_kms.c     |  6 +++-
>  drivers/gpu/drm/virtio/virtgpu_vq.c      | 41 +++++++++++++-----------
>  4 files changed, 36 insertions(+), 24 deletions(-)
>
> --
> 2.27.0
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>