[PATCH v2 1/2] drm/virtio: Defer the host dumb buffer creation
Gerd Hoffmann
kraxel at redhat.com
Wed Sep 4 07:10:04 UTC 2024
Hi,
> - } else {
> - virtio_gpu_cmd_create_resource(vgdev, bo, params,
> - objs, fence);
> - virtio_gpu_object_attach(vgdev, bo, ents, nents);
> + } else if (params->dumb) {
> + /* Create the host resource in virtio_gpu_user_framebuffer_create()
> + * because the pixel format is not specified yet
> + */
> + bo->ents = ents;
> + bo->nents = nents;
> + bo->deferred = true;
> }
else {
return -EINVAL;
}
I think this should not happen, because non-dumb buffers are only
created with virgl being active. Nevertheless we should catch this
case and return an error, maybe even have a WARN_ONCE() there.
Otherwise looks good to me, so with that fixed:
Reviewed-by: Gerd Hoffmann <kraxel at redhat.com>
take care,
Gerd
More information about the dri-devel
mailing list