Commit 1e5a73ed authored by Andres Oportus's avatar Andres Oportus Committed by Vincent Zvikaramba

ANDROID: Fix cpufreq stats table creation

cpufreq stats does not correctly supports multiple cpus per profile.
For instance Marlin/Sailfish per cpu stats struct does not get created
for all cpus (only one per policy).  This change does not provide full
support for multiple cpus per profile but allows stats creation per
cpu to allow b/34133340 to be completed.

Bug: 38244231
Bug: 34133340
Test: Boot Sailfish
Signed-off-by: default avatarAndres Oportus <>
Change-Id: I72ea548a199f57ed841618b08b9c41e99b493376
parent 43b8d157
......@@ -575,12 +575,11 @@ static void cpufreq_powerstats_free(void)
static int __cpufreq_stats_create_table(struct cpufreq_policy *policy,
struct cpufreq_frequency_table *table, int count)
int cpu, struct cpufreq_frequency_table *table, int count)
unsigned int i, j, ret = 0;
struct cpufreq_stats *stat;
unsigned int alloc_size;
unsigned int cpu = policy->cpu;
if (per_cpu(cpufreq_stats_table, cpu))
return 0;
......@@ -592,10 +591,6 @@ static int __cpufreq_stats_create_table(struct cpufreq_policy *policy,
ret = sysfs_create_group(&policy->kobj, &stats_attr_group);
if (ret) {
pr_err("Failed to create cpufreq_stats sysfs\n");
goto error_out;
stat->cpu = cpu;
per_cpu(cpufreq_stats_table, cpu) = stat;
......@@ -634,7 +629,6 @@ static int __cpufreq_stats_create_table(struct cpufreq_policy *policy,
return 0;
sysfs_remove_group(&policy->kobj, &stats_attr_group);
per_cpu(cpufreq_stats_table, cpu) = NULL;
return ret;
......@@ -850,7 +844,7 @@ static int cpufreq_stat_notifier_policy(struct notifier_block *nb,
cpufreq_powerstats_create(cpu, table, count);
ret = __cpufreq_stats_create_table(policy, table, count);
ret = __cpufreq_stats_create_table(policy, cpu, table, count);
return ret;
......@@ -885,7 +879,7 @@ static void cpufreq_stats_create_table(unsigned int cpu)
cpufreq_powerstats_create(cpu, table, count);
__cpufreq_stats_create_table(policy, table, count);
__cpufreq_stats_create_table(policy, cpu, table, count);
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment