Re: [PATCH v2 0/3] drm/sched: Documentation and refcount improvements

[Philipp Stanner <pstanner@redhat.com>]

On Tue, 2025-01-21 at 16:15 +0100, Philipp Stanner wrote:
> Changes in v2:
>   - Document what run_job() is allowed to return. (Tvrtko)
>   - Delete confusing comment about putting the fence. (Danilo)
>   - Apply Danilo's RB to patch 1.
>   - Delete info about job recovery for entities in patch 3. (Danilo,
> me)
>   - Set the term "ring" as fix term for both HW rings and FW rings. A
>     ring shall always be the thingy on the CPU ;) (Danilo)

s/CPU/GPU

obviously.

P.


>   - Many (all) other comments improvements in patch 3. (Danilo)
> 
> This is as series succeeding my previous patch [1].
> 
> I recognized that we are still referring to a non-existing function
> and
> a deprecated one in the callback docu. We should probably also point
> out
> the important distinction between hardware and firmware schedulers
> more
> cleanly.
> 
> Please give me feedback, especially on the RFC comments in patch3.
> 
> (This series still fires docu-build-warnings. I want to gather
> feedback
> on the opion questions first and will solve them in v2.)
> 
> Thank you,
> Philipp
> 
> [1]
> https://lore.kernel.org/all/20241220124515.93169-2-phasta@kernel.org/
> 
> Philipp Stanner (3):
>   drm/sched: Document run_job() refcount hazard
>   drm/sched: Adjust outdated docu for run_job()
>   drm/sched: Update timedout_job()'s documentation
> 
>  drivers/gpu/drm/scheduler/sched_main.c |   5 +-
>  include/drm/gpu_scheduler.h            | 106 ++++++++++++++++-------
> --
>  2 files changed, 71 insertions(+), 40 deletions(-)
>