ACK/Cmnt: [SRU][Jammy][PATCH 0/7] Patches for LP:#2003267 - intel_idle support for Eagle Stream

[Tim Gardner]

On 2/7/23 10:58 AM, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/2003267
> 
> == SRU Justification ==
> Intel has requested the following patches which enable enable Intel_idle for eagle stream.
> 
> Intel has customers that would like to use the intel_idle driver on latest SPR (Sapphire Rapids).
> These patches have all landed upstream, but are not in Jammy.
> 
> These patches will enable customers to use Ubuntu 22.04 on new hardware
> from Intel.
> 
> 
> == Fixes ==
> 9edf3c0ffef0 ("intel_idle: add SPR support")
> da0e58c038e6 ("intel_idle: add 'preferred_cstates' module argument")
> 3a9cf77b60dc ("intel_idle: add core C6 optimization for SPR")
> 03eb65224e57 ("cpuidle: intel_idle: Drop redundant backslash at line end")
> 39c184a6a9a7 ("intel_idle: Fix the 'preferred_cstates' module parameter")
> 7eac3bd38d18 ("intel_idle: Fix SPR C6 optimization")
> 1548fac47a11 ("intel_idle: make SPR C1 and C1E be independent")
> 
> 
> == Regression Potential ==
> Medium. These patches are specific to enable intel_idle support for Sapphire Rapids.  Changes
> are specific to Intel and tested by Intel.
> 
> 
> == Test Case ==
> A test kernel was built with these patches and tested by Intel.
> Intel tested on SPR-SP machine and see the correct 190us and 600us residency for the C6 state.
> 
> 
> Artem Bityutskiy (6):
>    intel_idle: add SPR support
>    intel_idle: add 'preferred_cstates' module argument
>    intel_idle: add core C6 optimization for SPR
>    intel_idle: Fix the 'preferred_cstates' module parameter
>    intel_idle: Fix SPR C6 optimization
>    intel_idle: make SPR C1 and C1E be independent
> 
> Rafael J. Wysocki (1):
>    cpuidle: intel_idle: Drop redundant backslash at line end
> 
>   drivers/idle/intel_idle.c | 95 +++++++++++++++++++++++++++++++++++++--
>   1 file changed, 92 insertions(+), 3 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>

Seems reasonably well isolated to Sapphire Rapids.
-- 
-----------
Tim Gardner
Canonical, Inc