Re: [REGRESSION] ALSA: usb-audio: Logitech PRO X Wireless 046d:0aba playback volume rejected as sticky
Takashi Iwai <[email protected]>
| Newsgroups | dev.linux.lists.regressions,org.kernel.vger.linux-sound |
|---|---|
| Message-ID | <[email protected]> |
On Fri, 14 Aug 2026 12:41:54 +0200,
Alexander N. wrote:
>
> Hello,
>
> I found a regression in snd-usb-audio affecting the Logitech PRO X
> Wireless Gaming Headset (USB ID 046d:0aba).
>
> The device has a valid UAC1 playback volume control, but the sticky mixer
> detection introduced by commit 86aa1ea1f15c ("ALSA: usb-audio: Do not
> expose sticky mixers") rejects it.
>
> Hardware
> ========
>
> Â Logitech PRO X Wireless Gaming Headset
> Â USB ID: 046d:0aba
>
> Observed kernel
> ===============
>
> Â 7.1.5-ogc5.1.fc44.x86_64
> Â Bazzite/Fedora 44 OGC kernel
>
> The OGC patch for this kernel does not modify sound/usb, and the relevant
> mixer.c code matches upstream Linux 7.1.5.
>
> I have not hardware-tested current mainline 7.2-rc7, but current upstream
> mixer.c still contains the same immediate SET_CUR -> GET_CUR sticky mixer
> test and I could not find a device quirk for 046d:0aba.
>
> Symptom
> =======
>
> Without a workaround, ALSA only exposes:
>
> Â Simple mixer control 'PCM',0
> Â Â Capabilities: pswitch pswitch-joined
>
> There is no PCM Playback Volume control.
>
> The kernel logs:
>
> Â usb 1-10: 2:0: sticky mixer values (-16384/0/256 => -3840), disabling
>
> I have also observed the same message ending in "=> 0", depending on the
> hardware volume at probe time.
>
> Because the playback volume control is removed, the physical volume wheel
> changes the headset's hardware volume independently of the PipeWire/KDE
> system volume.
>
> USB Audio control behavior
> ==========================
>
> The device is UAC1.
>
> Feature Unit 2 exposes a master playback volume control.
>
> Direct control requests, after temporarily unbinding AudioControl
> interface 0 from snd-usb-audio, report:
>
> Â GET_MIN = -16384Â (-64.00 dB)
> Â GET_MAX =Â Â Â 0Â (Â 0.00 dB)
> Â GET_RES =Â Â 256Â (Â 1.00 dB)
>
> GET_CUR and SET_CUR both work, but the device has two quirks relevant to
> the new sticky detection.
>
> 1. GET_CUR reflects normal SET_CUR changes with a delay of roughly 50 ms.
>
> Measured from 0 dB:
>
>  target   result  first visible GET_CUR change
>
> Â Â -1 dBÂ Â Â OKÂ Â Â Â 81.3 ms
> Â Â -2 dBÂ Â Â OKÂ Â Â Â 51.9 ms
> Â Â -4 dBÂ Â Â OKÂ Â Â Â 47.3 ms
> Â Â -8 dBÂ Â Â OKÂ Â Â Â 47.1 ms
> Â -16 dBÂ Â Â OKÂ Â Â Â 51.7 ms
> Â -32 dBÂ Â Â OKÂ Â Â Â 47.2 ms
>
> 2. The advertised minimum value -64 dB is not functional.
>
> A direct SET_CUR to -64 dB returns success, but GET_CUR remains at 0 dB
> even after 1000 ms.
>
> This appears to cause a false positive in check_sticky_volume_control():
>
> - cval->min is -16384 (-64 dB)
> - cval->max is 0
> - if the saved value is 0, max is skipped
> - SET_CUR(min) returns success
> - an immediate GET_CUR still returns the saved value
> - the mixer is classified as sticky and is not registered
>
> mixer_get_cur_broken is not appropriate
> =======================================
>
> I tested the mixer_get_cur_broken quirk.
>
> GET_CUR on this device is not broken or constant. It correctly reports
> host SET_CUR changes after the device delay, and it also reports volume
> changes caused by the physical headset wheel.
>
> Using an internal-only cached value would therefore lose useful hardware
> state.
>
> Local proof-of-concept fix
> ==========================
>
> I built a local snd-usb-audio.ko against the running 7.1.5 kernel.
>
> For this specific device, Feature Unit 2, UAC_FU_VOLUME, I:
>
> - clamp the unusable minimum from -64 dB to -63 dB
> - skip the probe-time sticky/resolution checks for this control
>
> With that module, ALSA exposes:
>
> Â Simple mixer control 'PCM',0
> Â Â Capabilities: pvolume pvolume-joined pswitch pswitch-joined
> Â Â Playback channels: Mono
> Â Â Limits: Playback 0 - 63
>
> /proc/asound/card*/usbmixer shows:
>
> Â Unit: 2
> Â Â Control: name="PCM Playback Volume", index=0
> Â Â Info: id=2, control=2, cmask=0x0, channels=1, type="S16"
> Â Â Volume: min=-16128, max=0, dBmin=-6300, dBmax=0
>
> Most importantly, no userspace workaround is required once the mixer
> control is restored.
>
> Example before rotating the physical headset wheel:
>
> Â ALSA:Â Â Â -27 dB / 57%
> Â PipeWire: 0.35
>
> After rotating the headset wheel:
>
> Â ALSA:Â Â Â -11 dB / 83%
> Â PipeWire: 0.65
>
> KDE system volume follows the physical wheel as expected.
>
> I attached the proof-of-concept diff and the measured results. I am happy
> to test a maintainer-preferred implementation or additional diagnostics.
>
> My suspicion is that this device exposes two assumptions in the sticky
> mixer probe that are not universally safe:
>
> 1. GET_CUR is assumed to reflect SET_CUR immediately.
> 2. advertised min/max values are assumed to be usable test values.
>
> Thanks.
The patch looks simple and safe enough, so if this works for you, it's
fine to take. In that case, please submit a proper patch.
thanks,
Takashi