Path: csiph.com!news.samoylyk.net!gothmog.csi.it!bofh.it!news.nic.it!robomod From: Salvatore Bonaccorso Newsgroups: linux.debian.bugs.dist,linux.debian.kernel Subject: Bug#1081833: linux-image-6.1.0-25-amd64: regression on linux-image-6.1.0-25.amd64 regular cracks in HDMI sound during playback - 6.1.0-23 works. Date: Wed, 25 Sep 2024 18:30:01 +0200 Message-ID: References: X-Mailbox-Line: From debian-bugs-dist-request@lists.debian.org Wed Sep 25 16:24:11 2024 Old-Return-Path: X-Spam-Flag: NO X-Spam-Score: -1.699 Reply-To: Salvatore Bonaccorso , 1081833@bugs.debian.org Resent-To: debian-bugs-dist@lists.debian.org Resent-Cc: Debian Kernel Team X-Debian-Pr-Message: followup 1081833 X-Debian-Pr-Package: src:linux X-Debian-Pr-Keywords: moreinfo X-Debian-Pr-Source: linux MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit X-Debian-User: carnil X-Debian-Message: from BTS X-Mailing-List: archive/latest/1860143 List-ID: List-URL: Approved: robomod@news.nic.it Lines: 76 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Date: Wed, 25 Sep 2024 18:21:48 +0200 X-Original-Message-ID: X-Original-References: <172640086161.25847.14378207867590842829.reportbug@gwaihir.grabatoulnz.fr> <172640086161.25847.14378207867590842829.reportbug@gwaihir.grabatoulnz.fr> <07454e99-0327-40b6-867a-3cf19eb4b511@grabatoulnz.fr> <172640086161.25847.14378207867590842829.reportbug@gwaihir.grabatoulnz.fr> <07454e99-0327-40b6-867a-3cf19eb4b511@grabatoulnz.fr> Xref: csiph.com linux.debian.bugs.dist:1214243 linux.debian.kernel:84102 Control: tags -1 - moreinfo Hi Eric, On Wed, Sep 25, 2024 at 07:54:40AM +0200, Eric wrote: > Hi, > > new bisect done ! > > On 24/09/2024 21:05, Salvatore Bonaccorso wrote: > > Hi, > > > > On Tue, Sep 24, 2024 at 08:17:34PM +0200, Eric wrote: > > > > On 22/09/2024 21:22, Eric wrote: > > > > The bisect is complete and yields this result > > > > > > > > 15115033f056cbd7649b8e1806287f71bdb7ce5c is the first bad commit > > > > commit 15115033f056cbd7649b8e1806287f71bdb7ce5c > > > > Author: Maciej Fijalkowski > > > > Date:   Fri Jul 26 20:17:10 2024 +0200 > > > > > > > >     ice: don't busy wait for Rx queue disable in ice_qp_dis() > > > > > > > > > > > > I posted too early ... looking back I found that nonsensical. Turns out on that last commit I had made a mistake. I suspect I booted on the wrong kernel that time. > > > > I'll redo the last few rounds and post only after checking that the "first bad commit" indeed is bad. > > > > Sorry for the noise > > No problem, and thanks for the update. Yes I was confused once I saw > > your mail and did not saw a relation of this to your HDMI sound issue. > > Now, that one make a lot more sense (even though I wouldn't know how to > handle this, it is clearly related) : > > > 92afcc310038ebe5d66c689bb0bf418f5451201c is the first bad commit > commit 92afcc310038ebe5d66c689bb0bf418f5451201c > Author: Takashi Iwai > Date:   Wed Jul 31 19:05:15 2024 +0200 > >     ALSA: hda: Conditionally use snooping for AMD HDMI > >     [ Upstream commit 478689b5990deb626a0b3f1ebf165979914d6be4 ] > >     The recent regression report revealed that the use of WC pages for AMD >     HDMI device together with AMD IOMMU leads to unexpected truncation or >     noises.  The issue seems triggered by the change in the kernel core >     memory allocation that enables IOMMU driver to use always S/G >     buffers.  Meanwhile, the use of WC pages has been a workaround for the >     similar issue with standard pages in the past.  So, now we need to >     apply the workaround conditionally, namely, only when IOMMU isn't in >     place. > >     This patch modifies the workaround code to check the DMA ops at first >     and apply the snoop-off only when needed. > >     Fixes: f5ff79fddf0e ("dma-mapping: remove CONFIG_DMA_REMAP") >     Link: https://bugzilla.kernel.org/show_bug.cgi?id=219087 >     Link: https://patch.msgid.link/20240731170521.31714-1-tiwai@suse.de >     Signed-off-by: Takashi Iwai >     Signed-off-by: Sasha Levin > >  sound/pci/hda/hda_controller.h |  2 +- >  sound/pci/hda/hda_intel.c      | 10 +++++++++- > >  2 files changed, 10 insertions(+), 2 deletions(-) > > > Regards Awesome, thanks for the bisect. Let's see how we can move from here on to report it to upstream with your case. Regards, Salvatore