Re: [PATCH 1/2] drm/lima: Use drm_sched_job_add_syncobj_dependency()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Patch is:
Reviewed-by: Qiang Yu <yuq825@xxxxxxxxx>

On Sat, Feb 25, 2023 at 5:41 AM Maíra Canal <mcanal@xxxxxxxxxx> wrote:
>
> As lima_gem_add_deps() performs the same steps as
> drm_sched_job_add_syncobj_dependency(), replace the open-coded
> implementation in Lima in order to simply use the DRM function.
>
> Signed-off-by: Maíra Canal <mcanal@xxxxxxxxxx>
> ---
>  drivers/gpu/drm/lima/lima_gem.c | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/lima/lima_gem.c b/drivers/gpu/drm/lima/lima_gem.c
> index 0f1ca0b0db49..10252dc11a22 100644
> --- a/drivers/gpu/drm/lima/lima_gem.c
> +++ b/drivers/gpu/drm/lima/lima_gem.c
> @@ -277,21 +277,13 @@ static int lima_gem_add_deps(struct drm_file *file, struct lima_submit *submit)
>         int i, err;
>
>         for (i = 0; i < ARRAY_SIZE(submit->in_sync); i++) {
> -               struct dma_fence *fence = NULL;
> -
>                 if (!submit->in_sync[i])
>                         continue;
>
> -               err = drm_syncobj_find_fence(file, submit->in_sync[i],
> -                                            0, 0, &fence);
> +               err = drm_sched_job_add_syncobj_dependency(&submit->task->base, file,
> +                                                          submit->in_sync[i], 0);
>                 if (err)
>                         return err;
> -
> -               err = drm_sched_job_add_dependency(&submit->task->base, fence);
> -               if (err) {
> -                       dma_fence_put(fence);
> -                       return err;
> -               }
>         }
>
>         return 0;
> --
> 2.39.2
>




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux