[AMD Official Use Only - Internal Distribution Only]

Reviewed-by: Sonny Jiang <sonny.ji...@amd.com>
________________________________
From: Christian König <ckoenig.leichtzumer...@gmail.com>
Sent: Friday, March 5, 2021 7:51 AM
To: dri-de...@lists.freedesktop.org <dri-de...@lists.freedesktop.org>; 
amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
Cc: Liu, Leo <leo....@amd.com>; Jiang, Sonny <sonny.ji...@amd.com>
Subject: [PATCH 1/2] 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.koe...@amd.com>
---
 drivers/gpu/drm/scheduler/sched_entity.c | 6 ++++--
 1 file 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;
 }

 /**
--
2.25.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to