Bug 216613

Summary: Sound doesn't work on 6.0.3
Product: Drivers Reporter: Grzegorz Alibożek (grzegorz.alibozek)
Component: Sound(ALSA)Assignee: Jaroslav Kysela (perex)
Status: RESOLVED CODE_FIX    
Severity: high CC: cezary.rojewski, cruzki123, daniel.serpell, msalle, tiwai, twisted.fall
Priority: P1    
Hardware: Intel   
OS: Linux   
Kernel Version: 6.0.3 Subsystem:
Regression: No Bisected commit-id:
Attachments: trace

Description Grzegorz Alibożek 2022-10-21 19:26:43 UTC 
After upgrade kernel from 6.0.2 to 6.0.3 on Lenovo T14 Gen2i, sound stopped working.
dmesg:

paź 21 21:11:45 kernel: snd_hda_codec_hdmi ehdaudio0D2: failed to create hda codec -12
paź 21 21:11:45 kernel: snd_hda_codec_hdmi ehdaudio0D2: ASoC: error at snd_soc_component_probe on ehdaudio0D2: -12
paź 21 21:11:45 kernel: skl_hda_dsp_generic skl_hda_dsp_generic: ASoC: failed to instantiate card -12
Comment 1 Grzegorz Alibożek 2022-10-21 19:56:43 UTC 
Created attachment 303070 [details]
trace
Comment 2 Takashi Iwai 2022-10-22 07:16:54 UTC 
Could you try to revert the commit 7494e2e6c55ed192f2b91c821fd6832744ba8741?
Comment 3 Grzegorz Alibożek 2022-10-22 08:02:30 UTC 
Revert commit: 7494e2e6c55ed192f2b91c821fd6832744ba874 fixes the problem, the sound works
Comment 4 twisted.fall 2022-10-22 08:17:49 UTC 
Reverting the mentioned commit also works in my case, I was having the same problem
Comment 5 Grzegorz Alibożek 2022-10-22 08:20:00 UTC 
related: https://github.com/thesofproject/linux/issues/3764
Comment 6 Takashi Iwai 2022-10-22 08:23:55 UTC 
Adding Cezary to Cc.  I don't remember exactly which more commits to be needed.

Meanwhile, can people do a quick check whether the issue is present on 6.1-rc1 kernel, too?
Comment 7 Grzegorz Alibożek 2022-10-22 14:44:22 UTC 
yes, on 6.1-rc1 (9abf2313adc1ca1b6180c508c25f22f9395cc780) the sound works fine
Comment 8 Takashi Iwai 2022-10-23 15:04:00 UTC 
Thanks, that's a good news.  So we just miss some commits.
Comment 9 Cezary Rojewski 2022-10-24 08:18:37 UTC 
Hello,

Related patchset on the alsa-devel: https://lore.kernel.org/alsa-devel/20220816111727.3218543-1-cezary.rojewski@intel.com/.

All of the patches in the series are a dependency for the last one. Is "ALSA: hda: Fix page fault in snd_hda_codec_shutdown()" in kernel tree on its own?
Comment 10 dsmc 2022-10-24 13:17:29 UTC 
Hi!

(In reply to Cezary Rojewski from comment #9)
> Related patchset on the alsa-devel:
> https://lore.kernel.org/alsa-devel/20220816111727.3218543-1-cezary.
> rojewski@intel.com/.
> 
> All of the patches in the series are a dependency for the last one. Is
> "ALSA: hda: Fix page fault in snd_hda_codec_shutdown()" in kernel tree on
> its own?

It seems only this patch was backported to 6.0.3: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.0.3&id=7494e2e6c55ed192f2b91c821fd6832744ba8741
Comment 11 Takashi Iwai 2022-10-24 13:26:12 UTC 
IMO, that's too many changes as a stable material, so maybe it's better to drop the patch from 6.0.y...
Comment 12 Cezary Rojewski 2022-10-24 14:25:50 UTC 
Both solutions are fine by me.

The page fault that appears during snd_hda_codec_shutdown() and is being fixed here, requires user to perform module reload scenario under certain conditions.
Comment 13 Cezary Rojewski 2022-10-24 14:27:40 UTC 
(finishing the above one..)

So, it's not a common case by any means.
Comment 14 Takashi Iwai 2022-10-24 14:37:09 UTC 
OK, I'm going to submit the revert patch for 6.0.y.
Comment 15 Takashi Iwai 2022-10-24 15:44:24 UTC 
The revert is queued for the next 6.0.y stable.