linux-kernel - RE: [PATCH 1/2] cpufreq/amd-pstate-ut: Handle the inconsistency

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]

Date: Wed, 26 Jun 2024 10:52:23 +0530
From: Gautham R.Shenoy <gautham.shenoy@....com>
To: "Meng, Li (Jassmine)" <Li.Meng@....com>, "Ugwekar, Dhananjay"
	<Dhananjay.Ugwekar@....com>, "rafael@...nel.org" <rafael@...nel.org>,
	"viresh.kumar@...aro.org" <viresh.kumar@...aro.org>, "Limonciello, Mario"
	<Mario.Limonciello@....com>, "Yuan, Perry" <Perry.Yuan@....com>,
	"skhan@...uxfoundation.org" <skhan@...uxfoundation.org>, "Huang, Ray"
	<Ray.Huang@....com>
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] cpufreq/amd-pstate-ut: Handle the inconsistency

"Meng, Li (Jassmine)" <Li.Meng@....com> writes:

Hello Jassmine,

>
> Hi Dhananjay:
> Sorry for the delay.
>
> I think the correction to this issue should be in function amd_pstae_init_freq() of file amd-pstate.c.
> The value of norminal_freq should be consistent with other
> values(max_freq,min_freq etc.).

Perry wanted nominal_freq to be in MHz since it is not exposed to the
user via any of the cpufreq sysfs interfaces.

IMO, it woyuld be good to to rename the variables to have their units
for better readability along with a bunch of other
cleanups/code deduplication. But can it be done separately ?


>
>> -----Original Message-----
>> From: Ugwekar, Dhananjay <Dhananjay.Ugwekar@....com>
>> Sent: Tuesday, June 25, 2024 9:52 PM
>> To: rafael@...nel.org; viresh.kumar@...aro.org; Shenoy, Gautham Ranjal
>> <gautham.shenoy@....com>; Limonciello, Mario
>> <Mario.Limonciello@....com>; Yuan, Perry <Perry.Yuan@....com>;
>> skhan@...uxfoundation.org; Meng, Li (Jassmine) <Li.Meng@....com>;
>> Huang, Ray <Ray.Huang@....com>
>> Cc: linux-pm@...r.kernel.org; linux-kernel@...r.kernel.org
>> Subject: Re: [PATCH 1/2] cpufreq/amd-pstate-ut: Handle the inconsistency
>>
>> Minor modification, the commit subject is supposed to be
>> "cpufreq/amd-pstate-ut: Handle the inconsistency between nominal_freq
>> and other *_freq units"

How about:

"cpufreq/amd-pstate-ut: Convert nominal_freq to khz during comparisons"

Otherwise the patch looks good to me.

Reviewed-by: Gautham R. Shenoy <gautham.shenoy@....com>

--
Thanks and Regards
gautham.