[PATCH] drm/virtio: make sure context is created in gem open
Gurchetan Singh
gurchetansingh at chromium.org
Fri Jan 8 01:46:58 UTC 2021
On Thu, Jan 7, 2021 at 1:07 PM Chia-I Wu <olvaffe at gmail.com> wrote:
> The context might still be missing when DRM_IOCTL_PRIME_FD_TO_HANDLE is
> the first ioctl on the drm_file.
>
> Fixes: 72b48ae800da ("drm/virtio: enqueue virtio_gpu_create_context after
> the first 3D ioctl")
> Cc: Gurchetan Singh <gurchetansingh at chromium.org>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Signed-off-by: Chia-I Wu <olvaffe at gmail.com>
> ---
> drivers/gpu/drm/virtio/virtgpu_gem.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/virtio/virtgpu_gem.c
> b/drivers/gpu/drm/virtio/virtgpu_gem.c
> index c30c75ee83fc..8502400b2f9c 100644
> --- a/drivers/gpu/drm/virtio/virtgpu_gem.c
> +++ b/drivers/gpu/drm/virtio/virtgpu_gem.c
> @@ -39,9 +39,6 @@ static int virtio_gpu_gem_create(struct drm_file *file,
> int ret;
> u32 handle;
>
> - if (vgdev->has_virgl_3d)
> - virtio_gpu_create_context(dev, file);
> -
> ret = virtio_gpu_object_create(vgdev, params, &obj, NULL);
> if (ret < 0)
> return ret;
> @@ -119,6 +116,11 @@ int virtio_gpu_gem_object_open(struct drm_gem_object
> *obj,
> if (!vgdev->has_virgl_3d)
> goto out_notify;
>
> + /* the context might still be missing when the first ioctl is
> + * DRM_IOCTL_MODE_CREATE_DUMB or DRM_IOCTL_PRIME_FD_TO_HANDLE
> + */
> + virtio_gpu_create_context(obj->dev, file);
> +
> objs = virtio_gpu_array_alloc(1);
> if (!objs)
> return -ENOMEM;
> --
> 2.29.2.729.g45daf8777d-goog
>
> Reviewed-by: Gurchetan Singh <gurchetansingh at chromium.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20210107/764c0b1f/attachment.htm>
More information about the dri-devel
mailing list