Re: [REGRESSION v6.1-rc1] rtc: cmos: rtcwake broken on NUC8i7HVK

From: Kalle Valo
Date: Tue Oct 18 2022 - 13:40:21 EST


"Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx> writes:

> On 10/18/2022 9:09 AM, Kalle Valo wrote:
>
>> Thorsten Leemhuis <regressions@xxxxxxxxxxxxx> writes:
>>
>>> On 17.10.22 18:58, Kalle Valo wrote:
>>>> I upgraded my NUC8i7HVK test box to v6.1-rc1 and noticed that my suspend
>>>> tests with ath11k were broken, the box never woke up from suspend
>>>> automatically and I had to manually push the power button to trigger
>>>> resume. This is the command I used:
>>>>
>>>> sudo rtcwake -m mem -s 30
>>>>
>>>> v6.0 works without problems and a bisect found this commit:
>>>>
>>>> commit 4919d3eb2ec0ee364f7e3cf2d99646c1b224fae8
>>>> Author: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>>>> AuthorDate: Wed Oct 12 20:07:01 2022 +0200
>>>> Commit: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>
>>>> CommitDate: Thu Oct 13 23:27:52 2022 +0200
>>>>
>>>> rtc: cmos: Fix event handler registration ordering issue
>>> FWIW, Todd also reported this yesterday:
>>> https://lore.kernel.org/all/967cfe4e2dba242352ccd1cd00bdbcfb48bdd697.camel@xxxxxxxxxxxxxxx/
>>> https://bugzilla.kernel.org/show_bug.cgi?id=216600
>>>
>>> And Zhang Rui provided a patch in bugzilla that should fix this.
>> Thanks, this is good info. Strangely enough Zhang's workaround in [1]
>> did not fix the issue for me, rtcwake is still broken. Unfortunately no
>> time right now to investigate further.
>>
>> [1] https://bugzilla.kernel.org/show_bug.cgi?id=216600#c3
>
> Please try https://lore.kernel.org/linux-acpi/5887691.lOV4Wx5bFT@kreacher/

This works, thanks!

Tested-by: Kalle Valo <kvalo@xxxxxxxxxx>

--
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches