mbox series

[RFC,0/1] x86: msr: add new 'msr_pkg_cst_config_control.h' header

Message ID 20250220154306.2166129-1-dedekind1@gmail.com (mailing list archive)
Headers show
Series x86: msr: add new 'msr_pkg_cst_config_control.h' header | expand

Message

Artem Bityutskiy Feb. 20, 2025, 3:43 p.m. UTC 
From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

In this patch I am trying to implement the suggestion from Rafael Wysocki
that can be found here:
https://patches.linaro.org/project/linux-pm/patch/20250212084232.2349984-1-dedekind1@gmail.com/

The suggestion is basically to have common wrappers for accessing
MSR_PKG_CST_CONFIG_CONTROL in order to make it clear there are multiple
users of the MSR.

I was not sure how to implement this, and here is what came to mind. Just
an RFC to demonstate the approach, please comment/criticize.

Just to linux-pm at this point, also avoid splitting on multiple patches for now.

This patch applies on top of the following patch-set that I sent to linux-pm
a bit earlier today:
"[PATCH v2 0/3] intel_idle: Add C1 demotion on/off sysfs knob"
https://lore.kernel.org/linux-pm/20250220151702.2153579-1-dedekind1@gmail.com/T/#t


Artem Bityutskiy (1):
  x86: msr: add new 'msr_pkg_cst_config_control.h' header

 .../include/asm/msr_pkg_cst_config_control.h  | 28 +++++++++++++++++++
 drivers/idle/intel_idle.c                     | 17 +++++------
 drivers/platform/x86/intel/pmc/cnp.c          | 13 +++++----
 3 files changed, 44 insertions(+), 14 deletions(-)
 create mode 100644 arch/x86/include/asm/msr_pkg_cst_config_control.h