ll-scheduler: remove an unused field

The .registered_cores field in struct ll_schedule_domain is never
used, remove it.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
This commit is contained in:
Guennadi Liakhovetski 2021-06-02 10:50:19 +02:00 committed by Liam Girdwood
parent 5abd8a9ea5
commit d8295f4c49
2 changed files with 3 additions and 11 deletions

View File

@ -46,7 +46,6 @@ struct ll_schedule_domain {
uint64_t new_target_tick; /**< for the next set, used during the reschedule stage */
spinlock_t lock; /**< standard lock */
atomic_t total_num_tasks; /**< total number of registered tasks */
atomic_t registered_cores; /**< number of registered cores */
atomic_t enabled_cores; /**< number of enabled cores */
uint32_t ticks_per_ms; /**< number of clock ticks per ms */
int type; /**< domain type */
@ -92,7 +91,6 @@ static inline struct ll_schedule_domain *domain_init
spinlock_init(&domain->lock);
atomic_init(&domain->total_num_tasks, 0);
atomic_init(&domain->registered_cores, 0);
atomic_init(&domain->enabled_cores, 0);
return domain;
@ -131,11 +129,9 @@ static inline int domain_register(struct ll_schedule_domain *domain,
/* registered one more task, increase the count */
atomic_add(&domain->total_num_tasks, 1);
if (!domain->registered[cpu_get_id()]) {
if (!domain->registered[cpu_get_id()])
/* first task of the core, new client registered */
domain->registered[cpu_get_id()] = true;
atomic_add(&domain->registered_cores, 1);
}
}
return ret;
@ -155,10 +151,8 @@ static inline void domain_unregister(struct ll_schedule_domain *domain,
atomic_sub(&domain->total_num_tasks, 1);
/* the last task of the core, unregister the client/core */
if (!num_tasks && domain->registered[cpu_get_id()]) {
if (!num_tasks && domain->registered[cpu_get_id()])
domain->registered[cpu_get_id()] = false;
atomic_sub(&domain->registered_cores, 1);
}
}
}

View File

@ -109,10 +109,8 @@ static void schedule_ll_task_done(struct ll_schedule_data *sch,
* If this was the last task of the core, unregister the client/core
*/
if (atomic_sub(&sch->num_tasks, 1) == 1 &&
sch->domain->registered[cpu_get_id()]) {
sch->domain->registered[cpu_get_id()])
sch->domain->registered[cpu_get_id()] = false;
atomic_sub(&sch->domain->registered_cores, 1);
}
tr_info(&ll_tr, "task complete %p %pU", task, task->uid);
tr_info(&ll_tr, "num_tasks %d total_num_tasks %d",