mbox series

[GIT,PULL] : tools/power/x86/intel-speed-select pull request for 6.7-rc1

Message ID 8680ce83af5ed5268bb05d87c579a42a1eeb97b4.camel@intel.com (mailing list archive)
State Accepted, archived
Headers show
Series [GIT,PULL] : tools/power/x86/intel-speed-select pull request for 6.7-rc1 | expand

Pull-request

https://github.com/spandruvada/linux-kernel.git intel-sst

Message

Pandruvada, Srinivas Oct. 17, 2023, 10:06 p.m. UTC
Hi Hans,

Pull request for Intel Speed Select version v1.18.

Summary of changes:
- CPU 0 hotplug support is deprecated in the upstream kernel. This
causes failures during level change for several customers. So, add a
silent workaround to use Cgroup v2, without user option.
- Increase number of CPUs in a single request
- Fix turbo mode enable/disable issue
- Error handling for invalid input options

The base branch for these changes
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
branch: review-hans


The following changes since commit
6cb9c8637c672584d4915559001020237840acba:

  platform/x86: think-lmi: Use strreplace() to replace a character by
nul (2023-09-21 18:31:19 +0200)

are available in the Git repository at:

  https://github.com/spandruvada/linux-kernel.git intel-sst

for you to fetch changes up to
a590ed62269a049a72484ce617fe2f34e2da66cf:

  tools/power/x86/intel-speed-select: v1.18 release (2023-10-17
14:54:44 -0700)

----------------------------------------------------------------
Srinivas Pandruvada (8):
      tools/power/x86/intel-speed-select: Sanitize integer arguments
      tools/power/x86/intel-speed-select: Update help for TRL
      tools/power/x86/intel-speed-select: turbo-mode enable disable
swapped
      tools/power/x86/intel-speed-select: No TRL for non compute
domains
      tools/power/x86/intel-speed-select: Display error for core-power
support
      tools/power/x86/intel-speed-select: Increase max CPUs in one
request
      tools/power/x86/intel-speed-select: Use cgroup isolate for CPU 0
      tools/power/x86/intel-speed-select: v1.18 release

 tools/power/x86/intel-speed-select/isst-config.c | 213
++++++++++++++++++++++++++++++++++----------
 tools/power/x86/intel-speed-select/isst-daemon.c |   3 +-
 tools/power/x86/intel-speed-select/isst.h        |   3 +-
 3 files changed, 168 insertions(+), 51 deletions(-)

Thanks,
Srinivas

Comments

Hans de Goede Oct. 18, 2023, 1:52 p.m. UTC | #1
Hi Srinivas, Ilpo,

On 10/18/23 00:06, Pandruvada, Srinivas wrote:
> Hi Hans,
> 
> Pull request for Intel Speed Select version v1.18.

Thanks, note I'm co-maintaining the pdx86 subsys with Ilpo
now the plan is that we alternate merging patches leading up to
the next merge-window. Ilpo is taking care of merging stuff for
the 6.7 merge-window. For future intel-speed-select pull-reqs it
is probably easiest to just send it to the both of us.

Ilpo, can you take care of merging this ?

Regards,

Hans





> 
> Summary of changes:
> - CPU 0 hotplug support is deprecated in the upstream kernel. This
> causes failures during level change for several customers. So, add a
> silent workaround to use Cgroup v2, without user option.
> - Increase number of CPUs in a single request
> - Fix turbo mode enable/disable issue
> - Error handling for invalid input options
> 
> The base branch for these changes
> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
> branch: review-hans
> 
> 
> The following changes since commit
> 6cb9c8637c672584d4915559001020237840acba:
> 
>   platform/x86: think-lmi: Use strreplace() to replace a character by
> nul (2023-09-21 18:31:19 +0200)
> 
> are available in the Git repository at:
> 
>   https://github.com/spandruvada/linux-kernel.git intel-sst
> 
> for you to fetch changes up to
> a590ed62269a049a72484ce617fe2f34e2da66cf:
> 
>   tools/power/x86/intel-speed-select: v1.18 release (2023-10-17
> 14:54:44 -0700)
> 
> ----------------------------------------------------------------
> Srinivas Pandruvada (8):
>       tools/power/x86/intel-speed-select: Sanitize integer arguments
>       tools/power/x86/intel-speed-select: Update help for TRL
>       tools/power/x86/intel-speed-select: turbo-mode enable disable
> swapped
>       tools/power/x86/intel-speed-select: No TRL for non compute
> domains
>       tools/power/x86/intel-speed-select: Display error for core-power
> support
>       tools/power/x86/intel-speed-select: Increase max CPUs in one
> request
>       tools/power/x86/intel-speed-select: Use cgroup isolate for CPU 0
>       tools/power/x86/intel-speed-select: v1.18 release
> 
>  tools/power/x86/intel-speed-select/isst-config.c | 213
> ++++++++++++++++++++++++++++++++++----------
>  tools/power/x86/intel-speed-select/isst-daemon.c |   3 +-
>  tools/power/x86/intel-speed-select/isst.h        |   3 +-
>  3 files changed, 168 insertions(+), 51 deletions(-)
> 
> Thanks,
> Srinivas
Ilpo Järvinen Oct. 18, 2023, 2:18 p.m. UTC | #2
On Wed, 18 Oct 2023, Hans de Goede wrote:
> On 10/18/23 00:06, Pandruvada, Srinivas wrote:
> > 
> > Pull request for Intel Speed Select version v1.18.
> 
> Thanks, note I'm co-maintaining the pdx86 subsys with Ilpo
> now the plan is that we alternate merging patches leading up to
> the next merge-window. Ilpo is taking care of merging stuff for
> the 6.7 merge-window. For future intel-speed-select pull-reqs it
> is probably easiest to just send it to the both of us.
> 
> Ilpo, can you take care of merging this ?

It's now merged into review-ilpo, thank you all!
Pandruvada, Srinivas Oct. 18, 2023, 2:45 p.m. UTC | #3
On Wed, 2023-10-18 at 17:18 +0300, Ilpo Järvinen wrote:
> On Wed, 18 Oct 2023, Hans de Goede wrote:
> > On 10/18/23 00:06, Pandruvada, Srinivas wrote:
> > > 
> > > Pull request for Intel Speed Select version v1.18.
> > 
> > Thanks, note I'm co-maintaining the pdx86 subsys with Ilpo
> > now the plan is that we alternate merging patches leading up to
> > the next merge-window. Ilpo is taking care of merging stuff for
> > the 6.7 merge-window. For future intel-speed-select pull-reqs it
> > is probably easiest to just send it to the both of us.
> > 
> > Ilpo, can you take care of merging this ?
> 
> It's now merged into review-ilpo, thank you all!
Thanks.

-Srinivas
>