[PATCH v3 00/10] hwmon: (fam15h_power) Introduce an accumulated power reporting algorithm

From: Huang Rui
Date: Fri Oct 30 2015 - 05:59:42 EST

Hi all,

This serial of patches introduces an accumulated power reporting
algorithm. It will calculate the average power consumption for the
processor. The cpu feature flag is CPUID.8000_0007H:EDX[12].

This algorithm is used to test the comparison of processor power
consumption with between MWAITX delay and TSC delay on AMD Carrizo


Commit f96756 at tip ("x86/asm: Add MONITORX/MWAITX instruction support")
Commit b466bd at tip ("x86/asm/delay: Introduce an MWAITX-based delay with a configurable timer")

V1: http://marc.info/?l=linux-kernel&m=144066380613299&w=2
V2: http://marc.info/?l=linux-kernel&m=144531018518473&w=2

Changes from v1 -> v2:
- Move fam15h_power_groups and fam15h_power_group into fam15h_power_data to
avoid overwrite on multi-CPU system.
- Rename FAM15H_MIN_NUM_ATTRS macro and fix return error code.
- Remove unnecessary warning print.
- Adds do_read_registers_on_cu to do all the read to all MSRs and run it on one
of the online cores on each compute unit with smp_call_function_many().
- Use power1_average and power1_average_interval standard entry
instread of power1_acc
- Fix the CPU-hotplug case.

Changes from v2 -> v3:
- As Guenter's suggestion, remove typecast, use &data->groups[0].
- Remove all "fam15_power_*" prefix at data.
- Remove unnecessary ( ).
- Fix the issue that is reported by build test robot, and add
CPU_SUP_AMD as the dependence of fam15h_power
- Remove the WARN_ON at do_read_registers_on_cu, because it must be
behind CPUID check. The MSR must be available since
CPUID.8000_0007H:EDX[12] is set
- Add get_online_cpus()/put_online_cpus() functions.
- Refine commments and the method which generate cpumask for cu.
- Add the interval scope to make the value suitable for user
- Remove the useless mutex.

A simple example:

ray@hr-ub:~/tip$ sensors
Adapter: PCI adapter
power1: 23.73 mW (avg = 634.63 mW, interval = 0.01 s)
(crit = 15.00 W)


These patches are rebased on groeck/hwmon-next.


Huang Rui (10):
hwmon: (fam15h_power) Refactor attributes for dynamically added
hwmon: (fam15h_power) Enable power1_input on AMD Carrizo
hwmon: (fam15h_power) Add max compute unit accumulated power
x86, amd: add accessor for number of cores per compute unit
hwmon: (fam15h_power) Add CPU_SUP_AMD as the dependence
hwmon: (fam15h_power) Add compute unit accumulated power
hwmon: (fam15h_power) Add ptsc counter value for accumulated power
hwmon: (fam15h_power) Introduce a cpu accumulated power reporting
hwmon: (fam15h_power) Add documentation for TDP and accumulated power
MAINTAINERS: change the maintainer of fam15h_power driver

CREDITS | 8 ++
Documentation/hwmon/fam15h_power | 57 +++++++-
arch/x86/include/asm/msr-index.h | 1 +
arch/x86/include/asm/processor.h | 1 +
arch/x86/kernel/cpu/amd.c | 19 ++-
drivers/hwmon/Kconfig | 2 +-
drivers/hwmon/fam15h_power.c | 278 +++++++++++++++++++++++++++++++++++----
8 files changed, 341 insertions(+), 29 deletions(-)


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/