mbox series

[0/6,V2,SRU,J] Fix RPL-S support on powercap/intel_rapl Edit

Message ID 20221007013334.326785-1-koba.ko@canonical.com
Headers show
Series Fix RPL-S support on powercap/intel_rapl Edit | expand

Message

Koba Ko Oct. 7, 2022, 1:33 a.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/1990161

[Impact]
proc_thermal driver is not loaded

[Fix]
Add RPL-S id on device id table of powercap/intel_rapl.

[Test Case]
1. sudo checkbox-cli run com.canonical.certification::miscellanea/proc_thermal

[Where problems could occur]
Low, only add RPL-S id on powercap/intel_rap but may lack of RPL-S features.

~~~
V2: add the missing pre-requested patches

George D Sworo (1):
  powercap: intel_rapl: Add support for RAPTORLAKE_P

Tony Luck (3):
  x86/cpu: Drop spurious underscore from RAPTOR_LAKE #define
  x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
  x86/cpu: Add new Raptor Lake CPU model number

Zhang Rui (2):
  powercap: intel_rapl: add support for RaptorLake
  powercap: intel_rapl: Add support for RAPTORLAKE_S

 arch/x86/include/asm/intel-family.h  | 7 +++++++
 drivers/powercap/intel_rapl_common.c | 3 +++
 2 files changed, 10 insertions(+)

Comments

Tim Gardner Oct. 7, 2022, 1:20 p.m. UTC | #1 
On 10/6/22 19:33, Koba Ko wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990161
> 
> [Impact]
> proc_thermal driver is not loaded
> 
> [Fix]
> Add RPL-S id on device id table of powercap/intel_rapl.
> 
> [Test Case]
> 1. sudo checkbox-cli run com.canonical.certification::miscellanea/proc_thermal
> 
> [Where problems could occur]
> Low, only add RPL-S id on powercap/intel_rap but may lack of RPL-S features.
> 
> ~~~
> V2: add the missing pre-requested patches
> 
> George D Sworo (1):
>    powercap: intel_rapl: Add support for RAPTORLAKE_P
> 
> Tony Luck (3):
>    x86/cpu: Drop spurious underscore from RAPTOR_LAKE #define
>    x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
>    x86/cpu: Add new Raptor Lake CPU model number
> 
> Zhang Rui (2):
>    powercap: intel_rapl: add support for RaptorLake
>    powercap: intel_rapl: Add support for RAPTORLAKE_S
> 
>   arch/x86/include/asm/intel-family.h  | 7 +++++++
>   drivers/powercap/intel_rapl_common.c | 3 +++
>   2 files changed, 10 insertions(+)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Cory Todd Oct. 13, 2022, 8:59 p.m. UTC | #2 
On Fri, Oct 07, 2022 at 09:33:28AM +0800, Koba Ko wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990161
> 
> [Impact]
> proc_thermal driver is not loaded
> 
> [Fix]
> Add RPL-S id on device id table of powercap/intel_rapl.
> 
> [Test Case]
> 1. sudo checkbox-cli run com.canonical.certification::miscellanea/proc_thermal
> 
> [Where problems could occur]
> Low, only add RPL-S id on powercap/intel_rap but may lack of RPL-S features.
> 
> ~~~
> V2: add the missing pre-requested patches
> 
> George D Sworo (1):
>   powercap: intel_rapl: Add support for RAPTORLAKE_P
> 
> Tony Luck (3):
>   x86/cpu: Drop spurious underscore from RAPTOR_LAKE #define
>   x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
>   x86/cpu: Add new Raptor Lake CPU model number
> 
> Zhang Rui (2):
>   powercap: intel_rapl: add support for RaptorLake
>   powercap: intel_rapl: Add support for RAPTORLAKE_S
> 
>  arch/x86/include/asm/intel-family.h  | 7 +++++++
>  drivers/powercap/intel_rapl_common.c | 3 +++
>  2 files changed, 10 insertions(+)
> 

Acked-by: Cory Todd <cory.todd@canonical.com>
Stefan Bader Nov. 10, 2022, 1:48 p.m. UTC | #3 
On 07.10.22 03:33, Koba Ko wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990161
> 
> [Impact]
> proc_thermal driver is not loaded
> 
> [Fix]
> Add RPL-S id on device id table of powercap/intel_rapl.
> 
> [Test Case]
> 1. sudo checkbox-cli run com.canonical.certification::miscellanea/proc_thermal
> 
> [Where problems could occur]
> Low, only add RPL-S id on powercap/intel_rap but may lack of RPL-S features.
> 
> ~~~
> V2: add the missing pre-requested patches
> 
> George D Sworo (1):
>    powercap: intel_rapl: Add support for RAPTORLAKE_P
> 
> Tony Luck (3):
>    x86/cpu: Drop spurious underscore from RAPTOR_LAKE #define
>    x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
>    x86/cpu: Add new Raptor Lake CPU model number
> 
> Zhang Rui (2):
>    powercap: intel_rapl: add support for RaptorLake
>    powercap: intel_rapl: Add support for RAPTORLAKE_S
> 
>   arch/x86/include/asm/intel-family.h  | 7 +++++++
>   drivers/powercap/intel_rapl_common.c | 3 +++
>   2 files changed, 10 insertions(+)
> 

Applied to jammy:linux/master-next. Thanks.

-Stefan