From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wp530.webpack.hosteurope.de (wp530.webpack.hosteurope.de [80.237.130.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 65D0F368 for ; Wed, 4 Jan 2023 10:35:11 +0000 (UTC) Received: from [2a02:8108:963f:de38:eca4:7d19:f9a2:22c5]; authenticated by wp530.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1pD16y-0006GG-7N; Wed, 04 Jan 2023 11:35:08 +0100 Message-ID: <60aa306c-2047-59ab-24ba-a6fba6667a3d@leemhuis.info> Date: Wed, 4 Jan 2023 11:35:04 +0100 Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 Subject: Re: Request for cherry-picks for sound (Re: [regression, 5.10.y] Bug 216861) Content-Language: en-US, de-DE To: Takashi Iwai , Greg Kroah-Hartman Cc: stable@vger.kernel.org, =?UTF-8?B?UMOBTEZGWSBEw6FuaWVs?= , Alsa-devel , "regressions@lists.linux.dev" , Mark Brown , LKML , Takashi Iwai , Richard Fitzgerald , Sergey , Salvatore Bonaccorso References: <87wn65umye.wl-tiwai@suse.de> <87h6x7r7w6.wl-tiwai@suse.de> <87sfgrpos6.wl-tiwai@suse.de> <87wn62obhm.wl-tiwai@suse.de> From: "Linux kernel regression tracking (#info)" Reply-To: Thorsten Leemhuis In-Reply-To: <87wn62obhm.wl-tiwai@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-bounce-key: webpack.hosteurope.de;regressions@leemhuis.info;1672828511;a40a4397; X-HE-SMSGID: 1pD16y-0006GG-7N [TLDR: This mail in primarily relevant for Linux kernel regression tracking. See link in footer if these mails annoy you.] On 04.01.23 11:11, Takashi Iwai wrote: > Greg, just in case you missed my previous post. Side note: thx for handling this, Takashi! > Could you cherry-pick the following two commits to 5.10.y and 5.15.y > stable trees? > > e8444560b4d9302a511f0996f4cfdf85b628f4ca > ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire > > 636110411ca726f19ef8e87b0be51bb9a4cdef06 > ASoC: Intel/SOF: use set_stream() instead of set_tdm_slots() for HDAudio #regzbot fix: ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire /me just picked one of them as the fix for the tracked regression, even if both are needed, as that's likely close enough, unless something really unexpected happens Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat) > On Tue, 03 Jan 2023 17:26:49 +0100, > Takashi Iwai wrote: >> >> 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 >>>> >>> > > -- Everything you wanna know about Linux kernel regression tracking: https://linux-regtracking.leemhuis.info/about/#tldr That page also explains what to do if mails like this annoy you. 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 88B15C4332F for ; Wed, 4 Jan 2023 10:36:11 +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 E820FA3E8; Wed, 4 Jan 2023 11:35:18 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz E820FA3E8 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1672828569; bh=D4NpByt381Wxc3AfgPTZUhsdAKSu1MrR6gFvAJh8z7Q=; h=Date:Subject:To:References:From:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Reply-To:Cc:From; b=RVQiQxmvx5RF77B/wAmFM8hu8FLNqcG65ISbETe83dRuL+5zB1G9vvdL+Np0Nr6FA 2k3mF8/uZ/gEuwKmj7W8AhdkSeD472jYCuwBBG6P1ZeIppe3pi9qzVbTaFwzTyisdL xEAv+OfjQEGJXV0ftQ8uCPhsubo6nrOhRf+vpKqI= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id A3AD0F80553; Wed, 4 Jan 2023 11:35:18 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 7644BF8055B; Wed, 4 Jan 2023 11:35:17 +0100 (CET) Received: from wp530.webpack.hosteurope.de (wp530.webpack.hosteurope.de [80.237.130.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 197BDF8023B for ; Wed, 4 Jan 2023 11:35:09 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 197BDF8023B Received: from [2a02:8108:963f:de38:eca4:7d19:f9a2:22c5]; authenticated by wp530.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1pD16y-0006GG-7N; Wed, 04 Jan 2023 11:35:08 +0100 Message-ID: <60aa306c-2047-59ab-24ba-a6fba6667a3d@leemhuis.info> Date: Wed, 4 Jan 2023 11:35:04 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 Subject: Re: Request for cherry-picks for sound (Re: [regression, 5.10.y] Bug 216861) Content-Language: en-US, de-DE To: Takashi Iwai , Greg Kroah-Hartman References: <87wn65umye.wl-tiwai@suse.de> <87h6x7r7w6.wl-tiwai@suse.de> <87sfgrpos6.wl-tiwai@suse.de> <87wn62obhm.wl-tiwai@suse.de> From: "Linux kernel regression tracking (#info)" In-Reply-To: <87wn62obhm.wl-tiwai@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-bounce-key: webpack.hosteurope.de; regressions@leemhuis.info; 1672828511; a40a4397; X-HE-SMSGID: 1pD16y-0006GG-7N 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: , Reply-To: Thorsten Leemhuis Cc: Alsa-devel , "regressions@lists.linux.dev" , LKML , stable@vger.kernel.org, Takashi Iwai , Richard Fitzgerald , Mark Brown , =?UTF-8?B?UMOBTEZGWSBEw6FuaWVs?= , Salvatore Bonaccorso , Sergey Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" [TLDR: This mail in primarily relevant for Linux kernel regression tracking. See link in footer if these mails annoy you.] On 04.01.23 11:11, Takashi Iwai wrote: > Greg, just in case you missed my previous post. Side note: thx for handling this, Takashi! > Could you cherry-pick the following two commits to 5.10.y and 5.15.y > stable trees? > > e8444560b4d9302a511f0996f4cfdf85b628f4ca > ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire > > 636110411ca726f19ef8e87b0be51bb9a4cdef06 > ASoC: Intel/SOF: use set_stream() instead of set_tdm_slots() for HDAudio #regzbot fix: ASoC/SoundWire: dai: expand 'stream' concept beyond SoundWire /me just picked one of them as the fix for the tracked regression, even if both are needed, as that's likely close enough, unless something really unexpected happens Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat) > On Tue, 03 Jan 2023 17:26:49 +0100, > Takashi Iwai wrote: >> >> 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 >>>> >>> > > -- Everything you wanna know about Linux kernel regression tracking: https://linux-regtracking.leemhuis.info/about/#tldr That page also explains what to do if mails like this annoy you.