2021-09-09 11:57:35

by Takashi Iwai

[permalink] [raw]
Subject: Re: [PATCH AUTOSEL 5.14 166/252] ALSA: hda: Drop workaround for a hang at shutdown again

On Thu, 09 Sep 2021 13:39:40 +0200,
Sasha Levin wrote:
>
> From: Takashi Iwai <[email protected]>
>
> [ Upstream commit 8fc8e903156f42c66245838441d03607e9067381 ]
>
> The commit 0165c4e19f6e ("ALSA: hda: Fix hang during shutdown due to
> link reset") modified the shutdown callback of the HD-audio controller
> for working around a hang. Meanwhile, the actual culprit of the hang
> was identified to be the leftover active codecs that may interfere
> with the powered down controller somehow, but we took a minimal fix
> approach for 5.14, and that was the commit above.
>
> Now, since the codec drivers go runtime-suspend at shutdown for 5.15,
> we can revert the change and make sure that the full runtime-suspend
> is performed at shutdown of HD-audio controller again. This patch
> essentially reverts the commit above to restore the behavior.
>
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=214045
> Link: https://lore.kernel.org/r/[email protected]
> Signed-off-by: Takashi Iwai <[email protected]>
> Signed-off-by: Sasha Levin <[email protected]>

Please drop this, it's not for 5.14.
As stated in the commit log, the fundamental fix was merged in 5.15
that obsoletes the workaround, but it isn't included in 5.14.


thanks,

Takashi