From b2f69d86bbe857b1b6445ec06437e2703e999759 Mon Sep 17 00:00:00 2001 From: Stephen Dickey Date: Tue, 5 Apr 2022 11:51:16 -0700 Subject: [PATCH] sched/walt/core_control: disable need_all_cpus() need_all_cpus() was put in place in past products, so that silver isolation could be enabled and disabled based upon the window size, as a work around for lack of user-space controls for disabling/enabling core_control. This blocks the ability to isolate silver cores for lower window sizes, which is a valid case that should be allowed for most products. Cleanup the code for need all cpus, such that silver core control may be used. Change-Id: I0e029a6c0df5eb1207c549cf843614c360e91b8e Signed-off-by: Stephen Dickey --- kernel/sched/walt/core_ctl.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/kernel/sched/walt/core_ctl.c b/kernel/sched/walt/core_ctl.c index 6cee4ad08efd..12abfd26947f 100644 --- a/kernel/sched/walt/core_ctl.c +++ b/kernel/sched/walt/core_ctl.c @@ -784,12 +784,6 @@ static bool adjustment_possible(const struct cluster_data *cluster, cluster_paused_cpus(cluster))); } -static bool need_all_cpus(const struct cluster_data *cluster) -{ - return (is_min_cluster_cpu(cluster->first_cpu) && - sched_ravg_window < DEFAULT_SCHED_RAVG_WINDOW); -} - static bool eval_need(struct cluster_data *cluster) { unsigned long flags; @@ -805,7 +799,7 @@ static bool eval_need(struct cluster_data *cluster) spin_lock_irqsave(&state_lock, flags); - if (cluster->boost || !cluster->enable || need_all_cpus(cluster)) { + if (cluster->boost || !cluster->enable) { need_cpus = cluster->max_cpus; } else { cluster->active_cpus = get_active_cpu_count(cluster);