[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <62da8b7c-3a90-cffc-4750-532c2b440764@linuxfoundation.org>
Date: Mon, 14 Aug 2023 17:22:48 -0600
From: Shuah Khan <skhan@...uxfoundation.org>
To: "Limonciello, Mario" <mario.limonciello@....com>,
Huang Rui <ray.huang@....com>
Cc: "Yuan, Perry" <Perry.Yuan@....com>,
"Karny, Wyes" <Wyes.Karny@....com>,
"trenn@...e.com" <trenn@...e.com>,
"shuah@...nel.org" <shuah@...nel.org>,
"rafael.j.wysocki@...el.com" <rafael.j.wysocki@...el.com>,
"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"chenhq2005@...il.com" <chenhq2005@...il.com>,
Shuah Khan <skhan@...uxfoundation.org>
Subject: Re: [PATCH] cpupower: Fix querying boost support for amd-pstate-epp
mode
On 8/14/23 15:50, Limonciello, Mario wrote:
>
>
> On 8/14/2023 4:28 PM, Shuah Khan wrote:
>> On 8/13/23 22:14, Huang Rui wrote:
>>> On Sat, Aug 12, 2023 at 05:35:01AM +0800, Limonciello, Mario wrote:
>>>> When running in active mode the driver for amd-pstate has -epp appended.
>>>> This throws off amd-pstate detection.
>>>>
>>>> To detect amd-pstate look for the prefix instead.
>>>>
>>>> Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors")
>>>> Reported-by: chenhq2005@...il.com
>>>> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217755
>>>> Signed-off-by: Mario Limonciello <mario.limonciello@....com>
>>>
>>> Acked-by: Huang Rui <ray.huang@....com>
>>>
>>
>> This patch doesn't apply on top of cpupower latest:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux.git coupower
>>
>> Please rebase and send it.
>>
>> thanks,
>> -- Shuah
>>
>
> It looks like a variation of it is already in your tree as:
>
> a1cf97c2d43ae ("cpupower: Recognise amd-pstate active mode driver")
>
> Considering it was also reported to a kernel bugzilla, perhaps this should be considered for fixes?
If it is already fixed, there is no need to revert and apply your fix.
Please close the bugzilla bug as fixed with the above commit.
thanks,
-- Shuah
Powered by blists - more mailing lists