Commit 2c0bf86c authored by Rafael J. Wysocki's avatar Rafael J. Wysocki

Merge tag 'linux-cpupower-5.1-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux

Pull cpupower updates for 5.1-rc1 from Shuah Khan:

"This cpupower update for Linux 5.1-rc1 consists of a patch to add
 support to display boost frequency separately from Abhishek Goel."

* tag 'linux-cpupower-5.1-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux:
  tools/power/cpupower: Display boost frequency separately
parents 5908e6b7 ae291709
...@@ -333,17 +333,20 @@ void cpufreq_put_available_governors(struct cpufreq_available_governors *any) ...@@ -333,17 +333,20 @@ void cpufreq_put_available_governors(struct cpufreq_available_governors *any)
} }
struct cpufreq_available_frequencies struct cpufreq_frequencies
*cpufreq_get_available_frequencies(unsigned int cpu) *cpufreq_get_frequencies(const char *type, unsigned int cpu)
{ {
struct cpufreq_available_frequencies *first = NULL; struct cpufreq_frequencies *first = NULL;
struct cpufreq_available_frequencies *current = NULL; struct cpufreq_frequencies *current = NULL;
char one_value[SYSFS_PATH_MAX]; char one_value[SYSFS_PATH_MAX];
char linebuf[MAX_LINE_LEN]; char linebuf[MAX_LINE_LEN];
char fname[MAX_LINE_LEN];
unsigned int pos, i; unsigned int pos, i;
unsigned int len; unsigned int len;
len = sysfs_cpufreq_read_file(cpu, "scaling_available_frequencies", snprintf(fname, MAX_LINE_LEN, "scaling_%s_frequencies", type);
len = sysfs_cpufreq_read_file(cpu, fname,
linebuf, sizeof(linebuf)); linebuf, sizeof(linebuf));
if (len == 0) if (len == 0)
return NULL; return NULL;
...@@ -389,9 +392,9 @@ struct cpufreq_available_frequencies ...@@ -389,9 +392,9 @@ struct cpufreq_available_frequencies
return NULL; return NULL;
} }
void cpufreq_put_available_frequencies(struct cpufreq_available_frequencies void cpufreq_put_frequencies(struct cpufreq_frequencies *any)
*any) { {
struct cpufreq_available_frequencies *tmp, *next; struct cpufreq_frequencies *tmp, *next;
if (!any) if (!any)
return; return;
......
...@@ -28,10 +28,10 @@ struct cpufreq_available_governors { ...@@ -28,10 +28,10 @@ struct cpufreq_available_governors {
struct cpufreq_available_governors *first; struct cpufreq_available_governors *first;
}; };
struct cpufreq_available_frequencies { struct cpufreq_frequencies {
unsigned long frequency; unsigned long frequency;
struct cpufreq_available_frequencies *next; struct cpufreq_frequencies *next;
struct cpufreq_available_frequencies *first; struct cpufreq_frequencies *first;
}; };
...@@ -129,14 +129,14 @@ void cpufreq_put_available_governors( ...@@ -129,14 +129,14 @@ void cpufreq_put_available_governors(
* *
* Only present on _some_ ->target() cpufreq drivers. For information purposes * Only present on _some_ ->target() cpufreq drivers. For information purposes
* only. Please free allocated memory by calling * only. Please free allocated memory by calling
* cpufreq_put_available_frequencies after use. * cpufreq_put_frequencies after use.
*/ */
struct cpufreq_available_frequencies struct cpufreq_frequencies
*cpufreq_get_available_frequencies(unsigned int cpu); *cpufreq_get_frequencies(const char *type, unsigned int cpu);
void cpufreq_put_available_frequencies( void cpufreq_put_frequencies(
struct cpufreq_available_frequencies *first); struct cpufreq_frequencies *first);
/* determine affected CPUs /* determine affected CPUs
......
...@@ -161,19 +161,12 @@ static void print_duration(unsigned long duration) ...@@ -161,19 +161,12 @@ static void print_duration(unsigned long duration)
return; return;
} }
/* --boost / -b */ static int get_boost_mode_x86(unsigned int cpu)
static int get_boost_mode(unsigned int cpu)
{ {
int support, active, b_states = 0, ret, pstate_no, i; int support, active, b_states = 0, ret, pstate_no, i;
/* ToDo: Make this more global */ /* ToDo: Make this more global */
unsigned long pstates[MAX_HW_PSTATES] = {0,}; unsigned long pstates[MAX_HW_PSTATES] = {0,};
if (cpupower_cpu_info.vendor != X86_VENDOR_AMD &&
cpupower_cpu_info.vendor != X86_VENDOR_HYGON &&
cpupower_cpu_info.vendor != X86_VENDOR_INTEL)
return 0;
ret = cpufreq_has_boost_support(cpu, &support, &active, &b_states); ret = cpufreq_has_boost_support(cpu, &support, &active, &b_states);
if (ret) { if (ret) {
printf(_("Error while evaluating Boost Capabilities" printf(_("Error while evaluating Boost Capabilities"
...@@ -248,6 +241,33 @@ static int get_boost_mode(unsigned int cpu) ...@@ -248,6 +241,33 @@ static int get_boost_mode(unsigned int cpu)
return 0; return 0;
} }
/* --boost / -b */
static int get_boost_mode(unsigned int cpu)
{
struct cpufreq_frequencies *freqs;
if (cpupower_cpu_info.vendor == X86_VENDOR_AMD ||
cpupower_cpu_info.vendor == X86_VENDOR_HYGON ||
cpupower_cpu_info.vendor == X86_VENDOR_INTEL)
return get_boost_mode_x86(cpu);
freqs = cpufreq_get_frequencies("boost", cpu);
if (freqs) {
printf(_(" boost frequency steps: "));
while (freqs->next) {
print_speed(freqs->frequency);
printf(", ");
freqs = freqs->next;
}
print_speed(freqs->frequency);
printf("\n");
cpufreq_put_frequencies(freqs);
}
return 0;
}
/* --freq / -f */ /* --freq / -f */
static int get_freq_kernel(unsigned int cpu, unsigned int human) static int get_freq_kernel(unsigned int cpu, unsigned int human)
...@@ -456,7 +476,7 @@ static int get_latency(unsigned int cpu, unsigned int human) ...@@ -456,7 +476,7 @@ static int get_latency(unsigned int cpu, unsigned int human)
static void debug_output_one(unsigned int cpu) static void debug_output_one(unsigned int cpu)
{ {
struct cpufreq_available_frequencies *freqs; struct cpufreq_frequencies *freqs;
get_driver(cpu); get_driver(cpu);
get_related_cpus(cpu); get_related_cpus(cpu);
...@@ -464,7 +484,7 @@ static void debug_output_one(unsigned int cpu) ...@@ -464,7 +484,7 @@ static void debug_output_one(unsigned int cpu)
get_latency(cpu, 1); get_latency(cpu, 1);
get_hardware_limits(cpu, 1); get_hardware_limits(cpu, 1);
freqs = cpufreq_get_available_frequencies(cpu); freqs = cpufreq_get_frequencies("available", cpu);
if (freqs) { if (freqs) {
printf(_(" available frequency steps: ")); printf(_(" available frequency steps: "));
while (freqs->next) { while (freqs->next) {
...@@ -474,7 +494,7 @@ static void debug_output_one(unsigned int cpu) ...@@ -474,7 +494,7 @@ static void debug_output_one(unsigned int cpu)
} }
print_speed(freqs->frequency); print_speed(freqs->frequency);
printf("\n"); printf("\n");
cpufreq_put_available_frequencies(freqs); cpufreq_put_frequencies(freqs);
} }
get_available_governors(cpu); get_available_governors(cpu);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment