platform-driver-x86.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
To: Hans de Goede <hdegoede@redhat.com>, Bastien Nocera <bnocera@redhat.com>
Cc: markgross@kernel.org, platform-driver-x86@vger.kernel.org,
	Patil.Reddy@amd.com
Subject: Re: [PATCH 0/4] platform/x86/amd/pmf: Introduce CnQF feature for AMD PMF
Date: Thu, 8 Sep 2022 15:44:44 +0530	[thread overview]
Message-ID: <7b5178e4-e03f-8c6f-5682-7e59d8aae415@amd.com> (raw)
In-Reply-To: <f7c02f07-f8f1-6fa8-4e88-0b4c651e4795@redhat.com>

Hi Bastien,

On 9/8/2022 2:38 PM, Hans de Goede wrote:
> Hi,
> 
> On 9/7/22 17:35, Bastien Nocera wrote:
>> On Wed, 7 Sept 2022 at 16:35, Hans de Goede <hdegoede@redhat.com> wrote:
>>>
>>> Hi Bastien,
>>>
>>> On 9/7/22 16:24, Bastien Nocera wrote:
>>>> Hey Shyam,
>>>>
>>>> I misunderstood that CnQF was a single setting, but it looks like it
>>>> has 4 different levels, right?
>>>> Unless there's a major malfunction, I don't think that offering to
>>>> switch between 2 different policies where the difference is how
>>>> "static" the performance boosts are is very useful, or comprehensible,
>>>> to end-users.
>>>>
>>>> If CnQF only has a single "on" setting, then this could replace the
>>>> balanced mode for what you call "static slider", so the end-user can
>>>> still make a choice and have agency on whether the system tries to
>>>> save power, or increase performance.
>>>>
>>>> If CnQF has multiple levels (Turbo, Performance, Balanced and Quiet,
>>>> right?), then I don't think it's useful to have a sysfs setting to
>>>> switch it at runtime, which only confuses user-space and the users.
>>>> BIOS setting and/or kernel command-line option are the way to go.
>>>>
>>>> Did I understand this correctly?
>>>
>>> Let me try clarify things:
>>>
>>> CnQF has 4 levels internally, between which it switches automatically
>>> based on the workload of the last 5 minutes.
>>
>> Oh, those profiles are internal only, OK. Do those automated levels
>> behave like the "static slider" ones, to the point of being
>> indistinguishable? So for example, does the static slider
>> "performance" behave like "CnQF" if the machine was heavily loaded
>> machine for 5 minutes?
> 
> This is more of a question for AMD to answer. But yes I believe that
> the CnQF internal performance mode which it boosts to if the machine
> is heavily loaded for 5 minutes is similar to the static slider
> performance setting.

Its a kind of "yes". But its still dependent on how the OEMs have tuned
the power profiling values and mapped it to the relavant CnQF modes.

Thanks,
Shyam

> 
> Regards,
> 
> Hans
> 

  reply	other threads:[~2022-09-08 10:14 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-23 10:29 [PATCH 0/4] platform/x86/amd/pmf: Introduce CnQF feature for AMD PMF Shyam Sundar S K
2022-08-23 10:29 ` [PATCH 1/4] platform/x86/amd/pmf: Add support for CnQF Shyam Sundar S K
2022-08-23 14:56   ` Limonciello, Mario
2022-08-23 10:29 ` [PATCH 2/4] platform/x86/amd/pmf: Add sysfs to toggle CnQF Shyam Sundar S K
2022-08-23 10:29 ` [PATCH 3/4] Documentation/ABI/testing/sysfs-amd-pmf: Add ABI doc for AMD PMF Shyam Sundar S K
2022-08-23 10:29 ` [PATCH 4/4] MAINTAINERS: Update ABI doc path " Shyam Sundar S K
2022-09-01 11:16 ` [PATCH 0/4] platform/x86/amd/pmf: Introduce CnQF feature " Hans de Goede
2022-09-01 12:24   ` Bastien Nocera
2022-09-01 12:44     ` Hans de Goede
2022-09-01 13:34       ` Bastien Nocera
2022-09-06  9:59         ` Shyam Sundar S K
2022-09-07 14:24           ` Bastien Nocera
2022-09-07 14:35             ` Hans de Goede
2022-09-07 15:35               ` Bastien Nocera
2022-09-08  9:08                 ` Hans de Goede
2022-09-08 10:14                   ` Shyam Sundar S K [this message]
2022-09-07 14:52           ` Hans de Goede
2022-09-06  9:53       ` Shyam Sundar S K
2022-09-07 14:48         ` Hans de Goede
2022-09-08 10:08           ` Shyam Sundar S K
2022-09-08 10:09             ` Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7b5178e4-e03f-8c6f-5682-7e59d8aae415@amd.com \
    --to=shyam-sundar.s-k@amd.com \
    --cc=Patil.Reddy@amd.com \
    --cc=bnocera@redhat.com \
    --cc=hdegoede@redhat.com \
    --cc=markgross@kernel.org \
    --cc=platform-driver-x86@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).