Re: [PATCH v5 0/4] Use idle_inject framework for intel_powerclamp

[Rafael J. Wysocki]

On Wed, Feb 1, 2023 at 7:35 PM Srinivas Pandruvada
<srinivas.pandruvada@xxxxxxxxxxxxxxx> wrote:
>
> Dropped the per core idle injection patch from this series. Hence the
> subject of this series is changed. Now it is just for using
> idle inject framework for intel_powerclamp.
> Add additionl patch for module paramters to set cpumask and max idle percent.
>
> The old cover letter:
> This series introduces per CPU idle injection. In preparation for this
> enhance the existing powercap/idle_inject and modify intel_powerclamp
> to use this. Then add per core idle injection driver.
>
> v5:
> - additionl patch for module paramters to set cpumask and max idle percent
> - One fix for update callback
> - Rebased on top of patch
>  "thermal: intel_powerclamp: Fix cur_state for multi package system"

Patches [1-3/4] have been applied as 6.3 material and I'll comment
patch [4/4] separately.

>
> v4:
> - Dropped the per core idle inject patch
>
> v3
> - Change callback from per CPU to per device and use in intel_powerclamp
> - Remove unused var in per cpu idle injection module
>
> v2
> - Update based on feedback from Rafael on patch 2/4
> - Kconfig dependency issue
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
>
> Srinivas Pandruvada (4):
>   powercap: idle_inject: Export symbols
>   powercap: idle_inject: Add update callback
>   thermal/drivers/intel_powerclamp: Use powercap idle-inject framework
>   thermal/drivers/intel_powerclamp: Add additional module params
>
>  .../driver-api/thermal/intel_powerclamp.rst   |  22 +
>  drivers/powercap/idle_inject.c                |  57 +-
>  drivers/thermal/intel/Kconfig                 |   2 +
>  drivers/thermal/intel/intel_powerclamp.c      | 512 ++++++++++--------
>  include/linux/idle_inject.h                   |   3 +
>  5 files changed, 367 insertions(+), 229 deletions(-)
>
> --
> 2.39.1
>