From: Takashi Iwai <tiwai@suse.de>
To: Salvatore Bonaccorso <carnil@debian.org>
Cc: "Igor Támara" <igor.tamara@gmail.com>,
"Takashi Iwai" <tiwai@suse.de>,
1108069@bugs.debian.org, stable@vger.kernel.org,
"Kuan-Wei Chiu" <visitorckw@gmail.com>,
linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org,
regressions@lists.linux.dev
Subject: Re: [regression] Builtin recognized microphone Asus X507UA does not record
Date: Thu, 26 Jun 2025 08:04:05 +0200 [thread overview]
Message-ID: <87zfdvght6.wl-tiwai@suse.de> (raw)
In-Reply-To: <aFzLTJg8MN5evbYL@eldamar.lan>
On Thu, 26 Jun 2025 06:23:40 +0200,
Salvatore Bonaccorso wrote:
>
> Hi Igor,
>
> On Wed, Jun 25, 2025 at 07:54:42PM -0500, Igor Támara wrote:
> > Hi Salvatore,
> >
> >
> > El mié, 25 jun 2025 a las 13:47, Salvatore Bonaccorso
> > (<carnil@debian.org>) escribió:
> > >
> > > Hi Igor,
> > >
> > > [For context, there was a regression report in Debian at
> > > https://bugs.debian.org/1108069]
> > >
> > > On Thu, Jun 19, 2025 at 09:36:13PM -0500, Igor Tamara wrote:
> > > > Package: src:linux
> > > > Version: 6.12.32-1
> > > > Severity: normal
> > > > Tags: a11y
> > > >
> > > > Dear Maintainer,
> > > >
> > > > The builtin microphone on my Asus X507UA does not record, is
> > > > recognized and some time ago it worked on Bookworm with image-6.1.0-31,
> > > > newer images are able to record when appending snd_hda_intel.model=1043:1271
> > > > to the boot as a workaround.
> > > >
> > > > The images that work with the boot option appended are, but not without
> > > > it are:
> > > >
> > > > linux-image-6.15-amd64
> > > > linux-image-6.12.32-amd64
> > > > linux-image-6.1.0-37-amd64
> > > > linux-image-6.1.0-0.a.test-amd64-unsigned_6.1.129-1a~test_amd64.deb
> > > > referenced by https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1100928
> > > > Also compiled from upstream 6.12.22 and 6.1.133 with the same result
> > > >
> > > > The image linux-image-6.1.0-31-amd64 worked properly, the problem was
> > > > introduced in 129 and the result of the bisect was
> > > >
> > > > d26408df0e25f2bd2808d235232ab776e4dd08b9 is the first bad commit
> > > > commit d26408df0e25f2bd2808d235232ab776e4dd08b9
> > > > Author: Kuan-Wei Chiu <visitorckw@gmail.com>
> > > > Date: Wed Jan 29 00:54:15 2025 +0800
> > > >
> > > > ALSA: hda: Fix headset detection failure due to unstable sort
> > > >
> > > > commit 3b4309546b48fc167aa615a2d881a09c0a97971f upstream.
> > > >
> > > > The auto_parser assumed sort() was stable, but the kernel's sort() uses
> > > > heapsort, which has never been stable. After commit 0e02ca29a563
> > > > ("lib/sort: optimize heapsort with double-pop variation"), the order of
> > > > equal elements changed, causing the headset to fail to work.
> > > >
> > > > Fix the issue by recording the original order of elements before
> > > > sorting and using it as a tiebreaker for equal elements in the
> > > > comparison function.
> > > >
> > > > Fixes: b9030a005d58 ("ALSA: hda - Use standard sort function in hda_auto_parser.c")
> > > > Reported-by: Austrum <austrum.lab@gmail.com>
> > > > Closes: https://bugzilla.kernel.org/show_bug.cgi?id=219158
> > > > Tested-by: Austrum <austrum.lab@gmail.com>
> > > > Cc: stable@vger.kernel.org
> > > > Signed-off-by: Kuan-Wei Chiu <visitorckw@gmail.com>
> > > > Link: https://patch.msgid.link/20250128165415.643223-1-visitorckw@gmail.com
> > > > Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > >
> > > > sound/pci/hda/hda_auto_parser.c | 8 +++++++-
> > > > sound/pci/hda/hda_auto_parser.h | 1 +
> > > > 2 files changed, 8 insertions(+), 1 deletion(-)
> > > >
> > > > I'm attaching the output of alsa-info_alsa-info.sh script
> > > >
> > > > Please let me know if I can provide more information.
> > >
> > > Might you be able to try please the attached patch to see if it fixes
> > > the issue?
> > >
> >
> > I recompiled and the mic is recording without issues when running on
> > 6.1.133 and 6.12.32
>
> Thanks for the confirmation! Takashi, can you apply the proposed
> change (slightly improved in attached variant to add Reported-by and
> Closes tags), hopefully getting it into required stable series?
Thanks, applied now.
Takashi
prev parent reply other threads:[~2025-06-26 6:04 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <175038697334.5297.17990232291668400728.reportbug@donsam>
2025-06-25 18:47 ` [regression] Builtin recognized microphone Asus X507UA does not record Salvatore Bonaccorso
2025-06-26 0:54 ` Igor Támara
2025-06-26 4:23 ` Salvatore Bonaccorso
2025-06-26 6:04 ` Takashi Iwai [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87zfdvght6.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=1108069@bugs.debian.org \
--cc=carnil@debian.org \
--cc=igor.tamara@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=regressions@lists.linux.dev \
--cc=stable@vger.kernel.org \
--cc=visitorckw@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.