drm/scheduler: improve job distribution with multiple queues
This patch uses score based logic to select a new rq for better loadbalance between multiple rq/scheds instead of num_jobs. Below are test results after running amdgpu_test from mesa drm Before this patch: sched_name num of many times it got scheduled ========= ================================== sdma0 314 sdma1 32 comp_1.0.0 56 comp_1.0.1 0 comp_1.1.0 0 comp_1.1.1 0 comp_1.2.0 0 comp_1.2.1 0 comp_1.3.0 0 comp_1.3.1 0 After this patch: sched_name num of many times it got scheduled ========= ================================== sdma0 216 sdma1 185 comp_1.0.0 39 comp_1.0.1 9 comp_1.1.0 12 comp_1.1.1 0 comp_1.2.0 12 comp_1.2.1 0 comp_1.3.0 12 comp_1.3.1 0 Signed-off-by: Nirmoy Das <nirmoy.das@amd.com> Reviewed-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
@@ -130,7 +130,7 @@ static struct drm_sched_rq *
|
||||
drm_sched_entity_get_free_sched(struct drm_sched_entity *entity)
|
||||
{
|
||||
struct drm_sched_rq *rq = NULL;
|
||||
unsigned int min_jobs = UINT_MAX, num_jobs;
|
||||
unsigned int min_score = UINT_MAX, num_score;
|
||||
int i;
|
||||
|
||||
for (i = 0; i < entity->num_sched_list; ++i) {
|
||||
@@ -141,9 +141,9 @@ drm_sched_entity_get_free_sched(struct drm_sched_entity *entity)
|
||||
continue;
|
||||
}
|
||||
|
||||
num_jobs = atomic_read(&sched->num_jobs);
|
||||
if (num_jobs < min_jobs) {
|
||||
min_jobs = num_jobs;
|
||||
num_score = atomic_read(&sched->score);
|
||||
if (num_score < min_score) {
|
||||
min_score = num_score;
|
||||
rq = &entity->sched_list[i]->sched_rq[entity->priority];
|
||||
}
|
||||
}
|
||||
@@ -498,7 +498,7 @@ void drm_sched_entity_push_job(struct drm_sched_job *sched_job,
|
||||
bool first;
|
||||
|
||||
trace_drm_sched_job(sched_job, entity);
|
||||
atomic_inc(&entity->rq->sched->num_jobs);
|
||||
atomic_inc(&entity->rq->sched->score);
|
||||
WRITE_ONCE(entity->last_user, current->group_leader);
|
||||
first = spsc_queue_push(&entity->job_queue, &sched_job->queue_node);
|
||||
|
||||
|
Reference in New Issue
Block a user