mirror of
https://github.com/lkl/linux.git
synced 2025-12-19 16:13:19 +09:00
drm/v3d: replace open-coded implementation of drm_gem_object_lookup
As v3d_submit_tfu_ioctl() performs the same steps as drm_gem_object_lookup(), replace the open-code implementation in v3d with its DRM core equivalent. Signed-off-by: Maíra Canal <mcanal@igalia.com> Reviewed-by: Melissa Wen <mwen@igalia.com> Signed-off-by: Melissa Wen <melissa.srw@gmail.com> Link: https://patchwork.freedesktop.org/patch/msgid/20221227200010.191351-1-mcanal@igalia.com
This commit is contained in:
@@ -861,7 +861,6 @@ v3d_submit_tfu_ioctl(struct drm_device *dev, void *data,
|
|||||||
|
|
||||||
job->args = *args;
|
job->args = *args;
|
||||||
|
|
||||||
spin_lock(&file_priv->table_lock);
|
|
||||||
for (job->base.bo_count = 0;
|
for (job->base.bo_count = 0;
|
||||||
job->base.bo_count < ARRAY_SIZE(args->bo_handles);
|
job->base.bo_count < ARRAY_SIZE(args->bo_handles);
|
||||||
job->base.bo_count++) {
|
job->base.bo_count++) {
|
||||||
@@ -870,20 +869,16 @@ v3d_submit_tfu_ioctl(struct drm_device *dev, void *data,
|
|||||||
if (!args->bo_handles[job->base.bo_count])
|
if (!args->bo_handles[job->base.bo_count])
|
||||||
break;
|
break;
|
||||||
|
|
||||||
bo = idr_find(&file_priv->object_idr,
|
bo = drm_gem_object_lookup(file_priv, args->bo_handles[job->base.bo_count]);
|
||||||
args->bo_handles[job->base.bo_count]);
|
|
||||||
if (!bo) {
|
if (!bo) {
|
||||||
DRM_DEBUG("Failed to look up GEM BO %d: %d\n",
|
DRM_DEBUG("Failed to look up GEM BO %d: %d\n",
|
||||||
job->base.bo_count,
|
job->base.bo_count,
|
||||||
args->bo_handles[job->base.bo_count]);
|
args->bo_handles[job->base.bo_count]);
|
||||||
ret = -ENOENT;
|
ret = -ENOENT;
|
||||||
spin_unlock(&file_priv->table_lock);
|
|
||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
drm_gem_object_get(bo);
|
|
||||||
job->base.bo[job->base.bo_count] = bo;
|
job->base.bo[job->base.bo_count] = bo;
|
||||||
}
|
}
|
||||||
spin_unlock(&file_priv->table_lock);
|
|
||||||
|
|
||||||
ret = v3d_lock_bo_reservations(&job->base, &acquire_ctx);
|
ret = v3d_lock_bo_reservations(&job->base, &acquire_ctx);
|
||||||
if (ret)
|
if (ret)
|
||||||
|
|||||||
Reference in New Issue
Block a user