Page MenuHomeFreeBSD

Add powerd(8) support for AMD family 17h (Zen) CPUs
ClosedPublic

Authored by cem on Aug 19 2017, 6:45 AM.
Tags
None
Referenced Files
F85204283: D12082.id.diff
Mon, Jun 3, 1:06 AM
F85184086: D12082.diff
Sun, Jun 2, 6:03 PM
Unknown Object (File)
Mon, May 27, 10:10 PM
Unknown Object (File)
Thu, May 9, 5:23 AM
Unknown Object (File)
Thu, May 9, 2:29 AM
Unknown Object (File)
Tue, May 7, 1:50 PM
Unknown Object (File)
Sat, May 4, 5:27 PM
Unknown Object (File)
Apr 27 2024, 1:46 PM
Subscribers
None

Details

Summary

Family 17h CPUs use a slightly different frequency formula than previous
generation models.

PR: 221621

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 11167
Build 11549: arc lint + arc unit

Event Timeline

So I will note that on my system hwpstate ends up opting to get these from acpi_perf:

kernel: hwpstate0: going to fetch info from acpi_perf
...
mjoras@icarium ~> sysctl dev.hwpstate.0.freq_settings
dev.hwpstate.0.freq_settings: 3400/3825 2800/2765 2200/1952

Forcing it to come from an MSR and using your patch I get:

dev.hwpstate.0.freq_settings: 3400/-1 2800/-1 2200/-1

So that looks good. The power dissipation is unknown, but apparently available when using ACPI. This is the case for older families too. powerd(8) only uses the power information in verbose mode to track total energy usage, apparently.

This patch doesn't fix the PR, as already noted there. It seems like this would in fact fix powerd(8) for the case where getting the information from ACPI fails, so I might mention that this fixes it for the MSR case specifically in the title.

This revision is now accepted and ready to land.Aug 20 2017, 12:32 AM
This revision was automatically updated to reflect the committed changes.