]> git.alsa-project.org Git - alsa-lib.git/commitdiff
pcm: Avoid lock for snd_pcm_nonblock()
authorTakashi Iwai <tiwai@suse.de>
Thu, 9 Feb 2017 16:23:22 +0000 (17:23 +0100)
committerTakashi Iwai <tiwai@suse.de>
Thu, 9 Feb 2017 16:26:52 +0000 (17:26 +0100)
snd_pcm_nonblock() is called as snd_pcm_abort().  Since
snd_pcm_abort() is called often from a signal handler to clean things
up (e.g. aplay does it), we may face a deadlock if the signal is
raised during the locked operation.

There can be some way to check the deadlock state, but they would cost
much.  Since the race condition of snd_pcm_nonblock() is quite small,
let's just drop the locking inside snd_pcm_nonblock() as a
workaround.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
src/pcm/pcm.c

index 0cf740f854a951092f408fbb302eed83b8d2f634..c136d55793207c41c29a559eebb25811e1762e28 100644 (file)
@@ -759,7 +759,10 @@ int snd_pcm_nonblock(snd_pcm_t *pcm, int nonblock)
        int err = 0;
 
        assert(pcm);
-       __snd_pcm_lock(pcm); /* forced lock due to pcm field change */
+       /* FIXME: __snd_pcm_lock() call below is commented out because of the
+        * the possible deadlock in signal handler calling snd_pcm_abort()
+        */
+       /* __snd_pcm_lock(pcm); */ /* forced lock due to pcm field change */
        if ((err = pcm->ops->nonblock(pcm->op_arg, nonblock)) < 0)
                goto unlock;
        if (nonblock == 2) {
@@ -775,7 +778,7 @@ int snd_pcm_nonblock(snd_pcm_t *pcm, int nonblock)
                        pcm->mode &= ~SND_PCM_NONBLOCK;
        }
  unlock:
-       __snd_pcm_unlock(pcm);
+       /* __snd_pcm_unlock(pcm); */ /* FIXME: see above */
        return err;
 }