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

Message

Philipp Stanner Jan. 21, 2025, 3:15 p.m. UTC

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)
  - 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/[e-mail]/

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(-)

Comments

Philipp Stanner Jan. 22, 2025, 8:23 a.m. UTC | #1

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/[e-mail]/
> 
> 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(-)
>