diff options
| author | Tvrtko Ursulin <tvrtko.ursulin@igalia.com> | 2024-10-16 13:20:09 +0100 |
|---|---|---|
| committer | Philipp Stanner <pstanner@redhat.com> | 2024-10-17 12:15:11 +0200 |
| commit | d42a254633c773921884a19e8a1a0f53a31150c3 (patch) | |
| tree | 2c6bd912108333219ab30e0e3a07da701386e9fc /drivers/ras/amd | |
| parent | fd3b2c5f40a1e028bc813284260d430257444334 (diff) | |
drm/sched: Optimise drm_sched_entity_push_job
In FIFO mode (which is the default), both drm_sched_entity_push_job() and
drm_sched_rq_update_fifo(), where the latter calls the former, are
currently taking and releasing the same entity->rq_lock.
We can avoid that design inelegance, and also have a miniscule
efficiency improvement on the submit from idle path, by introducing a new
drm_sched_rq_update_fifo_locked() helper and pulling up the lock taking to
its callers.
v2:
* Remove drm_sched_rq_update_fifo() altogether. (Christian)
v3:
* Improved commit message. (Philipp)
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Luben Tuikov <ltuikov89@gmail.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Philipp Stanner <pstanner@redhat.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Philipp Stanner <pstanner@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241016122013.7857-2-tursulin@igalia.com
Diffstat (limited to 'drivers/ras/amd')
0 files changed, 0 insertions, 0 deletions