Commit 115b96e5 authored by Takashi Iwai's avatar Takashi Iwai Committed by Greg Kroah-Hartman

ALSA: usb-audio: Fix regression by disconnection-race-fix patch

[NOTE: the regression below is found only in 3.2-3.4 stable trees, so
       there is no upstream commit corresponding to this patch]

The recent fix for the race at disconnection of usb-audio devices
(upstream commit 978520b7) triggers Oops when a device is unplugged
while playing on 3.2 and 3.4 kernels.  The culprit is that the
shutdown flag check was wrongly added around the urb deactivation code
snippet.  The urb deactivation code has to be performed even after the
device disconnected.  Otherwise it remains undead and pokes the wild
access in the end.

The regression fix is simply reverting the shutdown flag check in that
code.
Reported-and-tested-by: default avatarChris J Arges <christopherarges@gmail.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b7cd50fb
...@@ -148,10 +148,8 @@ void snd_usb_release_substream_urbs(struct snd_usb_substream *subs, int force) ...@@ -148,10 +148,8 @@ void snd_usb_release_substream_urbs(struct snd_usb_substream *subs, int force)
int i; int i;
/* stop urbs (to be sure) */ /* stop urbs (to be sure) */
if (!subs->stream->chip->shutdown) { deactivate_urbs(subs, force, 1);
deactivate_urbs(subs, force, 1); wait_clear_urbs(subs);
wait_clear_urbs(subs);
}
for (i = 0; i < MAX_URBS; i++) for (i = 0; i < MAX_URBS; i++)
release_urb_ctx(&subs->dataurb[i]); release_urb_ctx(&subs->dataurb[i]);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment