]> git.alsa-project.org Git - alsa-lib.git/commit
alsa-lib: Make sure SND_PCM_NO_xxx flags don't get lost when nonblocking mode is...
authorLennart Poettering <mznyfn@0pointer.de>
Sun, 31 Aug 2008 01:41:48 +0000 (03:41 +0200)
committerJaroslav Kysela <perex@perex.cz>
Sun, 31 Aug 2008 13:45:18 +0000 (15:45 +0200)
commit73d20069e072e2b62bbc3ea12e46ae19d8b6cac9
tree7791bbaa74394975240e8218d562120865a32970
parent119bfd65947898ad19c8a3ec2f586e73902dbb96
alsa-lib: Make sure SND_PCM_NO_xxx flags don't get lost when nonblocking mode is enabled

The plug PCM copies the 'mode' field from the slave PCM. If blocking mode is
enabled for the plug PCM the mode is subsequently overwritten with the original
requested 'mode'. If non-blocking mode is requested this does not happen.

Because the hw PCM synthesizes the 'mode' from the actual file descriptor flags
no SND_PCM_NO_xxx will ever be set for it. This has the effect that the 'mode'
of the plug PCM will also not include those flags anymore -- unless they are
overwritten as mentioned above. This basically means SND_PCM_NO_xxx is ignored
for plug:hw:4711 style device strings opened in non-blocking mode.

You can easily test this with "aplay --channels 7 --disable-channels -f S16_LE
-r 44100 -D plug:hw:0" on a device that cannot do 7 channels. Normally this
call should fail, however if you add "-N" to the command line this call will
succeed.

This patch simply copies the SND_PCM_NO_xxx flags back into the 'mode' field in
case we don't overwrite it with the original anyway.

Probably closes bug 3571 for good.

From: Lennart Poettering <mznyfn@0pointer.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
src/pcm/pcm_hw.c