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.
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 <maciej.fijalkowski@intel.com>
> > > > 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 <tiwai@suse.de>
> 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 <tiwai@suse.de>
> Signed-off-by: Sasha Levin <sashal@kernel.org>
>
> 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
Reply to: