All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled"
@ 2026-09-11 14:42 Dmitry Osipenko
  2026-09-11 14:55 ` sashiko-bot
  2026-09-11 15:32 ` Yiwei Zhang
  0 siblings, 2 replies; 3+ messages in thread
From: Dmitry Osipenko @ 2026-09-11 14:42 UTC (permalink / raw)
  To: David Airlie, Gerd Hoffmann, Gurchetan Singh, Chia-I Wu,
	Val Packett
  Cc: dri-devel, virtualization, linux-kernel, Yiwei Zhang

Guest userspace may import udmabuf to vrend. Vrend doesn't support guest
blobs, and thus, further 3d operations with the imported blob are failing.
Typical scenario of the problem shown with mouse cursor RGBA image imported
into virtio-gpu, which previously was rejected by virtio-gpu driver.
Revert enabling guest blobs importing into vrend to fix the regression.

Link: https://gitlab.freedesktop.org/virgl/virglrenderer/-/work_items/674
Fixes: df4dc947c46b ("drm/virtio: Allow importing prime buffers when 3D is enabled")
Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
---
 drivers/gpu/drm/virtio/virtgpu_prime.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/virtio/virtgpu_prime.c b/drivers/gpu/drm/virtio/virtgpu_prime.c
index 149e6bcb5878..ebf471044d06 100644
--- a/drivers/gpu/drm/virtio/virtgpu_prime.c
+++ b/drivers/gpu/drm/virtio/virtgpu_prime.c
@@ -349,7 +349,7 @@ struct drm_gem_object *virtgpu_gem_prime_import(struct drm_device *dev,
 		}
 	}
 
-	if (!vgdev->has_resource_blob)
+	if (!vgdev->has_resource_blob || vgdev->has_virgl_3d)
 		return drm_gem_prime_import(dev, buf);
 
 	bo = kzalloc_obj(*bo);
-- 
2.55.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled"
  2026-09-11 14:42 [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled" Dmitry Osipenko
@ 2026-09-11 14:55 ` sashiko-bot
  2026-09-11 15:32 ` Yiwei Zhang
  1 sibling, 0 replies; 3+ messages in thread
From: sashiko-bot @ 2026-09-11 14:55 UTC (permalink / raw)
  To: Dmitry Osipenko
  Cc: Eugenio Perez, dri-devel, virtualization, Michael S. Tsirkin

> Guest userspace may import udmabuf to vrend. Vrend doesn't support guest
> blobs, and thus, further 3d operations with the imported blob are failing.
> Typical scenario of the problem shown with mouse cursor RGBA image imported
> into virtio-gpu, which previously was rejected by virtio-gpu driver.
> Revert enabling guest blobs importing into vrend to fix the regression.
> 
> Link: https://gitlab.freedesktop.org/virgl/virglrenderer/-/work_items/674
> Fixes: df4dc947c46b ("drm/virtio: Allow importing prime buffers when 3D is enabled")
> Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260911144204.2089401-1-dmitry.osipenko@collabora.com?part=1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled"
  2026-09-11 14:42 [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled" Dmitry Osipenko
  2026-09-11 14:55 ` sashiko-bot
@ 2026-09-11 15:32 ` Yiwei Zhang
  1 sibling, 0 replies; 3+ messages in thread
From: Yiwei Zhang @ 2026-09-11 15:32 UTC (permalink / raw)
  To: Dmitry Osipenko
  Cc: David Airlie, Gerd Hoffmann, Gurchetan Singh, Chia-I Wu,
	Val Packett, dri-devel, virtualization, linux-kernel

On Fri, Sep 11, 2026 at 7:42 AM Dmitry Osipenko
<dmitry.osipenko@collabora.com> wrote:
>
> Guest userspace may import udmabuf to vrend. Vrend doesn't support guest
> blobs, and thus, further 3d operations with the imported blob are failing.
> Typical scenario of the problem shown with mouse cursor RGBA image imported
> into virtio-gpu, which previously was rejected by virtio-gpu driver.
> Revert enabling guest blobs importing into vrend to fix the regression.
>
> Link: https://gitlab.freedesktop.org/virgl/virglrenderer/-/work_items/674
> Fixes: df4dc947c46b ("drm/virtio: Allow importing prime buffers when 3D is enabled")
> Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
> ---
>  drivers/gpu/drm/virtio/virtgpu_prime.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/virtio/virtgpu_prime.c b/drivers/gpu/drm/virtio/virtgpu_prime.c
> index 149e6bcb5878..ebf471044d06 100644
> --- a/drivers/gpu/drm/virtio/virtgpu_prime.c
> +++ b/drivers/gpu/drm/virtio/virtgpu_prime.c
> @@ -349,7 +349,7 @@ struct drm_gem_object *virtgpu_gem_prime_import(struct drm_device *dev,
>                 }
>         }
>
> -       if (!vgdev->has_resource_blob)
> +       if (!vgdev->has_resource_blob || vgdev->has_virgl_3d)
>                 return drm_gem_prime_import(dev, buf);
>
>         bo = kzalloc_obj(*bo);
> --
> 2.55.0
>

r-b. Thanks for the prompt revert!

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-09-11 15:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-11 14:42 [PATCH v1] Revert "drm/virtio: Allow importing prime buffers when 3D is enabled" Dmitry Osipenko
2026-09-11 14:55 ` sashiko-bot
2026-09-11 15:32 ` Yiwei Zhang

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.