<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 12, 2021 at 10:36 PM Vivek Kasireddy <<a href="mailto:vivek.kasireddy@intel.com">vivek.kasireddy@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If support for Blob resources is available, then dumb BOs created<br>
by the driver can be considered as guest Blobs.<br>
<br>
v2: Don't skip transfer and flush commands as part of plane update<br>
as the device may have created a shared mapping. (Gerd)<br>
<br>
v3: Don't create dumb BOs as Guest blobs if Virgl is enabled. (Gurchetan)<br></blockquote><div><br></div><div>I think it is a good start and advances QEMU blobs. Improvements are always possible, but may be made at a future time.</div><div><br></div><div>Acked-by: Gurchetan Singh <<a href="mailto:gurchetansingh@chromium.org">gurchetansingh@chromium.org</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Cc: Gerd Hoffmann <<a href="mailto:kraxel@redhat.com" target="_blank">kraxel@redhat.com</a>><br>
Signed-off-by: Vivek Kasireddy <<a href="mailto:vivek.kasireddy@intel.com" target="_blank">vivek.kasireddy@intel.com</a>><br>
---<br>
drivers/gpu/drm/virtio/virtgpu_gem.c | 8 ++++++++<br>
drivers/gpu/drm/virtio/virtgpu_object.c | 3 +++<br>
2 files changed, 11 insertions(+)<br>
<br>
diff --git a/drivers/gpu/drm/virtio/virtgpu_gem.c b/drivers/gpu/drm/virtio/virtgpu_gem.c<br>
index 8502400b2f9c..2de61b63ef91 100644<br>
--- a/drivers/gpu/drm/virtio/virtgpu_gem.c<br>
+++ b/drivers/gpu/drm/virtio/virtgpu_gem.c<br>
@@ -64,6 +64,7 @@ int virtio_gpu_mode_dumb_create(struct drm_file *file_priv,<br>
{<br>
struct drm_gem_object *gobj;<br>
struct virtio_gpu_object_params params = { 0 };<br>
+ struct virtio_gpu_device *vgdev = dev->dev_private;<br>
int ret;<br>
uint32_t pitch;<br>
<br>
@@ -79,6 +80,13 @@ int virtio_gpu_mode_dumb_create(struct drm_file *file_priv,<br>
params.height = args->height;<br>
params.size = args->size;<br>
params.dumb = true;<br>
+<br>
+ if (vgdev->has_resource_blob && !vgdev->has_virgl_3d) {<br>
+ params.blob_mem = VIRTGPU_BLOB_MEM_GUEST;<br>
+ params.blob_flags = VIRTGPU_BLOB_FLAG_USE_SHAREABLE;<br>
+ params.blob = true;<br>
+ }<br>
+<br>
ret = virtio_gpu_gem_create(file_priv, dev, ¶ms, &gobj,<br>
&args->handle);<br>
if (ret)<br>
diff --git a/drivers/gpu/drm/virtio/virtgpu_object.c b/drivers/gpu/drm/virtio/virtgpu_object.c<br>
index 4ff1ec28e630..f648b0e24447 100644<br>
--- a/drivers/gpu/drm/virtio/virtgpu_object.c<br>
+++ b/drivers/gpu/drm/virtio/virtgpu_object.c<br>
@@ -254,6 +254,9 @@ int virtio_gpu_object_create(struct virtio_gpu_device *vgdev,<br>
}<br>
<br>
if (params->blob) {<br>
+ if (params->blob_mem == VIRTGPU_BLOB_MEM_GUEST)<br>
+ bo->guest_blob = true;<br>
+<br>
virtio_gpu_cmd_resource_create_blob(vgdev, bo, params,<br>
ents, nents);<br>
} else if (params->virgl) {<br>
-- <br>
2.26.2<br>
<br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank">dri-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</blockquote></div></div>