Skip to content

cpu: fix cpu_power=0W on Zen 5 when cpu_temp is also enabled #2042

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Motaphe wants to merge 2 commits into
base: master
Choose a base branch
from
+28 −16
Open

cpu: fix cpu_power=0W on Zen 5 when cpu_temp is also enabled #2042

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
43773cb
cpu: fix cpu_power=0W on Zen 5 when cpu_temp is also enabled
Motaphe May 4, 2026
78a75d8
Merge branch 'master' into fix/zenpower5-cpu-power-zen5
Motaphe Jun 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 28 additions & 16 deletions src/cpu.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -359,21 +359,24 @@ static bool get_cpu_power_k10temp(CPUPowerData* cpuPowerData, float& power) {
static bool get_cpu_power_zenpower(CPUPowerData* cpuPowerData, float& power) {
CPUPowerData_zenpower* powerData_zenpower = (CPUPowerData_zenpower*)cpuPowerData;

if (!powerData_zenpower->corePowerFile || !powerData_zenpower->socPowerFile)
if (!powerData_zenpower->corePowerFile)
return false;

rewind(powerData_zenpower->corePowerFile);
rewind(powerData_zenpower->socPowerFile);

fflush(powerData_zenpower->corePowerFile);
fflush(powerData_zenpower->socPowerFile);

int corePower, socPower;
int corePower = 0, socPower = 0;

if (fscanf(powerData_zenpower->corePowerFile, "%d", &corePower) != 1)
return false;
if (fscanf(powerData_zenpower->socPowerFile, "%d", &socPower) != 1)
return false;

// socPowerFile is null for zenpower5/Zen5 which reports a single RAPL_P_Package value
if (powerData_zenpower->socPowerFile) {
rewind(powerData_zenpower->socPowerFile);
fflush(powerData_zenpower->socPowerFile);
if (fscanf(powerData_zenpower->socPowerFile, "%d", &socPower) != 1)
return false;
}

power = (corePower + socPower) / 1000000;

Expand Down Expand Up @@ -677,16 +680,25 @@ static CPUPowerData_zenpower* init_cpu_power_data_zenpower(const std::string pat

std::string corePowerInput, socPowerInput;

if(!find_input(path, "power", corePowerInput, "SVI2_P_Core")) return nullptr;
if(!find_input(path, "power", socPowerInput, "SVI2_P_SoC")) return nullptr;

SPDLOG_DEBUG("hwmon: using input: {}", corePowerInput);
SPDLOG_DEBUG("hwmon: using input: {}", socPowerInput);
// Zen 4 and older: SVI2 separate core and SoC power rails
if(find_input(path, "power", corePowerInput, "SVI2_P_Core") &&
find_input(path, "power", socPowerInput, "SVI2_P_SoC")) {
SPDLOG_DEBUG("hwmon: using input: {}", corePowerInput);
SPDLOG_DEBUG("hwmon: using input: {}", socPowerInput);
powerData->corePowerFile = fopen(corePowerInput.c_str(), "r");
powerData->socPowerFile = fopen(socPowerInput.c_str(), "r");
return powerData.release();
}

powerData->corePowerFile = fopen(corePowerInput.c_str(), "r");
powerData->socPowerFile = fopen(socPowerInput.c_str(), "r");
// Zen 5 (zenpower5): single RAPL package power reading
if(find_input(path, "power", corePowerInput, "RAPL_P_Package")) {
SPDLOG_DEBUG("hwmon: using input (RAPL_P_Package): {}", corePowerInput);
powerData->corePowerFile = fopen(corePowerInput.c_str(), "r");
// socPowerFile intentionally left null; get_cpu_power_zenpower handles this
return powerData.release();
}

return powerData.release();
return nullptr;
}

static CPUPowerData_zenergy* init_cpu_power_data_zenergy(const std::string path) {
Expand Down Expand Up @@ -755,7 +767,7 @@ bool CPUStats::InitCpuPowerData() {
cpuPowerData = (CPUPowerData*)init_cpu_power_data_k10temp(path);
} else if (name == "zenpower") {
cpuPowerData = (CPUPowerData*)init_cpu_power_data_zenpower(path);
break;
if (cpuPowerData) break;
} else if (name == "zenergy") {
cpuPowerData = (CPUPowerData*)init_cpu_power_data_zenergy(path);
break;
Expand Down