diff options
author | Takashi Sakamoto <o-takashi@sakamocchi.jp> | 2019-10-30 00:14:14 +0900 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2019-10-30 13:13:22 +0100 |
commit | a07eb8e235dd01f60bc8b2ea4be783e201e28311 (patch) | |
tree | ab058ef0ca14073ad524d72af92731bcafff5f62 /axfer | |
parent | e5e6a7838b06f884bb36f217c7c439196579da17 (diff) | |
download | alsa-utils-a07eb8e235dd01f60bc8b2ea4be783e201e28311.tar.gz |
axfer: handle -ETIMEDOUT before handle mmap I/O operation
When running for IRQ-based scheduling model, transmission backend is
expected to queue PCM event in different context from the main thread.
When queueing no events during time equivalent to one period of PCM
buffer, the backend has problems.
This commit outputs report it for mmap operation for IRQ-based
scheduling model. The timeout is equivalent to all frames in buffer,
instead of period for safe.
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'axfer')
-rw-r--r-- | axfer/xfer-libasound-irq-mmap.c | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/axfer/xfer-libasound-irq-mmap.c b/axfer/xfer-libasound-irq-mmap.c index 71ee79f..a13b3c3 100644 --- a/axfer/xfer-libasound-irq-mmap.c +++ b/axfer/xfer-libasound-irq-mmap.c @@ -82,10 +82,28 @@ static int irq_mmap_process_frames(struct libasound_state *state, int err; if (state->use_waiter) { + unsigned int msec_per_buffer; unsigned short revents; + // Wait during msec equivalent to all audio data frames in + // buffer instead of period, for safe. + err = snd_pcm_hw_params_get_buffer_time(state->hw_params, + &msec_per_buffer, NULL); + if (err < 0) + return err; + msec_per_buffer /= 1000; + // Wait for hardware IRQ when no avail space in buffer. - err = xfer_libasound_wait_event(state, -1, &revents); + err = xfer_libasound_wait_event(state, msec_per_buffer, + &revents); + if (err == -ETIMEDOUT) { + logging(state, + "No event occurs for PCM substream during %u " + "msec. The implementaion of kernel driver or " + "userland backend causes this issue.\n", + msec_per_buffer); + return err; + } if (err < 0) return err; if (revents & POLLERR) { |