AMD Pstate Driver Core Performance Boost
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v13 Latest9 patches 2 repliesIn review
- v129 patches 14 repliesSuperseded
- v119 patches 14 repliesSuperseded
Earlier versions · 10
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v10 | 7 patches | 15 replies | Superseded | |
| v9 | 7 patches | 3 replies | Superseded | |
| v8 | 6 patches | 6 replies | Superseded | |
| v7 | 6 patches | 8 replies | Superseded | |
| v6 | 6 patches | 5 replies | Superseded | |
| v5 | 8 patches | 1 reply | Superseded | |
| v4 | 7 patches | 10 replies | Superseded | |
| v3 | 7 patches | 7 replies | Superseded | |
| v2 | 6 patches | 2 replies | Superseded | |
| v1 | 7 patches | 11 replies | Superseded |
Changes in v13 from the cover letter
* fix the braces for patch #3(Mario)
* fix the return value for patch #4(Mario)
* use enabled and disabled for cpb_boost (Mario)
* pick RB flags and Ack flags from Mario and Gautham
* update commit log for patch #7 (Gautham)
* add more details to doc for the patch #8(Mario)
* address feedback from Gautham
* rebaed to Mario kernel branch
* fix the commit typos for patches #5(Mario)
* update cover letter for cpb_boost state change.
* tested on shared memory system for cpb_boost control(no regression found)
Patches in v13 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | cpufreq: acpi: move MSR_K7_HWCR_CPB_DIS_BIT into msr-index.harch/ drivers/ | +2 | 0 replies | |
| 2/9 | cpufreq: simplify boolean parsing with kstrtobool in store functiondrivers/ | +5 | 0 replies | |
| 3/9 | cpufreq: introduce init_boost callback to initialize boost state for pstate driversdrivers/ include/ | +14 | 1 reply | |
| 4/9 | cpufreq: amd-pstate: initialize new core precision boost statedrivers/ drivers/ | +67 | 0 replies | |
| 5/9 | cpufreq: amd-pstate: implement cpb_boost sysfs entry for boost controldrivers/ drivers/ drivers/ | +130 | 0 replies | |
| 6/9 | cpufreq: amd-pstate: Add set_boost callback for active modedrivers/ | +5 | 0 replies | |
| 7/9 | cpufreq:amd-pstate: Cap the CPPC.max_perf to nominal_perf if CPB is offdrivers/ | +5 | 0 replies | |
| 8/9 | Documentation: cpufreq: amd-pstate: introduce the new cpu boost control methodDocumentation/ | +17 | 0 replies | |
| 9/9 | Documentation: cpufreq: amd-pstate: update doc for Per CPU boost control methodDocumentation/ | +21 | 0 replies |
Reviews trailers per patch
6 of 9 patches have a Reviewed-by or Acked-by.
| Patch | Rafael J. Wysocki | Huang Rui | Mario Limonciello | Gautham R. Shenoy |
|---|---|---|---|---|
| 1/9 | A-b | A-b | ||
| 2/9 | R-b | R-b | ||
| 3/9 | ||||
| 4/9 | ||||
| 5/9 | ||||
| 6/9 | R-b | A-b | ||
| 7/9 | A-b | R-b | R-b | |
| 8/9 | R-b | |||
| 9/9 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.
Cover letter [PATCH v13 0/9] AMD Pstate Driver Core Performance Boost
Hi all,
This patch series implements the Core Performance Boost (CPB) feature for the AMD pstate driver, including support for passive, guided, and active modes.
Users can change the global core frequency boost control with a new sysfs entry:
/sys/devices/system/cpu/amd_pstate/cpb_boost
It also supports updating the individual CPU boost state in the sysfs boost file:
/sys/devices/system/cpu/cpuX/cpufreq/boost
By default, the global control will override the individual CPU boost state.
1) disable core boost globally:
$ sudo bash -c "echo "disabled" > /sys/devices/system/cpu/amd_pstate/cpb_boost" $ cat /sys/devices/system/cpu/amd_pstate/cpb_boost disabled
$ lscpu -ae CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ 0 0 0 0 0:0:0:0 yes 4201.0000 400.0000 2983.578 1 0 0 1 1:1:1:0 yes 4201.0000 400.0000 2983.578 2 0 0 2 2:2:2:0 yes 4201.0000 400.0000 2583.855 3 0 0 3 3:3:3:0 yes 4201.0000 400.0000 2983.578 4 0 0 4 4:4:4:0 yes 4201.0000 400.0000 2983.578
2) enable core boost globally:
$ sudo bash -c "echo "enabled" > /sys/devices/system/cpu/amd_pstate/cpb_boost" $ cat /sys/devices/system/cpu/amd_pstate/cpb_boost enabled $ lscpu -ae 0 0 0 0 0:0:0:0 yes 5759.0000 400.0000 2983.578 1 0 0 1 1:1:1:0 yes 5759.0000 400.0000 2983.578 2 0 0 2 2:2:2:0 yes 5759.0000 400.0000 2983.578 3 0 0 3 3:3:3:0 yes 5759.0000 400.0000 2983.578 4 0 0 4 4:4:4:0 yes 5759.0000 400.0000 2983.578
============================================================================
The V9 patches add per CPU boost control, user can enable/disable CPUs boost as the below command tested on a laptop system.
# before CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ 0 0 0 0 0:0:0:0 yes 4208.0000 400.0000 1666.7740 1 0 0 0 0:0:0:0 yes 4208.0000 400.0000 400.0000 2 0 0 1 1:1:1:0 yes 4208.0000 400.0000 3386.1260 3 0 0 1 1:1:1:0 yes 4208.0000 400.0000 400.0000 $ sudo rdmsr 0xc00102b3 -p 0 10a6
$ sudo bash -c "echo 1 > /sys/devices/system/cpu/cpu0/cpufreq/boost"
# after
CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ
0 0 0 0 0:0:0:0 yes 3501.0000 400.0000 400.0000
1 0 0 0 0:0:0:0 yes 4208.0000 400.0000 1391.0690
2 0 0 1 1:1:1:0 yes 4208.0000 400.0000 3654.4541
3 0 0 1 1:1:1:0 yes 4208.0000 400.0000 400.0000
$ sudo rdmsr 0xc00102b3 -p 0
108arebasd to keep syncing to Mario kernel tree: bleeding-edge
https://git.kernel.org/pub/scm/linux/kernel/git/superm1/linux.git
Tested result:
CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ
0 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1110.7140 1 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1110.7140 2 0 0 1 1:1:1:0 yes 4354.0000 400.0000 1110.7140 3 0 0 1 1:1:1:0 yes 4354.0000 400.0000 3110.0000 4 0 0 2 2:2:2:0 yes 4354.0000 400.0000 2732.3569 5 0 0 2 2:2:2:0 yes 4354.0000 400.0000 1110.7140 6 0 0 3 3:3:3:0 yes 4354.0000 400.0000 1110.7140 7 0 0 3 3:3:3:0 yes 4354.0000 400.0000 1110.7140 8 0 0 4 4:4:4:0 yes 4354.0000 400.0000 2312.2109 9 0 0 4 4:4:4:0 yes 4354.0000 400.0000 1110.7140 10 0 0 5 5:5:5:0 yes 4354.0000 400.0000 2310.1011 11 0 0 5 5:5:5:0 yes 4354.0000 400.0000 1110.7140
sudo bash -c "echo 0 > /sys/devices/system/cpu/cpu11/cpufreq/boost" sudo bash -c "echo 0 > /sys/devices/system/cpu/cpu10/cpufreq/boost" sudo bash -c "echo 0 > /sys/devices/system/cpu/cpu9/cpufreq/boost"
CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ
0 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1242.7240 1 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1110.7140 2 0 0 1 1:1:1:0 yes 4354.0000 400.0000 2754.5710 3 0 0 1 1:1:1:0 yes 4354.0000 400.0000 2659.8159 4 0 0 2 2:2:2:0 yes 4354.0000 400.0000 2308.9929 5 0 0 2 2:2:2:0 yes 4354.0000 400.0000 1110.7140 6 0 0 3 3:3:3:0 yes 4354.0000 400.0000 1110.7140 7 0 0 3 3:3:3:0 yes 4354.0000 400.0000 1110.7140 8 0 0 4 4:4:4:0 yes 4354.0000 400.0000 1110.7140 9 0 0 4 4:4:4:0 yes 2801.0000 400.0000 1110.7140 10 0 0 5 5:5:5:0 yes 2801.0000 400.0000 1110.7140 11 0 0 5 5:5:5:0 yes 2801.0000 400.0000 1110.7140
sudo bash -c "echo 0 > /sys/devices/system/cpu/amd_pstate/cpb_boost"
CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ
0 0 0 0 0:0:0:0 yes 2801.0000 400.0000 1233.8630 1 0 0 0 0:0:0:0 yes 2801.0000 400.0000 1110.7140 2 0 0 1 1:1:1:0 yes 2801.0000 400.0000 2714.4851 3 0 0 1 1:1:1:0 yes 2801.0000 400.0000 2732.3569 4 0 0 2 2:2:2:0 yes 2801.0000 400.0000 2564.2639 5 0 0 2 2:2:2:0 yes 2801.0000 400.0000 1110.7140 6 0 0 3 3:3:3:0 yes 2801.0000 400.0000 2732.3569 7 0 0 3 3:3:3:0 yes 2801.0000 400.0000 1110.7140 8 0 0 4 4:4:4:0 yes 2801.0000 400.0000 1233.8660 9 0 0 4 4:4:4:0 yes 2801.0000 400.0000 1110.7140 10 0 0 5 5:5:5:0 yes 2801.0000 400.0000 1233.6630 11 0 0 5 5:5:5:0 yes 2801.0000 400.0000 1233.5050
sudo bash -c "echo 1 > /sys/devices/system/cpu/amd_pstate/cpb_boost"
CPU NODE SOCKET CORE L1d:L1i:L2:L3 ONLINE MAXMHZ MINMHZ MHZ
0 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1234.0200 1 0 0 0 0:0:0:0 yes 4354.0000 400.0000 1110.7140 2 0 0 1 1:1:1:0 yes 4354.0000 400.0000 1110.7140 3 0 0 1 1:1:1:0 yes 4354.0000 400.0000 1110.7140 4 0 0 2 2:2:2:0 yes 4354.0000 400.0000 1110.7140 5 0 0 2 2:2:2:0 yes 4354.0000 400.0000 1110.7140 6 0 0 3 3:3:3:0 yes 4354.0000 400.0000 2278.8491 7 0 0 3 3:3:3:0 yes 4354.0000 400.0000 2732.3569 8 0 0 4 4:4:4:0 yes 4354.0000 400.0000 1110.7140 9 0 0 4 4:4:4:0 yes 4354.0000 400.0000 1110.7140 10 0 0 5 5:5:5:0 yes 4354.0000 400.0000 2353.0449 11 0 0 5 5:5:5:0 yes 4354.0000 400.0000 1110.7140
Perry.
Changes from v12:
* fix the braces for patch #3(Mario)
* fix the return value for patch #4(Mario)
* use enabled and disabled for cpb_boost (Mario)
* pick RB flags and Ack flags from Mario and Gautham
* update commit log for patch #7 (Gautham)
* add more details to doc for the patch #8(Mario)
* address feedback from Gautham
* rebaed to Mario kernel branch
* fix the commit typos for patches #5(Mario)
* update cover letter for cpb_boost state change.
* tested on shared memory system for cpb_boost control(no regression found)
Changes from v11:
* add “goto out_free_policy" for error handling and update error mesasge for init_boost in cpufreq.c (Mario)
* drop dead code policy check in amd_pstate_cpu_boost_update (Mario)
* pick RB flags and Ack flags from Mario
* rebased to Mario bleeding edge kerne branch
Changes from v10:
* rework the boost interface with cpufreq core boost control, align the sysfs file created from cpufreq.c and allow indivial CPU boost control (Mario)
* fix the pr_warn code format with %zd (Oleksandr Natalenko)
* replace sscanf with kstrtobool for cpufreq.c (new)
* drop the boost sysfs file creation from amd pstate patch #6
* add init_boost for cpufreq.c to unify the boost file creation(Mario)
* add set_boost callback for EPP driver mode
* fix syncronization issue for indivial boost control and global CPB control, now the two control way will keep syncronization after anyone CPU boost state changed.
* rebased to Mario kernel tree: bleeding-edge
* run testing on local system, no regression issue found so far.
Changes from v9:
* change per CPU boost sysfs file name to boost (Mario)
* rebased to latest linux-pm/bleeding-edge
Changes from v8:
* pick RB flag for patch 4 (Mario)
* change boot_cpu_has to cpu_feature_enabled for patch 2 (Boris)
* merge patch 6 into patch 3 (Mario)
* add two patch for per CPU boost control patch 6 & 7(Mario)
* rebased to latest linux-pm/bleeding-edge
Changes from v7:
* fix the mutext locking issue in the sysfs file update(Ray, Mario)
* pick ack flag from Ray
* use X86_FEATURE_CPB to verify the CPB function in Patch #2(Ray)
* rerun the testing to check function works well
* rebased to linux-pm/bleeding-edge latest
Changes from v6:
* reword patch 2 commit log (Gautham)
* update cover letter description(Gautham)
* rebase to kernel v6.9-rc5
Changes from v4:
* drop the legacy boost remove patch, let us keep the legacy interface in case some applications break.
* rebase to linux-pm/bleeding-edge branch
* rework the patchset base on [PATCH v8 0/8] AMD Pstate Fixes And Enhancements which has some intial work done there.
Changes from v4:
* move MSR_K7_HWCR_CPB_DIS_BIT into msr-index.h
* pick RB flag from Gautham R. Shenoy
* add Cc Oleksandr Natalenko <[email protected]>
* rebase to latest linux-pm/bleeding-edge branch
* rebase the patch set on top of [PATCH v7 0/6] AMD Pstate Fixes And Enhancements
* update [PATCH v7 2/6] to use MSR_K7_HWCR_CPB_DIS_BIT
Changes from v3:
* rebased to linux-pm/bleeding-edge v6.8
* rename global to amd_pstate_global_params(Oleksandr Natalenko)
* remove comments for boot_supported in amd_pstate.h
* fix the compiler warning for amd-pstate-ut.ko
* use for_each_online_cpu in cpb_boost_store which fix the null pointer error during testing
* fix the max frequency value to be KHz when cpb boost disabled(Gautham R. Shenoy)
Changes from v2:
* move global struct to amd-pstate.h
* fix the amd-pstate-ut with new cpb control interface
Changes from v1:
* drop suspend/resume fix patch 6/7 because of the fix should be in another fix series instead of CPB feature
* move the set_boost remove patch to the last(Mario)
* Fix commit info with "Closes:" (Mario)
* simplified global.cpb_supported initialization(Mario)
* Add guide mode support for CPB control
* Fixed some Doc typos and add guide mode info to Doc as well.
v1: https://lore.kernel.org/all/[email protected]/
v2: https://lore.kernel.org/lkml/[email protected]/
v3: https://lore.kernel.org/lkml/[email protected]/
v4: https://lore.kernel.org/lkml/[email protected]/
v5: https://lore.kernel.org/lkml/[email protected]/
v6: https://lore.kernel.org/lkml/[email protected]/
v7: https://lore.kernel.org/lkml/[email protected]/
v8: https://lore.kernel.org/lkml/[email protected]/
v9: https://lore.kernel.org/lkml/[email protected]/
v10: https://lore.kernel.org/lkml/[email protected]/
v11: https://lore.kernel.org/lkml/[email protected]/
v12: https://lore.kernel.org/lkml/[email protected]/
Perry Yuan (9):
cpufreq: acpi: move MSR_K7_HWCR_CPB_DIS_BIT into msr-index.h
cpufreq: simplify boolean parsing with kstrtobool in store function
cpufreq: introduce init_boost callback to initialize boost state for
pstate drivers
cpufreq: amd-pstate: initialize new core precision boost state
cpufreq: amd-pstate: implement cpb_boost sysfs entry for boost control
cpufreq: amd-pstate: Add set_boost callback for active mode cpufreq:amd-pstate: Cap the CPPC.max_perf to nominal_perf if CPB is
off
Documentation: cpufreq: amd-pstate: introduce the new cpu boost
control method
Documentation: cpufreq: amd-pstate: update doc for Per CPU boost
control method
Documentation/admin-guide/pm/amd-pstate.rst | 38 ++++ arch/x86/include/asm/msr-index.h | 2 + drivers/cpufreq/acpi-cpufreq.c | 2 - drivers/cpufreq/amd-pstate-ut.c | 2 +- drivers/cpufreq/amd-pstate.c | 215 +++++++++++++++++--- drivers/cpufreq/amd-pstate.h | 14 ++ drivers/cpufreq/cpufreq.c | 25 ++- include/linux/cpufreq.h | 2 + 8 files changed, 265 insertions(+), 35 deletions(-)
Signature · 1 line
2.34.1