[PATCH] drm/sched: Delete unused update_job_credits

[Philipp Stanner]

On Fri, 2025-01-10 at 11:13 +0000, Tvrtko Ursulin wrote:
> No driver is using the update_job_credits() schduler vfunc
> so lets remove it.
> 
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at igalia.com>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Danilo Krummrich <dakr at redhat.com>
> Cc: Matthew Brost <matthew.brost at intel.com>
> Cc: Philipp Stanner <pstanner at redhat.com>
> Acked-by: Danilo Krummrich <dakr at kernel.org>
> Acked-by: Boris Brezillon <boris.brezillon at collabora.com>
> Acked-by: Matt Coster <matt.coster at imgtec.com>

Applied to drm-misc-next

Thank you Tvrtko

P.

> ---
>  drivers/gpu/drm/scheduler/sched_main.c | 13 -------------
>  include/drm/gpu_scheduler.h            | 13 -------------
>  2 files changed, 26 deletions(-)
> 
> diff --git a/drivers/gpu/drm/scheduler/sched_main.c
> b/drivers/gpu/drm/scheduler/sched_main.c
> index 57da84908752..81d69c2ff6ab 100644
> --- a/drivers/gpu/drm/scheduler/sched_main.c
> +++ b/drivers/gpu/drm/scheduler/sched_main.c
> @@ -64,12 +64,6 @@
>   * credit limit, the job won't be executed. Instead, the scheduler
> will wait
>   * until the credit count has decreased enough to not overflow its
> credit limit.
>   * This implies waiting for previously executed jobs.
> - *
> - * Optionally, drivers may register a callback (update_job_credits)
> provided by
> - * struct drm_sched_backend_ops to update the job's credits
> dynamically. The
> - * scheduler executes this callback every time the scheduler
> considers a job for
> - * execution and subsequently checks whether the job fits the
> scheduler's credit
> - * limit.
>   */
>  
>  #include <linux/wait.h>
> @@ -133,13 +127,6 @@ static bool drm_sched_can_queue(struct
> drm_gpu_scheduler *sched,
>  	if (!s_job)
>  		return false;
>  
> -	if (sched->ops->update_job_credits) {
> -		s_job->credits = sched->ops-
> >update_job_credits(s_job);
> -
> -		drm_WARN(sched, !s_job->credits,
> -			 "Jobs with zero credits bypass job-flow
> control.\n");
> -	}
> -
>  	/* If a job exceeds the credit limit, truncate it to the
> credit limit
>  	 * itself to guarantee forward progress.
>  	 */
> diff --git a/include/drm/gpu_scheduler.h
> b/include/drm/gpu_scheduler.h
> index 95e17504e46a..e2e6af8849c6 100644
> --- a/include/drm/gpu_scheduler.h
> +++ b/include/drm/gpu_scheduler.h
> @@ -476,19 +476,6 @@ struct drm_sched_backend_ops {
>           * and it's time to clean it up.
>  	 */
>  	void (*free_job)(struct drm_sched_job *sched_job);
> -
> -	/**
> -	 * @update_job_credits: Called when the scheduler is
> considering this
> -	 * job for execution.
> -	 *
> -	 * This callback returns the number of credits the job would
> take if
> -	 * pushed to the hardware. Drivers may use this to
> dynamically update
> -	 * the job's credit count. For instance, deduct the number
> of credits
> -	 * for already signalled native fences.
> -	 *
> -	 * This callback is optional.
> -	 */
> -	u32 (*update_job_credits)(struct drm_sched_job *sched_job);
>  };
>  
>  /**