Linux Power Management development
 help / color / mirror / Atom feed
From: K Prateek Nayak <kprateek.nayak@amd.com>
To: Mario Limonciello <superm1@kernel.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Viresh Kumar <viresh.kumar@linaro.org>,
	Huang Rui <ray.huang@amd.com>
Cc: Perry Yuan <perry.yuan@amd.com>, <linux-pm@vger.kernel.org>,
	<linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 3/6] cpufreq/amd-pstate: Repurpose "amd_dynamic_epp" cmdline and corresponding sysfs
Date: Fri, 3 Jul 2026 02:06:30 +0530	[thread overview]
Message-ID: <5f271215-6e83-4d75-a562-16d8e70ce672@amd.com> (raw)
In-Reply-To: <8155d7c7-2a10-4fec-bfde-0aeb017724ac@kernel.org>

Hello Mario,

On 7/2/2026 3:10 AM, Mario Limonciello wrote:
> On 6/30/26 13:59, K Prateek Nayak wrote:
>> Since dynamic_epp has been converted to an
>> "energy_performance_preference", toggling the feature via the sysfs file
>> is now redundant.
>>
>> Repurpose "amd_dynamic_epp=enable" command line to opt into dynamic EPP
>> by default when the active driver is loaded, or switched into. The
>> "disable" counterpart will "dynamic" option out of
>> "energyy_performance_preference" to ensure dynamic EPP feature can never
>> be toggled.
>>
>> Use a tri-state enum to differentiate if the user has supplied a command
>> line parameter or not to enable the feature by default vs keeping the
>> current defaults as is and allowing users to toggle it later via the
>> "dynamic" option in "energy_performance_preference" selection.
>>
>> Signed-off-by: K Prateek Nayak <kprateek.nayak@amd.com>
> 
> Honestly - I have to question if we even want to have the kernel command line option anymore.  If userspace can easily opt in and out as in this series, do we still need a kernel command line for default policy?
> 
> I'm of the opinion rip it out, less to maintain.

Ack! I'll rip out the cmdline and the "dynamic_epp" sysfs file then in
the next version?

-- 
Thanks and Regards,
Prateek


  reply	other threads:[~2026-07-02 20:36 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-30 18:58 [RFC PATCH 0/6] cpufreq/amd-pstate: Rework dynamic_epp as an energy_performance_preference mode K Prateek Nayak
2026-06-30 18:58 ` [RFC PATCH 1/6] cpufreq/amd-pstate: Extract platform profile to EPP conversion into a helper K Prateek Nayak
2026-07-01 21:31   ` Mario Limonciello
2026-06-30 18:59 ` [RFC PATCH 2/6] cpufreq/amd-pstate: Add dynamic EPP as an "energy_performance_preference" mode K Prateek Nayak
2026-07-01 21:33   ` Mario Limonciello
2026-07-02 20:30     ` K Prateek Nayak
2026-06-30 18:59 ` [RFC PATCH 3/6] cpufreq/amd-pstate: Repurpose "amd_dynamic_epp" cmdline and corresponding sysfs K Prateek Nayak
2026-07-01 21:40   ` Mario Limonciello
2026-07-02 20:36     ` K Prateek Nayak [this message]
2026-06-30 18:59 ` [RFC PATCH 4/6] Documentation/amd-pstate: Update dynamic_epp documentation with new behavior K Prateek Nayak
2026-06-30 19:03 ` K Prateek Nayak
2026-06-30 19:03 ` [RFC PATCH 5/6] cpufreq/amd-pstate: Reduce the scope of exported symbols K Prateek Nayak
2026-07-01 21:38   ` Mario Limonciello
2026-06-30 19:03 ` [RFC PATCH 6/6] cpufreq/amd-pstate-ut: Add unit test for "dynamic" EPP mode K Prateek Nayak

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=5f271215-6e83-4d75-a562-16d8e70ce672@amd.com \
    --to=kprateek.nayak@amd.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=perry.yuan@amd.com \
    --cc=rafael@kernel.org \
    --cc=ray.huang@amd.com \
    --cc=superm1@kernel.org \
    --cc=viresh.kumar@linaro.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