ALSA: emu10k1: Fix deadlock in synth voice lookup
authorTakashi Iwai <tiwai@suse.de>
Mon, 13 Oct 2014 21:18:02 +0000 (23:18 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 Oct 2014 16:38:24 +0000 (09:38 -0700)
commit725a14505cd1a93a1a75cb971b119a2931730d18
treebf3c9a681ab3c32cb16e45d045e8dc9cfba7e795
parent4f5de9214861a47c7b295f41d2af2783455101d2
ALSA: emu10k1: Fix deadlock in synth voice lookup

commit 95926035b187cc9fee6fb61385b7da9c28123f74 upstream.

The emu10k1 voice allocator takes voice_lock spinlock.  When there is
no empty stream available, it tries to release a voice used by synth,
and calls get_synth_voice.  The callback function,
snd_emu10k1_synth_get_voice(), however, also takes the voice_lock,
thus it deadlocks.

The fix is simply removing the voice_lock holds in
snd_emu10k1_synth_get_voice(), as this is always called in the
spinlock context.

Reported-and-tested-by: Arthur Marsh <arthur.marsh@internode.on.net>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/pci/emu10k1/emu10k1_callback.c