diff options
| author | Christian König <christian.koenig@amd.com> | 2021-02-20 11:50:42 +0300 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2021-03-08 16:06:17 +0300 |
| commit | ac4eb83ab255de9c31184df51fd1534ba36fd212 (patch) | |
| tree | 9d68a6b4e2f4bdf6c19de0542a8f77be4e4efeb4 /drivers/gpu/drm/scheduler | |
| parent | a9cdf68092908313320c6959c4e1bdfdb7103613 (diff) | |
| download | linux-ac4eb83ab255de9c31184df51fd1534ba36fd212.tar.xz | |
drm/sched: select new rq even if there is only one v3
This is necessary when changing priorities of an entity.
v2: test the sched_list instead of num_sched.
v3: set the sched_list to NULL when there is only one entry
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Sonny Jiang <sonny.jiang@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210305125155.2312-1-christian.koenig@amd.com
Diffstat (limited to 'drivers/gpu/drm/scheduler')
| -rw-r--r-- | drivers/gpu/drm/scheduler/sched_entity.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/gpu/drm/scheduler/sched_entity.c b/drivers/gpu/drm/scheduler/sched_entity.c index 92d965b629c6..f0790e9471d1 100644 --- a/drivers/gpu/drm/scheduler/sched_entity.c +++ b/drivers/gpu/drm/scheduler/sched_entity.c @@ -453,7 +453,7 @@ void drm_sched_entity_select_rq(struct drm_sched_entity *entity) struct drm_gpu_scheduler *sched; struct drm_sched_rq *rq; - if (spsc_queue_count(&entity->job_queue) || entity->num_sched_list <= 1) + if (spsc_queue_count(&entity->job_queue) || !entity->sched_list) return; fence = READ_ONCE(entity->last_scheduled); @@ -467,8 +467,10 @@ void drm_sched_entity_select_rq(struct drm_sched_entity *entity) drm_sched_rq_remove_entity(entity->rq, entity); entity->rq = rq; } - spin_unlock(&entity->rq_lock); + + if (entity->num_sched_list == 1) + entity->sched_list = NULL; } /** |