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 <quic_dickey@quicinc.com>
This commit is contained in:
Stephen Dickey 2022-04-05 11:51:16 -07:00 committed by Rishabh Bhatnagar
parent 498e57c028
commit b2f69d86bb

View File

@ -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);