Re: Master Plan on rewinding

[David Henningsson <david.henningsson@canonical.com>]

On 2014-09-07 17:16, Alexander E. Patrakov wrote:
> Hello.
>
> (TL;DR: nothing really new except the strawman proposal about threads
> and the note about interaction of variable sample rate with rewindability)
>
> As Takashi Iwai told me that the audio miniconference is for discussion
> only, and not for presentation of anything, I guess that I need to
> present the plans and options now. That's why this e-mail. The goal
> here, besides merely presenting the plan, is to identify points that
> everyone agrees upon, so that they are not discussed pointlessly at the
> miniconference. Also, this e-mail serves as a justification for the
> pending seemingly-destructive work.
>
> First, the status quo. If anyone disagrees with the facts below, please
> complain loudly, before I make any conclusions!
>
> 1. PulseAudio does not call snd_pcm_rewindable(), because for some ALSA
> plugins it crashed. This crash is completely fixed in alsa-lib 1.0.28,
> but in some cases snd_pcm_rewindable() still returns wrong results.

I don't know if we ever tried snd_pcm_rewindable() in PulseAudio.

> 2. PulseAudio blindly assumes that it can rewind up to hwbuf_frames -
> (snd_pcm_avail() + rewind_safeguard) frames. The rewind safeguard is
> needed due to reasons that I don't completely understand, but one of
> them is imprecise reporting of the hardware pointer, and another one is
> that the hardware transfers several bytes at a time, and the bytes we
> need to overwrite may be already cached by the hardware.

Pierre added the rewind safeguard due to DMA controller problems. IIRC, 
some DMA controllers go nuts (such as breaking the stream, causing 
interrupt storms, or something else seriously buggy) when trying to 
write to data that the DMA controller is just about to transfer. Pierre 
(now cc:ed) would know more about this than I do, though.

In my world, since this is a very hardware near problem, ALSA rather 
than PulseAudio should take these kinds of problems into account when 
reporting back snd_pcm_rewindable() so PulseAudio does not have to.

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic