ALSA: dice: fix hang when unplugging a running device
authorClemens Ladisch <clemens@ladisch.de>
Sun, 4 Sep 2011 20:13:09 +0000 (22:13 +0200)
committerClemens Ladisch <clemens@ladisch.de>
Sun, 20 Oct 2013 20:07:57 +0000 (22:07 +0200)
When aborting a PCM stream, the xrun is signaled only if the stream is
running.  When disconnecting a PCM stream, calling snd_card_disconnect()
too early would change the stream into a non-running state and thus
prevent the xrun from being noticed by user space.

To prevent this, move the snd_card_disconnect() call after the xrun.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
sound/firewire/dice.c

index 63446f86cdd78459d4ef17d4d93e6e41ef57f427..d0575a96ea70c636912fa5d3d0201b79ac0a9a48 100644 (file)
@@ -1000,12 +1000,15 @@ static void dice_remove(struct fw_unit *unit)
 {
        struct dice *dice = dev_get_drvdata(&unit->device);
 
-       snd_card_disconnect(dice->card);
-
        mutex_lock(&dice->mutex);
+
        amdtp_out_stream_pcm_abort(&dice->stream);
+
+       snd_card_disconnect(dice->card);
+
        dice_stream_stop(dice);
        dice_owner_clear(dice);
+
        mutex_unlock(&dice->mutex);
 
        snd_card_free_when_closed(dice->card);