From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 30EBE8483 for ; Tue, 3 Jan 2023 16:26:52 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 6045B67E98; Tue, 3 Jan 2023 16:26:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1672763210; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pY3jzxaENXIf/F3kBvSDUUjACo7aZEQ2xT84WOtvWIk=; b=n5WwMClmrUr7C2YCBNPycTpVcSloPJI1OglbHOsK8t9WzilezolS7QWZrQ8qqTqIkN5rDj snD0mmlZq3QycW75tvRiXjGuoaTGbo1sPkm4/nPkC0l7JZnLw3SueSgfojywtwV56AqG6j L4nxwqvz/gJ9SNhB4M8XQoFPf9N970o= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1672763210; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pY3jzxaENXIf/F3kBvSDUUjACo7aZEQ2xT84WOtvWIk=; b=l6rnYthXUigefSFln7FGpieHONOnzIhQMunHWDr38i7I1JNuE6waHuv8b58c60K3lFj73P h0tt/TNEVtLMUfCw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 23E581392B; Tue, 3 Jan 2023 16:26:50 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id zvP5B0pXtGPjDAAAMHmgww (envelope-from ); Tue, 03 Jan 2023 16:26:50 +0000 Date: Tue, 03 Jan 2023 17:26:49 +0100 Message-ID: <87sfgrpos6.wl-tiwai@suse.de> From: Takashi Iwai To: Greg Kroah-Hartman Cc: stable@vger.kernel.org, =?ISO-8859-1?Q?P=C1LFFY_D=E1niel?= , Alsa-devel , "regressions@lists.linux.dev" , Mark Brown , LKML , Takashi Iwai , Richard Fitzgerald , Thorsten Leemhuis , Sergey , Salvatore Bonaccorso Subject: Re: [regression, 5.10.y] Bug 216861 - sound disappearance on Acer Swift 3 SF314-59-78UR after update to 5.10.157 In-Reply-To: <87h6x7r7w6.wl-tiwai@suse.de> References: <87wn65umye.wl-tiwai@suse.de> <87h6x7r7w6.wl-tiwai@suse.de> User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0 Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit On Tue, 03 Jan 2023 15:48:41 +0100, Takashi Iwai wrote: > > On Tue, 03 Jan 2023 14:04:50 +0100, > PÁLFFY Dániel wrote: > > > > And confirming, 5.10.161 with e8444560b4d9302a511f0996f4cfdf85b628f4ca > > and 636110411ca726f19ef8e87b0be51bb9a4cdef06 cherry-picked works for > > me. > > That's a good news. Then we can ask stable people to pick up those > commits for 5.10.y and 5.15.y. I confirmed that the latest 5.15.y requires those fixes, too. Greg, could you cherry-pick the following two commits to both 5.10.y and 5.15.y stable trees? This fixes the recent regression caused by the backport of 39bd801d6908. e8444560b4d9302a511f0996f4cfdf85b628f4ca ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire 636110411ca726f19ef8e87b0be51bb9a4cdef06 ASoC: Intel/SOF: use set_stream() instead of set_tdm_slots() for HDAudio Thanks! Takashi > > > Takashi > > > > > On Tue, Jan 3, 2023 at 1:05 PM PÁLFFY Dániel wrote: > > > > > > Another report: https://bugs.archlinux.org/task/76795 > > > Apparently, folks at alsa-devel traced down the dependencies of that patch, see the mail thread at https://lore.kernel.org/all/dc65501c-c2fd-5608-c3d9-7cea184c3989%40opensource.cirrus.com/ > > > > > > On Mon, Jan 2, 2023 at 1:42 PM Takashi Iwai wrote: > > >> > > >> On Mon, 02 Jan 2023 11:43:36 +0100, > > >> Salvatore Bonaccorso wrote: > > >> > > > >> > Hi, > > >> > > > >> > [Adding as well Richard Fitzgerald and PÁLFFY Dániel to recipients] > > >> > > > >> > On Fri, Dec 30, 2022 at 09:08:57AM +0100, Thorsten Leemhuis wrote: > > >> > > Hi, this is your Linux kernel regression tracker speaking. > > >> > > > > >> > > I noticed a regression report in bugzilla.kernel.org. As many (most?) > > >> > > kernel developer don't keep an eye on it, I decided to forward it by > > >> > > mail. Quoting from https://bugzilla.kernel.org/show_bug.cgi?id=216861 : > > >> > > > > >> > > > Sergey 2022-12-29 10:07:51 UTC > > >> > > > > > >> > > > Created attachment 303497 [details] > > >> > > > pulseaudio.log > > >> > > > > > >> > > > Sudden sound disappearance was reported for some laptops, e.g. > > >> > > > > > >> > > > Acer Swift 3 SF314-59-78UR 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz > > >> > > > > > >> > > > # lspci > > >> > > > 0000:00:1f.3 Multimedia audio controller: Intel Corporation Tiger Lake-LP Smart Sound Technology Audio Controller (rev 20) > > >> > > > Subsystem: Acer Incorporated [ALI] Device 148c > > >> > > > Flags: bus master, fast devsel, latency 32, IRQ 197, IOMMU group 12 > > >> > > > Memory at 601f270000 (64-bit, non-prefetchable) [size=16K] > > >> > > > Memory at 601f000000 (64-bit, non-prefetchable) [size=1M] > > >> > > > Capabilities: [50] Power Management version 3 > > >> > > > Capabilities: [80] Vendor Specific Information: Len=14 > > >> > > > Capabilities: [60] MSI: Enable+ Count=1/1 Maskable- 64bit+ > > >> > > > Kernel driver in use: sof-audio-pci > > >> > > > > > >> > > > I am attaching the pulseaudio and dmesg logs > > >> > > > > > >> > > > This bug started reproducing after updating the kernel from 5.10.156 to 5.10.157 > > >> > > > > > >> > > > Bisection revealed the commit being reverted: > > >> > > > > > >> > > > c34db0d6b88b1da95e7ab3353e674f4f574cccee is the first bad commit > > >> > > > commit c34db0d6b88b1da95e7ab3353e674f4f574cccee > > >> > > > Author: Richard Fitzgerald > > >> > > > Date: Fri Nov 4 13:22:13 2022 +0000 > > >> > > > > > >> > > > ASoC: soc-pcm: Don't zero TDM masks in __soc_pcm_open() > > >> > > > > > >> > > > [ Upstream commit 39bd801d6908900e9ab0cdc2655150f95ddd4f1a ] > > >> > > > > > >> > > > The DAI tx_mask and rx_mask are set by snd_soc_dai_set_tdm_slot() > > >> > > > and used by later code that depends on the TDM settings. So > > >> > > > __soc_pcm_open() should not be obliterating those mask values. > > >> > > > > > >> > > > [...] > > >> > > > Original bug report: https://bugzilla.altlinux.org/44690 > > >> > > > > >> > > See the ticket for more details. > > >> > > > > >> > > BTW, let me use this mail to also add the report to the list of tracked > > >> > > regressions to ensure it's doesn't fall through the cracks: > > >> > > > > >> > > #regzbot introduced: c34db0d6b88b1d > > >> > > https://bugzilla.kernel.org/show_bug.cgi?id=216861 > > >> > > #regzbot title: sound: asoc: sudden sound disappearance > > >> > > #regzbot ignore-activity > > >> > > > >> > FWIW, we had as well reports in Debian after having updated the kernel > > >> > from 5.10.149 based one to 5.10.158 based one in the last point > > >> > releases, they are at least: > > >> > > > >> > https://bugs.debian.org/1027483 > > >> > https://bugs.debian.org/1027430 > > >> > > >> I got another report while the commit was backported to 5.14-based > > >> openSUSE Leap kernel, and I ended up with dropping it. > > >> > > >> So, IMO, it's safer to drop this patch from the older stable trees. > > >> As far as I see, 5.15.y and 5.10.y got this. > > >> > > >> Unless anyone gives a better fix, I'm going to submit a revert patch > > >> for those trees. > > >> > > >> > > >> thanks, > > >> > > >> Takashi > > > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 72588C3DA7D for ; Tue, 3 Jan 2023 16:27:47 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 10679530E; Tue, 3 Jan 2023 17:26:55 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 10679530E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1672763265; bh=CZ5jQxUZ/A8eEHJqgKv81eL++P47gI2oxmhDh0VWePw=; h=Date:From:To:Subject:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=LpbFM1mP6BepVwHii/+AYatsj26RE16WFaNxiVNthbYCqVCRQDfGZ6KiCcvc/m8rh 1fIMYxo4AOQUxgiw2dpubXG8su7jyD3RLql5b4geVAJq/6Pi03NLPXiN04MQKCPg8d YsvYtJnX106KCMbsQj2uWySD18C3ygkq0BrhWMfE= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id ACE52F803DC; Tue, 3 Jan 2023 17:26:54 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 4FE91F8047B; Tue, 3 Jan 2023 17:26:53 +0100 (CET) Received: from smtp-out2.suse.de (smtp-out2.suse.de [IPv6:2001:67c:2178:6::1d]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 763DBF8003C for ; Tue, 3 Jan 2023 17:26:50 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 763DBF8003C Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key, unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=n5WwMClm; dkim=pass header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=l6rnYthX Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 6045B67E98; Tue, 3 Jan 2023 16:26:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1672763210; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pY3jzxaENXIf/F3kBvSDUUjACo7aZEQ2xT84WOtvWIk=; b=n5WwMClmrUr7C2YCBNPycTpVcSloPJI1OglbHOsK8t9WzilezolS7QWZrQ8qqTqIkN5rDj snD0mmlZq3QycW75tvRiXjGuoaTGbo1sPkm4/nPkC0l7JZnLw3SueSgfojywtwV56AqG6j L4nxwqvz/gJ9SNhB4M8XQoFPf9N970o= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1672763210; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pY3jzxaENXIf/F3kBvSDUUjACo7aZEQ2xT84WOtvWIk=; b=l6rnYthXUigefSFln7FGpieHONOnzIhQMunHWDr38i7I1JNuE6waHuv8b58c60K3lFj73P h0tt/TNEVtLMUfCw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 23E581392B; Tue, 3 Jan 2023 16:26:50 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id zvP5B0pXtGPjDAAAMHmgww (envelope-from ); Tue, 03 Jan 2023 16:26:50 +0000 Date: Tue, 03 Jan 2023 17:26:49 +0100 Message-ID: <87sfgrpos6.wl-tiwai@suse.de> From: Takashi Iwai To: Greg Kroah-Hartman Subject: Re: [regression, 5.10.y] Bug 216861 - sound disappearance on Acer Swift 3 SF314-59-78UR after update to 5.10.157 In-Reply-To: <87h6x7r7w6.wl-tiwai@suse.de> References: <87wn65umye.wl-tiwai@suse.de> <87h6x7r7w6.wl-tiwai@suse.de> User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0 MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alsa-devel , "regressions@lists.linux.dev" , Thorsten Leemhuis , LKML , stable@vger.kernel.org, Takashi Iwai , Richard Fitzgerald , Mark Brown , =?ISO-8859-1?Q?P=C1LFFY_D=E1niel?= , Salvatore Bonaccorso , Sergey Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Tue, 03 Jan 2023 15:48:41 +0100, Takashi Iwai wrote: > > On Tue, 03 Jan 2023 14:04:50 +0100, > PÁLFFY Dániel wrote: > > > > And confirming, 5.10.161 with e8444560b4d9302a511f0996f4cfdf85b628f4ca > > and 636110411ca726f19ef8e87b0be51bb9a4cdef06 cherry-picked works for > > me. > > That's a good news. Then we can ask stable people to pick up those > commits for 5.10.y and 5.15.y. I confirmed that the latest 5.15.y requires those fixes, too. Greg, could you cherry-pick the following two commits to both 5.10.y and 5.15.y stable trees? This fixes the recent regression caused by the backport of 39bd801d6908. e8444560b4d9302a511f0996f4cfdf85b628f4ca ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire 636110411ca726f19ef8e87b0be51bb9a4cdef06 ASoC: Intel/SOF: use set_stream() instead of set_tdm_slots() for HDAudio Thanks! Takashi > > > Takashi > > > > > On Tue, Jan 3, 2023 at 1:05 PM PÁLFFY Dániel wrote: > > > > > > Another report: https://bugs.archlinux.org/task/76795 > > > Apparently, folks at alsa-devel traced down the dependencies of that patch, see the mail thread at https://lore.kernel.org/all/dc65501c-c2fd-5608-c3d9-7cea184c3989%40opensource.cirrus.com/ > > > > > > On Mon, Jan 2, 2023 at 1:42 PM Takashi Iwai wrote: > > >> > > >> On Mon, 02 Jan 2023 11:43:36 +0100, > > >> Salvatore Bonaccorso wrote: > > >> > > > >> > Hi, > > >> > > > >> > [Adding as well Richard Fitzgerald and PÁLFFY Dániel to recipients] > > >> > > > >> > On Fri, Dec 30, 2022 at 09:08:57AM +0100, Thorsten Leemhuis wrote: > > >> > > Hi, this is your Linux kernel regression tracker speaking. > > >> > > > > >> > > I noticed a regression report in bugzilla.kernel.org. As many (most?) > > >> > > kernel developer don't keep an eye on it, I decided to forward it by > > >> > > mail. Quoting from https://bugzilla.kernel.org/show_bug.cgi?id=216861 : > > >> > > > > >> > > > Sergey 2022-12-29 10:07:51 UTC > > >> > > > > > >> > > > Created attachment 303497 [details] > > >> > > > pulseaudio.log > > >> > > > > > >> > > > Sudden sound disappearance was reported for some laptops, e.g. > > >> > > > > > >> > > > Acer Swift 3 SF314-59-78UR 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz > > >> > > > > > >> > > > # lspci > > >> > > > 0000:00:1f.3 Multimedia audio controller: Intel Corporation Tiger Lake-LP Smart Sound Technology Audio Controller (rev 20) > > >> > > > Subsystem: Acer Incorporated [ALI] Device 148c > > >> > > > Flags: bus master, fast devsel, latency 32, IRQ 197, IOMMU group 12 > > >> > > > Memory at 601f270000 (64-bit, non-prefetchable) [size=16K] > > >> > > > Memory at 601f000000 (64-bit, non-prefetchable) [size=1M] > > >> > > > Capabilities: [50] Power Management version 3 > > >> > > > Capabilities: [80] Vendor Specific Information: Len=14 > > >> > > > Capabilities: [60] MSI: Enable+ Count=1/1 Maskable- 64bit+ > > >> > > > Kernel driver in use: sof-audio-pci > > >> > > > > > >> > > > I am attaching the pulseaudio and dmesg logs > > >> > > > > > >> > > > This bug started reproducing after updating the kernel from 5.10.156 to 5.10.157 > > >> > > > > > >> > > > Bisection revealed the commit being reverted: > > >> > > > > > >> > > > c34db0d6b88b1da95e7ab3353e674f4f574cccee is the first bad commit > > >> > > > commit c34db0d6b88b1da95e7ab3353e674f4f574cccee > > >> > > > Author: Richard Fitzgerald > > >> > > > Date: Fri Nov 4 13:22:13 2022 +0000 > > >> > > > > > >> > > > ASoC: soc-pcm: Don't zero TDM masks in __soc_pcm_open() > > >> > > > > > >> > > > [ Upstream commit 39bd801d6908900e9ab0cdc2655150f95ddd4f1a ] > > >> > > > > > >> > > > The DAI tx_mask and rx_mask are set by snd_soc_dai_set_tdm_slot() > > >> > > > and used by later code that depends on the TDM settings. So > > >> > > > __soc_pcm_open() should not be obliterating those mask values. > > >> > > > > > >> > > > [...] > > >> > > > Original bug report: https://bugzilla.altlinux.org/44690 > > >> > > > > >> > > See the ticket for more details. > > >> > > > > >> > > BTW, let me use this mail to also add the report to the list of tracked > > >> > > regressions to ensure it's doesn't fall through the cracks: > > >> > > > > >> > > #regzbot introduced: c34db0d6b88b1d > > >> > > https://bugzilla.kernel.org/show_bug.cgi?id=216861 > > >> > > #regzbot title: sound: asoc: sudden sound disappearance > > >> > > #regzbot ignore-activity > > >> > > > >> > FWIW, we had as well reports in Debian after having updated the kernel > > >> > from 5.10.149 based one to 5.10.158 based one in the last point > > >> > releases, they are at least: > > >> > > > >> > https://bugs.debian.org/1027483 > > >> > https://bugs.debian.org/1027430 > > >> > > >> I got another report while the commit was backported to 5.14-based > > >> openSUSE Leap kernel, and I ended up with dropping it. > > >> > > >> So, IMO, it's safer to drop this patch from the older stable trees. > > >> As far as I see, 5.15.y and 5.10.y got this. > > >> > > >> Unless anyone gives a better fix, I'm going to submit a revert patch > > >> for those trees. > > >> > > >> > > >> thanks, > > >> > > >> Takashi > > >