Re: [PATCH 1/1] amd-pmu: fix s2idle failures on certain AMD laptops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 11/30/21 16:42, Shyam Sundar S K wrote:
> 
> 
> On 11/30/2021 7:34 PM, Limonciello, Mario wrote:
>> On 11/30/2021 06:29, Shyam Sundar S K wrote:
>>>
>>>
>>> On 11/30/2021 4:57 PM, Hans de Goede wrote:
>>>> Hi,
>>>>
>>>> On 11/30/21 05:15, Fabrizio Bertocci wrote:
>>>>> On some AMD hardware laptops, the system fails communicating with the
>>>>> PMU when entering s2idle and the machine is battery powered.
>>>>>
>>>>> Hardware description: HP Pavilion Aero Laptop 13-be0097nr
>>>>> CPU: AMD Ryzen 7 5800U with Radeon Graphics
>>>>> GPU: 03:00.0 VGA compatible controller [0300]: Advanced Micro Devices,
>>>>> Inc. [AMD/ATI] Device [1002:1638] (rev c1)
>>>>>
>>>>> Detailed description of the problem (and investigation) here:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fdrm%2Famd%2F-%2Fissues%2F1799&data=04%7C01%7Cshyam-sundar.s-k%40amd.com%7Ce8ed4219951f4df1dbe708d9b3f4761c%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637738684806355205%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=EwYxPgELsgQ2YpNQROFyptvt2Z0jQ0Yso8wptp6Dz4g%3D&reserved=0
>>>>>
>>>>>
>>>>> Patch is a single line: reduce the polling delay in half, from 100uSec
>>>>> to 50uSec when waiting for a change in state from the PMC after a
>>>>> write command operation.
>>>>>
>>>>> Tested on kernel tree detached at tag 5.14
>>>>> (7d2a07b769330c34b4deabeed939325c77a7ec2f)
>>>>> After changing the delay, I did not see a single failure on this
>>>>> machine (I have this fix for now more than one week and s2idle worked
>>>>> every single time on battery power).
>>>>>
>>>>> Signed-off-by: Fabrizio Bertocci <fabriziobertocci@xxxxxxxxx>
>>>>
>>>> Thank you for your patch. I've added a couple of AMD developers who
>>>> work on this driver to the Cc.
>>>>
>>>> AMD folks, can you review/ack this patch please?
>>>
>>> Looks good to me.
>>>
>>> Acked-by: Shyam Sundar S K <Shyam-sundar.S-k@xxxxxxx>
>>>
>>> Thanks,
>>> Shyam
>>>
>>
>> Fabrizio,
>>
>> Thanks!
>>
>> Hans,
>>
>> Can you please add Cc: stable@xxxxxxxxxxxxxxx when you commit this?
> 
> Also, please help on the subject-line correction, from
> amd-pmu to amd-pmc.

Good point, I've added the Cc: stable and also fixed the Subject.

I will also include this patch in my next fixes pull-req for 5.16.

Regards,

Hans




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux