cpufreq_stats: Adds the fucntionality to load current values for each frequency
authorRuchi Kandoi <kandoiruchi@google.com>
Thu, 16 Apr 2015 23:32:02 +0000 (16:32 -0700)
committerRuchi Kandoi <kandoiruchi@google.com>
Sat, 30 May 2015 01:12:25 +0000 (01:12 +0000)
for all the cores.

The current values for the cpu cores needs to be added to the device
tree for this functionaly to work. It loads the current values for each
frequecy in uA for all the cores.

Change-Id: If03311aaeb3e4c09375dd0beb9ad4fbb254b5c08
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
drivers/cpufreq/cpufreq_stats.c

index d811f5d4b32b32c1aede467a88d222718e3451bb..89889bff56151c9a80b1a30c7caa3e4f5a801b25 100644 (file)
@@ -22,6 +22,7 @@
 #include <linux/notifier.h>
 #include <linux/sort.h>
 #include <linux/err.h>
+#include <linux/of.h>
 #include <asm/cputime.h>
 
 static spinlock_t cpufreq_stats_lock;
@@ -46,6 +47,12 @@ struct all_cpufreq_stats {
        unsigned int *freq_table;
 };
 
+struct cpufreq_power_stats {
+       unsigned int state_num;
+       unsigned int *curr;
+       unsigned int *freq_table;
+};
+
 struct all_freq_table {
        unsigned int *freq_table;
        unsigned int table_size;
@@ -55,6 +62,7 @@ static struct all_freq_table *all_freq_table;
 
 static DEFINE_PER_CPU(struct all_cpufreq_stats *, all_cpufreq_stats);
 static DEFINE_PER_CPU(struct cpufreq_stats *, cpufreq_stats_table);
+static DEFINE_PER_CPU(struct cpufreq_power_stats *, cpufreq_power_stats);
 
 struct cpufreq_stats_attribute {
        struct attribute attr;
@@ -125,6 +133,29 @@ static int get_index_all_cpufreq_stat(struct all_cpufreq_stats *all_stat,
        return -1;
 }
 
+static ssize_t show_current_in_state(struct kobject *kobj,
+               struct kobj_attribute *attr, char *buf)
+{
+       ssize_t len = 0;
+       unsigned int i, cpu;
+       struct cpufreq_power_stats *powerstats;
+
+       spin_lock(&cpufreq_stats_lock);
+       for_each_possible_cpu(cpu) {
+               powerstats = per_cpu(cpufreq_power_stats, cpu);
+               if (!powerstats)
+                       continue;
+               len += scnprintf(buf + len, PAGE_SIZE - len, "CPU%d:", cpu);
+               for (i = 0; i < powerstats->state_num; i++)
+                       len += scnprintf(buf + len, PAGE_SIZE - len,
+                                       "%d=%d ", powerstats->freq_table[i],
+                                       powerstats->curr[i]);
+               len += scnprintf(buf + len, PAGE_SIZE - len, "\n");
+       }
+       spin_unlock(&cpufreq_stats_lock);
+       return len;
+}
+
 static ssize_t show_all_time_in_state(struct kobject *kobj,
                struct kobj_attribute *attr, char *buf)
 {
@@ -234,6 +265,9 @@ static struct attribute_group stats_attr_group = {
 static struct kobj_attribute _attr_all_time_in_state = __ATTR(all_time_in_state,
                0444, show_all_time_in_state, NULL);
 
+static struct kobj_attribute _attr_current_in_state = __ATTR(current_in_state,
+               0444, show_current_in_state, NULL);
+
 static int freq_table_get_index(struct cpufreq_stats *stat, unsigned int freq)
 {
        int index;
@@ -303,10 +337,27 @@ static void cpufreq_allstats_free(void)
        }
 }
 
+static void cpufreq_powerstats_free(void)
+{
+       int cpu;
+       struct cpufreq_power_stats *powerstats;
+
+       sysfs_remove_file(cpufreq_global_kobject, &_attr_current_in_state.attr);
+
+       for_each_possible_cpu(cpu) {
+               powerstats = per_cpu(cpufreq_power_stats, cpu);
+               if (!powerstats)
+                       continue;
+               kfree(powerstats->curr);
+               kfree(powerstats);
+               per_cpu(cpufreq_power_stats, cpu) = NULL;
+       }
+}
+
 static int cpufreq_stats_create_table(struct cpufreq_policy *policy,
-               struct cpufreq_frequency_table *table)
+               struct cpufreq_frequency_table *table, int count)
 {
-       unsigned int i, j, count = 0, ret = 0;
+       unsigned int i, j, ret = 0;
        struct cpufreq_stats *stat;
        struct cpufreq_policy *data;
        unsigned int alloc_size;
@@ -330,12 +381,6 @@ static int cpufreq_stats_create_table(struct cpufreq_policy *policy,
        stat->cpu = cpu;
        per_cpu(cpufreq_stats_table, cpu) = stat;
 
-       for (i = 0; table[i].frequency != CPUFREQ_TABLE_END; i++) {
-               unsigned int freq = table[i].frequency;
-               if (freq == CPUFREQ_ENTRY_INVALID)
-                       continue;
-               count++;
-       }
 
        alloc_size = count * sizeof(int) + count * sizeof(u64);
 
@@ -389,6 +434,54 @@ static void cpufreq_stats_update_policy_cpu(struct cpufreq_policy *policy)
        stat->cpu = policy->cpu;
 }
 
+static void cpufreq_powerstats_create(unsigned int cpu,
+               struct cpufreq_frequency_table *table, int count) {
+       unsigned int alloc_size, i = 0, j = 0, ret = 0;
+       struct cpufreq_power_stats *powerstats;
+       struct device_node *cpu_node;
+       char device_path[16];
+
+       powerstats = kzalloc(sizeof(struct cpufreq_power_stats),
+                       GFP_KERNEL);
+       if (!powerstats)
+               return;
+
+       /* Allocate memory for freq table per cpu as well as clockticks per
+        * freq*/
+       alloc_size = count * sizeof(unsigned int) +
+               count * sizeof(unsigned int);
+       powerstats->curr = kzalloc(alloc_size, GFP_KERNEL);
+       if (!powerstats->curr) {
+               kfree(powerstats);
+               return;
+       }
+       powerstats->freq_table = powerstats->curr + count;
+
+       spin_lock(&cpufreq_stats_lock);
+       for (i = 0; table[i].frequency != CPUFREQ_TABLE_END && j < count; i++) {
+               unsigned int freq = table[i].frequency;
+
+               if (freq == CPUFREQ_ENTRY_INVALID)
+                       continue;
+               powerstats->freq_table[j++] = freq;
+       }
+       powerstats->state_num = j;
+
+       snprintf(device_path, sizeof(device_path), "/cpus/cpu@%d", cpu);
+       cpu_node = of_find_node_by_path(device_path);
+       if (cpu_node) {
+               ret = of_property_read_u32_array(cpu_node, "current",
+                               powerstats->curr, count);
+               if (ret) {
+                       kfree(powerstats->curr);
+                       kfree(powerstats);
+                       powerstats = NULL;
+               }
+       }
+       per_cpu(cpufreq_power_stats, cpu) = powerstats;
+       spin_unlock(&cpufreq_stats_lock);
+}
+
 static int compare_for_sort(const void *lhs_ptr, const void *rhs_ptr)
 {
        unsigned int lhs = *(const unsigned int *)(lhs_ptr);
@@ -433,24 +526,14 @@ static void add_all_freq_table(unsigned int freq)
        all_freq_table->freq_table[all_freq_table->table_size++] = freq;
 }
 
-static void cpufreq_allstats_create(unsigned int cpu)
+static void cpufreq_allstats_create(unsigned int cpu,
+               struct cpufreq_frequency_table *table, int count)
 {
        int i , j = 0;
-       unsigned int alloc_size, count = 0;
-       struct cpufreq_frequency_table *table = cpufreq_frequency_get_table(cpu);
+       unsigned int alloc_size;
        struct all_cpufreq_stats *all_stat;
        bool sort_needed = false;
 
-       if (!table)
-               return;
-
-       for (i = 0; table[i].frequency != CPUFREQ_TABLE_END; i++) {
-               unsigned int freq = table[i].frequency;
-               if (freq == CPUFREQ_ENTRY_INVALID)
-                       continue;
-               count++;
-       }
-
        all_stat = kzalloc(sizeof(struct all_cpufreq_stats),
                        GFP_KERNEL);
        if (!all_stat) {
@@ -492,7 +575,7 @@ static void cpufreq_allstats_create(unsigned int cpu)
 static int cpufreq_stat_notifier_policy(struct notifier_block *nb,
                unsigned long val, void *data)
 {
-       int ret;
+       int ret, count = 0, i;
        struct cpufreq_policy *policy = data;
        struct cpufreq_frequency_table *table;
        unsigned int cpu = policy->cpu;
@@ -508,10 +591,21 @@ static int cpufreq_stat_notifier_policy(struct notifier_block *nb,
        if (!table)
                return 0;
 
+       for (i = 0; table[i].frequency != CPUFREQ_TABLE_END; i++) {
+               unsigned int freq = table[i].frequency;
+
+               if (freq == CPUFREQ_ENTRY_INVALID)
+                       continue;
+               count++;
+       }
+
        if (!per_cpu(all_cpufreq_stats, cpu))
-               cpufreq_allstats_create(cpu);
+               cpufreq_allstats_create(cpu, table, count);
+
+       if (!per_cpu(cpufreq_power_stats, cpu))
+               cpufreq_powerstats_create(cpu, table, count);
 
-       ret = cpufreq_stats_create_table(policy, table);
+       ret = cpufreq_stats_create_table(policy, table, count);
        if (ret)
                return ret;
        return 0;
@@ -557,7 +651,7 @@ static int cpufreq_stats_create_table_cpu(unsigned int cpu)
 {
        struct cpufreq_policy *policy;
        struct cpufreq_frequency_table *table;
-       int ret = -ENODEV;
+       int ret = -ENODEV, i, count = 0;
 
        policy = cpufreq_cpu_get(cpu);
        if (!policy)
@@ -567,10 +661,21 @@ static int cpufreq_stats_create_table_cpu(unsigned int cpu)
        if (!table)
                goto out;
 
+       for (i = 0; table[i].frequency != CPUFREQ_TABLE_END; i++) {
+               unsigned int freq = table[i].frequency;
+
+               if (freq == CPUFREQ_ENTRY_INVALID)
+                       continue;
+               count++;
+       }
+
        if (!per_cpu(all_cpufreq_stats, cpu))
-               cpufreq_allstats_create(cpu);
+               cpufreq_allstats_create(cpu, table, count);
+
+       if (!per_cpu(cpufreq_power_stats, cpu))
+               cpufreq_powerstats_create(cpu, table, count);
 
-       ret = cpufreq_stats_create_table(policy, table);
+       ret = cpufreq_stats_create_table(policy, table, count);
 
 out:
        cpufreq_cpu_put(policy);
@@ -649,7 +754,12 @@ static int __init cpufreq_stats_init(void)
        ret = sysfs_create_file(cpufreq_global_kobject,
                        &_attr_all_time_in_state.attr);
        if (ret)
-               pr_warn("Error creating sysfs file for cpufreq stats\n");
+               pr_warn("Cannot create sysfs file for cpufreq stats\n");
+
+       ret = sysfs_create_file(cpufreq_global_kobject,
+                       &_attr_current_in_state.attr);
+       if (ret)
+               pr_warn("Cannot create sysfs file for cpufreq current stats\n");
 
        return 0;
 }
@@ -667,6 +777,7 @@ static void __exit cpufreq_stats_exit(void)
                cpufreq_stats_free_sysfs(cpu);
        }
        cpufreq_allstats_free();
+       cpufreq_powerstats_free();
 }
 
 MODULE_AUTHOR("Zou Nan hai <nanhai.zou@intel.com>");