mbox series

[v3,0/2] intel_powerclamp: New module parameter

Message ID 20230207173219.4190013-1-srinivas.pandruvada@linux.intel.com (mailing list archive)
Headers show
Series intel_powerclamp: New module parameter | expand

Message

srinivas pandruvada Feb. 7, 2023, 5:32 p.m. UTC
Split from the series for powerclamp user of powercap idle-inject
series:
Use idle_inject framework for intel_powerclamp version v5.


v3:
minor document change

v2
- Build warnings reported by Rui
- Moved the powerclamp documentation to admin guide folder
- Commit log updated as suggested by Rafael and other code suggestion

Srinivas Pandruvada (2):
  Documentation:admin-guide: Move intel_powerclamp documentation
  thermal/drivers/intel_powerclamp: Add two module parameters

 Documentation/admin-guide/index.rst           |   1 +
 .../thermal/intel_powerclamp.rst              |  22 +++
 Documentation/driver-api/thermal/index.rst    |   1 -
 MAINTAINERS                                   |   1 +
 drivers/thermal/intel/intel_powerclamp.c      | 177 +++++++++++++++---
 5 files changed, 180 insertions(+), 22 deletions(-)
 rename Documentation/{driver-api => admin-guide}/thermal/intel_powerclamp.rst (93%)

Comments

Zhang, Rui Feb. 8, 2023, 7:37 a.m. UTC | #1
On Tue, 2023-02-07 at 09:32 -0800, Srinivas Pandruvada wrote:
> Split from the series for powerclamp user of powercap idle-inject
> series:
> Use idle_inject framework for intel_powerclamp version v5.
> 
> 
> v3:
> minor document change

I have tested the V2 version of this patch series + the whole series of
Richardo's scheduler fixes from
https://lore.kernel.org/lkml/20230207045838.11243-1-ricardo.neri-calderon@linux.intel.com/

The previous scheduler problem is gone and idle injection with a
cpumask works well.

Give that we only have document change in V3,

Tested-by: Zhang Rui <rui.zhang@intel.com>

thanks,
rui

> 
> v2
> - Build warnings reported by Rui
> - Moved the powerclamp documentation to admin guide folder
> - Commit log updated as suggested by Rafael and other code suggestion
> 
> Srinivas Pandruvada (2):
>   Documentation:admin-guide: Move intel_powerclamp documentation
>   thermal/drivers/intel_powerclamp: Add two module parameters
> 
>  Documentation/admin-guide/index.rst           |   1 +
>  .../thermal/intel_powerclamp.rst              |  22 +++
>  Documentation/driver-api/thermal/index.rst    |   1 -
>  MAINTAINERS                                   |   1 +
>  drivers/thermal/intel/intel_powerclamp.c      | 177 +++++++++++++++-
> --
>  5 files changed, 180 insertions(+), 22 deletions(-)
>  rename Documentation/{driver-api => admin-
> guide}/thermal/intel_powerclamp.rst (93%)
>