Re: [PATCH 0/2] selftests/alsa: Catch writes that silently change other controls

From: Takashi Iwai

Date: Sun Oct 04 2026 - 14:24:20 EST


On Wed, 30 Sep 2026 13:15:32 +0200,
HyeongJun An wrote:
>
> Mark asked for this in the es8316 thread [1]: have mixer-test check
> for other controls changing when one is written. Patch 2 adds that.
>
> Patch 1 fixes a bug that testing patch 2 turned up. With a driver that
> notifies a neighbour with a value event, wait_for_event() returns on
> the neighbour's event and the next test reports a spurious one.
>
> The cost is two extra writes per control and a read of every other
> control on the card after each. On the two cards here, 39 controls in
> all, the run time does not move. I have no card with hundreds of
> controls to measure against the 45 second kselftest timeout.
>
> [1] https://lore.kernel.org/all/0fefbfe1-021b-46b2-8945-15956e75e37f@xxxxxxxxxxxxx/
>
> HyeongJun An (2):
> selftests/alsa: Do not take another control's event as our own
> selftests/alsa: Check that a write does not silently change other
> controls

Applied both to for-next branch. Thanks.


Takashi