linux-kernel - Re: [PATCH 1/2] thermal: intel: int340x: Add performance control for platform temperature control

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5a0dc3858c802a5d6247424529377c6dbfd450ca.camel@linux.intel.com>
Date: Thu, 05 Jun 2025 10:25:25 -0700
From: srinivas pandruvada <srinivas.pandruvada@...ux.intel.com>
To: "Zhang, Rui" <rui.zhang@...el.com>, "lukasz.luba@....com"
	 <lukasz.luba@....com>, "rafael@...nel.org" <rafael@...nel.org>, 
 "daniel.lezcano@...aro.org"
	 <daniel.lezcano@...aro.org>
Cc: "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>, 
 "linux-kernel@...r.kernel.org"
	 <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] thermal: intel: int340x: Add performance control
 for platform temperature control

On Thu, 2025-06-05 at 03:06 +0000, Zhang, Rui wrote:
> On Wed, 2025-06-04 at 13:35 -0700, Srinivas Pandruvada wrote:
> > Add additional attribute to control performance of platform
> > temperature
> > control feature. Two attributes are added:
> > 
> > gain: 0-7 levels, with 0 being most aggressive.
> > 	7 – graceful, favors performance at the expense of
> > temperature
> > 	overshoots
> > 	0 – aggressive, favors tight regulation over performance
> 
> By reading this, I know that setting "pts_0_control/gain" to a larger
> value means less aggressive ptc control.
> 
> But what does "gain" mean here?
This is the input to the algorithm on how much performance gain is
allowed to be in acceptable range for trip violation. May be call
performance levels instead. I am using the same term as in the spec.

Thanks,
Srinivas

> 
> May be my English problem, I'm trying hard to understand this, but it
> is
> still a bit confusing to me.
> 
> thanks,
> rui