From: Chia-I Wu <olva...@gmail.com>

[ Upstream commit 8aeef9d4f48917ce85710949b079548974b4a638 ]

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 <gurchetansi...@chromium.org>
Cc: Gerd Hoffmann <kra...@redhat.com>
Signed-off-by: Chia-I Wu <olva...@gmail.com>
Link: 
http://patchwork.freedesktop.org/patch/msgid/20210107210726.269584-1-olva...@gmail.com
Reviewed-by: Gurchetan Singh <gurchetansi...@chromium.org>
Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 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 c30c75ee83fce..8502400b2f9c9 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.27.0



Reply via email to