[Intel-gfx] [PATCH] drm: Remove unused drm_file parameter to drm_syncobj_replace_fence()
Jason Ekstrand
jason at jlekstrand.net
Wed Jul 5 21:19:10 UTC 2017
On Wed, Jul 5, 2017 at 1:12 PM, Chris Wilson <chris at chris-wilson.co.uk>
wrote:
> the drm_file parameter is unused, so remove it.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Dave Airlie <airlied at redhat.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 6 ++----
> drivers/gpu/drm/drm_syncobj.c | 8 +++-----
> include/drm/drm_syncobj.h | 3 +--
> 3 files changed, 6 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index aeee6840e82b..8c73b91f0ddc 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -1069,10 +1069,8 @@ static void amdgpu_cs_post_dependencies(struct
> amdgpu_cs_parser *p)
> {
> int i;
>
> - for (i = 0; i < p->num_post_dep_syncobjs; ++i) {
> - drm_syncobj_replace_fence(p->filp,
> p->post_dep_syncobjs[i],
> - p->fence);
> - }
> + for (i = 0; i < p->num_post_dep_syncobjs; ++i)
> + drm_syncobj_replace_fence(p->post_dep_syncobjs[i],
> p->fence);
> }
>
> static int amdgpu_cs_submit(struct amdgpu_cs_parser *p,
> diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
> index 89441bc78591..789ba0b37f7b 100644
> --- a/drivers/gpu/drm/drm_syncobj.c
> +++ b/drivers/gpu/drm/drm_syncobj.c
> @@ -77,17 +77,15 @@ EXPORT_SYMBOL(drm_syncobj_find);
>
> /**
> * drm_syncobj_replace_fence - replace fence in a sync object.
> - * @file_private: drm file private pointer.
> * @syncobj: Sync object to replace fence in
> * @fence: fence to install in sync file.
> *
> * This replaces the fence on a sync object.
> */
> -void drm_syncobj_replace_fence(struct drm_file *file_private,
> - struct drm_syncobj *syncobj,
> +void drm_syncobj_replace_fence(struct drm_syncobj *syncobj,
> struct dma_fence *fence)
> {
> - struct dma_fence *old_fence = NULL;
> + struct dma_fence *old_fence;
>
This change looks unrelated. Valid, but unrelated. :-)
Having worked through your i915 syncobj patch, this definitely makes some
things a little bit nicer.
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
>
> if (fence)
> dma_fence_get(fence);
> @@ -292,7 +290,7 @@ int drm_syncobj_import_sync_file_fence(struct
> drm_file *file_private,
> return -ENOENT;
> }
>
> - drm_syncobj_replace_fence(file_private, syncobj, fence);
> + drm_syncobj_replace_fence(syncobj, fence);
> dma_fence_put(fence);
> drm_syncobj_put(syncobj);
> return 0;
> diff --git a/include/drm/drm_syncobj.h b/include/drm/drm_syncobj.h
> index 2c3610a4a074..89976da542b1 100644
> --- a/include/drm/drm_syncobj.h
> +++ b/include/drm/drm_syncobj.h
> @@ -79,8 +79,7 @@ drm_syncobj_put(struct drm_syncobj *obj)
>
> struct drm_syncobj *drm_syncobj_find(struct drm_file *file_private,
> u32 handle);
> -void drm_syncobj_replace_fence(struct drm_file *file_private,
> - struct drm_syncobj *syncobj,
> +void drm_syncobj_replace_fence(struct drm_syncobj *syncobj,
> struct dma_fence *fence);
> int drm_syncobj_fence_get(struct drm_file *file_private,
> u32 handle,
> --
> 2.13.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20170705/d47779f7/attachment.html>
More information about the Intel-gfx
mailing list