linux-kernel - Re: [PATCH] Documentation: amd-pstate:fix minimum performance state label error

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] [day] [month] [year] [list]

Message-ID: <d1281f23-69c6-4386-9d26-6f2cd95343cc@amd.com>
Date: Thu, 22 May 2025 17:35:20 +0000
From: "Limonciello, Mario" <Mario.Limonciello@....com>
To: shouyeliu <shouyeliu@...il.com>, "Huang, Ray" <Ray.Huang@....com>,
	"Shenoy, Gautham Ranjal" <gautham.shenoy@....com>, "Yuan, Perry"
	<Perry.Yuan@....com>, "corbet@....net" <corbet@....net>
CC: "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Shouye Liu
	<shouyeliu@...cent.com>
Subject: Re: [PATCH] Documentation: amd-pstate:fix minimum performance state
 label error

On 5/22/25 2:01 AM, shouyeliu wrote:
> In the AMD P-States Performance Scale diagram, the labels for "Max Perf"
> and "Lowest Perf" were incorrectly used to define the range for
> "Desired Perf".The "Desired performance target" should be bounded by the
> "Maximum requested performance" and the "Minimum requested performance",
> which corresponds to "Max Perf" and "Min Perf", respectively.
> 
> Signed-off-by: Shouye Liu <shouyeliu@...cent.com>
> ---
>   Documentation/admin-guide/pm/amd-pstate.rst | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/admin-guide/pm/amd-pstate.rst b/Documentation/admin-guide/pm/amd-pstate.rst
> index 412423c54f25..e1771f2225d5 100644
> --- a/Documentation/admin-guide/pm/amd-pstate.rst
> +++ b/Documentation/admin-guide/pm/amd-pstate.rst
> @@ -72,7 +72,7 @@ to manage each performance update behavior. ::
>     Lowest non-        |                       |                         |                       |
>     linear perf ------>+-----------------------+                         +-----------------------+
>                        |                       |                         |                       |
> -                     |                       |       Lowest perf  ---->|                       |
> +                     |                       |          Min perf  ---->|                       |
>                        |                       |                         |                       |
>     Lowest perf ------>+-----------------------+                         +-----------------------+
>                        |                       |                         |                       |

Makes sense to me.  Thanks.

Reviewed-by: Mario Limonciello <mario.limonciello@....com>