Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 21 Oct 2022 08:18:47 +0200
Message-ID: <8735bh8zso.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     "Maciej S. Szmigiero" <mail@maciej.szmigiero.name>
Cc:     Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/6] Fix direct renaming of hashed controls
In-Reply-To: <cover.1666296963.git.maciej.szmigiero@oracle.com>
References: <cover.1666296963.git.maciej.szmigiero@oracle.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=US-ASCII
Precedence: bulk

On Thu, 20 Oct 2022 22:46:20 +0200,
Maciej S. Szmigiero wrote:
> 
> From: "Maciej S. Szmigiero" <maciej.szmigiero@oracle.com>
> 
> I've noticed that some of mixer controls on my sound card seem to
> be partially broken on the 6.0 kernel - alsactl wasn't able to find them
> when restoring the mixer state.
> 
> The issue was traced down to the recent addition of hashed controls lookup
> in commit c27e1efb61c5 ("ALSA: control: Use xarray for faster lookups").
> 
> Since that commit it is *not* enough to just directly update the control
> name field (like some of ALSA drivers were doing).
> Now the hash entries for the modified control have to be updated too.
> 
> This patch set adds a snd_ctl_rename() function that takes care of doing
> this operation properly for callers that already have the relevant
> struct snd_kcontrol at hand and hold the control write lock (or simply
> haven't registered the card yet).
> 
> These prerequisites hold true for all the call sites modified.
>     
> The core controls change and the emu10k1 patch were runtime tested.
> Similar patches for other devices were only compile tested.

Good catch!
Applied all patches now.


thanks,

Takashi