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: [eric.4.debian@grabatoulnz.fr: Re: 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:10 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/1860142 List-ID: List-URL: Approved: robomod@news.nic.it Lines: 68 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Date: Wed, 25 Sep 2024 18:20:39 +0200 X-Original-Message-ID: X-Original-References: <172640086161.25847.14378207867590842829.reportbug@gwaihir.grabatoulnz.fr> Xref: csiph.com linux.debian.bugs.dist:1214244 linux.debian.kernel:84103 ----- Forwarded message from Eric ----- 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 Eric