linux-kernel - Re: [PATCH v1 0/9] Move RAPL defaults to interface drivers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ed7cd1ea051f07bb59fea83f359aedaaab4cf423.camel@linux.intel.com>
Date: Thu, 29 Jan 2026 11:20:28 -0800
From: srinivas pandruvada <srinivas.pandruvada@...ux.intel.com>
To: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@...ux.intel.com>,
  "Rafael J . Wysocki"	 <rafael@...nel.org>, Daniel Lezcano
 <daniel.lezcano@...aro.org>
Cc: Zhang Rui <rui.zhang@...el.com>, Lukasz Luba <lukasz.luba@....com>, 
	linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 0/9] Move RAPL defaults to interface drivers

On Thu, 2026-01-29 at 10:36 -0800, Kuppuswamy Sathyanarayanan wrote:
> Hi All,
> 
> The Intel RAPL common driver has accumulated a significant amount of
> interface-specific logic over time. There is very little common code
> shared across the MSR, TPMI, and MMIO interface-specific defaults.
> Keeping these interface-specific defaults in the common layer
> therefore
> provides no real benefit and instead increases complexity and
> maintenance burden.
> 
> As a first step toward cleaning this up, this series moves
> rapl_defaults ownership from the common driver into the individual
> interface drivers and allows each interface to provide its own
> defaults
> directly. Additional interface-specific cleanups in the RAPL common
> driver will be addressed in follow-up work. This series is a
> continuation of the earlier cleanup and refactoring effort initiated
> by Zhang Rui.
> 

For the series:

Acked-by: Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>


> Patch Summary:
> 
> Patch 1-4/9: Preparatory patches that do code cleanups in
> intel_rapl_common
>              driver.
> Patch 5/9: Preparatory patch that renames/declares the common
> functions.
> Patch 6/9: Move TPMI default settings from the common driver into the
>            TPMI interface driver.
> 
> Patch 7/9: Move MMIO default settings from the common driver into the
>            MMIO interface driver.
> 
> Patch 8/9: Register the PM notifier only when a RAPL package exists.
> 
> Patch 9/9: Move MSR default settings from the common driver into the
> MSR
>            interface driver.
> 
> No functional changes are intended across the series. This is
> strictly a
> structural refactoring to simplify the common RAPL code and prepare
> it
> for further cleanups.
> 
> Kuppuswamy Sathyanarayanan (9):
>   powercap: intel_rapl: Add a symbol namespace for intel_rapl exports
>   powercap: intel_rapl: Cleanup coding style
>   powercap: intel_rapl: Use GENMASK() and BIT() macros
>   powercap: intel_rapl: Use unit conversion macros from units.h
>   powercap: intel_rapl: Allow interface drivers to configure
>     rapl_defaults
>   powercap: intel_rapl: Move TPMI default settings into TPMI
> interface
>     driver
>   thermal: intel: int340x: processor: Move RAPL defaults to MMIO
> driver
>   powercap: intel_rapl: Register PM notifier only when RAPL package
>     exists
>   powercap: intel_rapl: Move MSR default settings into MSR interface
>     driver
> 
>  drivers/powercap/intel_rapl_common.c          | 755 ++++++----------
> --
>  drivers/powercap/intel_rapl_msr.c             | 251 +++++-
>  drivers/powercap/intel_rapl_tpmi.c            |  48 ++
>  .../int340x_thermal/processor_thermal_rapl.c  |   9 +
>  include/linux/intel_rapl.h                    |  17 +-
>  include/linux/units.h                         |   3 +
>  6 files changed, 561 insertions(+), 522 deletions(-)