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>
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) {
pcm->mode &= ~SND_PCM_NONBLOCK;
}
unlock:
- __snd_pcm_unlock(pcm);
+ /* __snd_pcm_unlock(pcm); */ /* FIXME: see above */
return err;
}