drm/virtio: Don't attach GEM to a non-created context in gem_object_open()

The vfpriv->ctx_id is always initialized to a non-zero value. Check whether
context was created before attaching GEM to this context ID. This left
unnoticed previously because host silently skips attachment if context
doesn't exist, still we shouldn't do that for consistency.

Fixes: 086b9f27f0ab ("drm/virtio: Don't create a context with default param if context_init is supported")
Cc: <stable@vger.kernel.org> # v6.14+
Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Link: https://lore.kernel.org/r/20250401123842.2232205-1-dmitry.osipenko@collabora.com

diff --git a/drivers/gpu/drm/virtio/virtgpu_gem.c b/drivers/gpu/drm/virtio/virtgpu_gem.c
index 5aab588fc400e7ad9795b9a8568b13e08d931e59..3d6aa26fdb5343cc82ba90957e3beeb6705d3e32 100644 (file)
--- a/drivers/gpu/drm/virtio/virtgpu_gem.c
+++ b/drivers/gpu/drm/virtio/virtgpu_gem.c
@@ -115,13 +115,14 @@ int virtio_gpu_gem_object_open(struct drm_gem_object *obj,
        if (!vgdev->has_context_init)
                virtio_gpu_create_context(obj->dev, file);
 
-       objs = virtio_gpu_array_alloc(1);
-       if (!objs)
-               return -ENOMEM;
-       virtio_gpu_array_add_obj(objs, obj);
+       if (vfpriv->context_created) {
+               objs = virtio_gpu_array_alloc(1);
+               if (!objs)
+                       return -ENOMEM;
+               virtio_gpu_array_add_obj(objs, obj);
 
-       if (vfpriv->ctx_id)
                virtio_gpu_cmd_context_attach_resource(vgdev, vfpriv->ctx_id, objs);
+       }
 
 out_notify:
        virtio_gpu_notify(vgdev);