Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230620172433.21325-1-mario.limonciello@amd.com>
In-Reply-To: <20230620172433.21325-1-mario.limonciello@amd.com>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Wed, 21 Jun 2023 19:38:02 +0200
Message-ID: <CAJZ5v0hjrKS7Cb+wJu9cP8VV9DggUR+2iZ4DheFOSd0bRx-=jw@mail.gmail.com>
Subject: Re: [PATCH v3 0/3] Enable amd-pstate active mode by default
To:     Mario Limonciello <mario.limonciello@amd.com>
Cc:     "Rafael J . Wysocki" <rafael@kernel.org>,
        Len Brown <lenb@kernel.org>, Huang Rui <ray.huang@amd.com>,
        linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-pm@vger.kernel.org,
        Gautham Ranjal Shenoy <gautham.shenoy@amd.com>,
        Wyes Karny <Wyes.Karny@amd.com>,
        Perry Yuan <perry.yuan@amd.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Precedence: bulk

On Tue, Jun 20, 2023 at 7:27 PM Mario Limonciello
<mario.limonciello@amd.com> wrote:
>
> Active mode for amd-pstate has shown enough success now that it makes sense
> to enable it by default on client systems.
>
> This series introduces a new kernel configuration option to set the default
> policy for amd-pstate modes for a kernel.
>
> Server systems will by identified by the PM preferred profile and still be
> set as disabled by default for now.
>
> v2->v3:
>  * Drop patch 4; Intel intentionally doesn't want intel-pstate on SOHO
>    server
>  * Move symbols from patch 1 into patch 2
>  * Add tags
> Mario Limonciello (3):
>   ACPI: CPPC: Add definition for undefined FADT preferred PM profile
>     value
>   cpufreq: amd-pstate: Set a fallback policy based on preferred_profile
>   cpufreq: amd-pstate: Add a kernel config option to set default mode
>
>  drivers/cpufreq/Kconfig.x86  |  17 ++++++
>  drivers/cpufreq/amd-pstate.c | 101 +++++++++++++++++++++++++----------
>  include/acpi/actbl.h         |   3 +-
>  include/linux/amd-pstate.h   |   4 +-
>  4 files changed, 96 insertions(+), 29 deletions(-)
>
> --

All applied as 6.5 material, thanks!