diff options
author | Alexander E. Patrakov <patrakov@gmail.com> | 2014-09-14 00:30:20 +0600 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2014-09-13 21:05:24 +0200 |
commit | 6db0fe495ef1db1fd8f0ed791996f834665a926b (patch) | |
tree | 673c33859272472acc211047abbf4cc57007c9df | |
parent | e5e1af83881e4dbe20749a314703db7d0fd091c7 (diff) | |
download | alsa-lib-6db0fe495ef1db1fd8f0ed791996f834665a926b.tar.gz |
pcm: rewindable, forwardable: don't return stale data
The current behavior of snd_pcm_rewindable and snd_pcm_forwardable means
that the returned value is only accurate to one period. Or maybe even
meaningless if period interrupts are off. Fetch the up-to-date position
of the hardware pointer, as that's what is wanted by callers.
Signed-off-by: Alexander E. Patrakov <patrakov@gmail.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
-rw-r--r-- | src/pcm/pcm_dmix.c | 6 | ||||
-rw-r--r-- | src/pcm/pcm_dshare.c | 6 | ||||
-rw-r--r-- | src/pcm/pcm_dsnoop.c | 6 | ||||
-rw-r--r-- | src/pcm/pcm_hw.c | 6 | ||||
-rw-r--r-- | src/pcm/pcm_ioplug.c | 2 | ||||
-rw-r--r-- | src/pcm/pcm_plugin.c | 6 |
6 files changed, 32 insertions, 0 deletions
diff --git a/src/pcm/pcm_dmix.c b/src/pcm/pcm_dmix.c index babde6a1..d9565f94 100644 --- a/src/pcm/pcm_dmix.c +++ b/src/pcm/pcm_dmix.c @@ -661,6 +661,9 @@ static int snd_pcm_dmix_pause(snd_pcm_t *pcm ATTRIBUTE_UNUSED, int enable ATTRIB static snd_pcm_sframes_t snd_pcm_dmix_rewindable(snd_pcm_t *pcm) { + int err = snd_pcm_dmix_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_playback_hw_rewindable(pcm); } @@ -744,6 +747,9 @@ static snd_pcm_sframes_t snd_pcm_dmix_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t f static snd_pcm_sframes_t snd_pcm_dmix_forwardable(snd_pcm_t *pcm) { + int err = snd_pcm_dmix_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_avail(pcm); } diff --git a/src/pcm/pcm_dshare.c b/src/pcm/pcm_dshare.c index 020e6f7d..de0b2426 100644 --- a/src/pcm/pcm_dshare.c +++ b/src/pcm/pcm_dshare.c @@ -412,6 +412,9 @@ static int snd_pcm_dshare_pause(snd_pcm_t *pcm ATTRIBUTE_UNUSED, int enable ATTR static snd_pcm_sframes_t snd_pcm_dshare_rewindable(snd_pcm_t *pcm) { + int err = snd_pcm_dshare_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_playback_hw_rewindable(pcm); } @@ -428,6 +431,9 @@ static snd_pcm_sframes_t snd_pcm_dshare_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t static snd_pcm_sframes_t snd_pcm_dshare_forwardable(snd_pcm_t *pcm) { + int err = snd_pcm_dshare_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_playback_avail(pcm); } diff --git a/src/pcm/pcm_dsnoop.c b/src/pcm/pcm_dsnoop.c index 8333eefd..00cd461b 100644 --- a/src/pcm/pcm_dsnoop.c +++ b/src/pcm/pcm_dsnoop.c @@ -335,6 +335,9 @@ static int snd_pcm_dsnoop_pause(snd_pcm_t *pcm ATTRIBUTE_UNUSED, int enable ATTR static snd_pcm_sframes_t snd_pcm_dsnoop_rewindable(snd_pcm_t *pcm) { + int err = snd_pcm_dsnoop_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_capture_hw_avail(pcm); } @@ -351,6 +354,9 @@ static snd_pcm_sframes_t snd_pcm_dsnoop_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t static snd_pcm_sframes_t snd_pcm_dsnoop_forwardable(snd_pcm_t *pcm) { + int err = snd_pcm_dsnoop_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_capture_avail(pcm); } diff --git a/src/pcm/pcm_hw.c b/src/pcm/pcm_hw.c index c34b766e..4a527031 100644 --- a/src/pcm/pcm_hw.c +++ b/src/pcm/pcm_hw.c @@ -659,6 +659,9 @@ static int snd_pcm_hw_pause(snd_pcm_t *pcm, int enable) static snd_pcm_sframes_t snd_pcm_hw_rewindable(snd_pcm_t *pcm) { + int err = snd_pcm_hw_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_hw_rewindable(pcm); } @@ -679,6 +682,9 @@ static snd_pcm_sframes_t snd_pcm_hw_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t fra static snd_pcm_sframes_t snd_pcm_hw_forwardable(snd_pcm_t *pcm) { + int err = snd_pcm_hw_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_avail(pcm); } diff --git a/src/pcm/pcm_ioplug.c b/src/pcm/pcm_ioplug.c index fe9347c8..3861bc2e 100644 --- a/src/pcm/pcm_ioplug.c +++ b/src/pcm/pcm_ioplug.c @@ -503,6 +503,7 @@ static int snd_pcm_ioplug_pause(snd_pcm_t *pcm, int enable) static snd_pcm_sframes_t snd_pcm_ioplug_rewindable(snd_pcm_t *pcm) { + snd_pcm_ioplug_hw_ptr_update(pcm); return snd_pcm_mmap_hw_rewindable(pcm); } @@ -514,6 +515,7 @@ static snd_pcm_sframes_t snd_pcm_ioplug_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t static snd_pcm_sframes_t snd_pcm_ioplug_forwardable(snd_pcm_t *pcm) { + snd_pcm_ioplug_hw_ptr_update(pcm); return snd_pcm_mmap_avail(pcm); } diff --git a/src/pcm/pcm_plugin.c b/src/pcm/pcm_plugin.c index c19e2f17..57a1953f 100644 --- a/src/pcm/pcm_plugin.c +++ b/src/pcm/pcm_plugin.c @@ -198,6 +198,9 @@ static int snd_pcm_plugin_reset(snd_pcm_t *pcm) static snd_pcm_sframes_t snd_pcm_plugin_rewindable(snd_pcm_t *pcm) { + int err = snd_pcm_generic_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_hw_rewindable(pcm); } @@ -226,6 +229,9 @@ snd_pcm_sframes_t snd_pcm_plugin_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t frames static snd_pcm_sframes_t snd_pcm_plugin_forwardable(snd_pcm_t *pcm) { + int err = snd_pcm_generic_hwsync(pcm); + if (err < 0) + return err; return snd_pcm_mmap_avail(pcm); } |