[alsa-devel] [PATCH] ad1848-lib: remove open_mutex

Takashi Iwai tiwai at suse.de
Wed Sep 19 21:53:07 CEST 2007


At Wed, 19 Sep 2007 18:06:31 +0200,
Krzysztof Helt wrote:
> 
> From: Krzysztof Helt <krzysztof.h1 at wp.pl>
> 
> This patch removes open_mutex from the ad1848-lib as
> open and close operations are called only from pcm layer
> and mutexed there with pcm->open_mutex.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1 at wp.pl>
> 
> ---
> 
> Another step into crusade against the ad1848-lib. Inspired by
> Trent's comments about missing locking I have checked and
> open and close operations are already mutexed in the pcm layer
> by similarly called pcm->open_mutex. The OSS pcm layer
> also uses the pcm->open_mutex.
> 
> Is it right thing to do? (Takashi, Jaroslav)?

Yes, it looks fine.
But, I'd like to postpone after 1.0.15.  OK?


Thanks,

Takashi


More information about the Alsa-devel mailing list