Skip to content

Commit

Permalink
cpupower: Add the function to check AMD P-State enabled
Browse files Browse the repository at this point in the history
The processor with AMD P-State function also supports legacy ACPI
hardware P-States feature as well. Once driver sets AMD P-State eanbled,
the processor will respond the finer grain AMD P-State feature instead of
legacy ACPI P-States. So it introduces the cpupower_amd_pstate_enabled()
to check whether the current kernel enables AMD P-State or AMD CPUFreq
module.

Signed-off-by: Huang Rui <ray.huang@amd.com>
  • Loading branch information
huangrui authored and xanmod committed Feb 23, 2022
1 parent 762e040 commit 4fe6d24
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 0 deletions.
10 changes: 10 additions & 0 deletions tools/power/cpupower/utils/helpers/helpers.h
Expand Up @@ -11,6 +11,7 @@

#include <libintl.h>
#include <locale.h>
#include <stdbool.h>

#include "helpers/bitmask.h"
#include <cpupower.h>
Expand Down Expand Up @@ -136,6 +137,12 @@ extern int decode_pstates(unsigned int cpu, int boost_states,

extern int cpufreq_has_boost_support(unsigned int cpu, int *support,
int *active, int * states);

/* AMD P-State stuff **************************/
bool cpupower_amd_pstate_enabled(void);

/* AMD P-State stuff **************************/

/*
* CPUID functions returning a single datum
*/
Expand Down Expand Up @@ -168,6 +175,9 @@ static inline int cpufreq_has_boost_support(unsigned int cpu, int *support,
int *active, int * states)
{ return -1; }

static inline bool cpupower_amd_pstate_enabled(void)
{ return false; }

/* cpuid and cpuinfo helpers **************************/

static inline unsigned int cpuid_eax(unsigned int op) { return 0; };
Expand Down
18 changes: 18 additions & 0 deletions tools/power/cpupower/utils/helpers/misc.c
Expand Up @@ -3,9 +3,11 @@
#include <stdio.h>
#include <errno.h>
#include <stdlib.h>
#include <string.h>

#include "helpers/helpers.h"
#include "helpers/sysfs.h"
#include "cpufreq.h"

#if defined(__i386__) || defined(__x86_64__)

Expand Down Expand Up @@ -83,6 +85,22 @@ int cpupower_intel_set_perf_bias(unsigned int cpu, unsigned int val)
return 0;
}

bool cpupower_amd_pstate_enabled(void)
{
char *driver = cpufreq_get_driver(0);
bool ret = false;

if (!driver)
return ret;

if (!strcmp(driver, "amd-pstate"))
ret = true;

cpufreq_put_driver(driver);

return ret;
}

#endif /* #if defined(__i386__) || defined(__x86_64__) */

/* get_cpustate
Expand Down

0 comments on commit 4fe6d24

Please sign in to comment.