[pulseaudio-discuss] [PATCH] Change the default fragment size to 5 ms

[Pierre-Louis Bossart]

On 10/22/14, 11:21 AM, Alexander E. Patrakov wrote:
> Of course, this only applies to ALSA devices that can't use tsched, as
> well as to OSS on more exotic platforms.
>
> This is needed for compatibility with Wine games that use DirectSound8
> on distributions such as Arch Linux who don't accept the unofficial
> winepulse patch. The reason is that Wine DirectSound8 emulation requests
> a period of 10 ms and expects it to work for the purposes of timing. And
> in fact, it cannot know (via ALSA API) that it is not going to work!
>
> A known malware-free test application is Foobar2000 version 1.2. Later
> versions no longer use DirectSound.
>
> This issue is not going to be fixed on the Wine side, because the audio
> subsystem in Wine is not really maintained, and the maintainer has
> already privately expressed some negative thoughts about the state of
> the linux audio ecosystem in general.
>
> The correct solution (at least from the "isolate the clients from
> hardware limitations as thoroughly as possible" viewpoint) would be to
> consume audio from sink inputs using a timer if the actual sink does not
> provide sufficiently small minreq. In other words, adapt tsched to BATCH
> cards and drop non-tsched. But that's a whole project. So, as a stopgap
> solution for running Wine games on e.g. USB audio (which is a topic that
> pops up regularly on #pulseaudio IRC channel), let's reduce the default
> fragment size to a value suitable for unpatched Wine.
>
> Regarding the value: on my system, for mp3 playback through Foobar2000,
> 8ms fragments work, 9ms fragments result in constant underruns.
>
> Wine wiki recommends 5ms.

Maybe it should be made clearer that the actual periods may differ 
slightly from the default settings due to hardware constraints or 
rounding issues, e.g. with your 5ms setting at 44100 Hz the period of 
220.5 samples is going to be rounded....

>
> On devices that can use tsched, there is no problem.
>
> Signed-off-by: Alexander E. Patrakov <patrakov at gmail.com>
> ---
>   man/pulse-daemon.conf.5.xml.in | 4 ++--
>   src/daemon/daemon-conf.c       | 2 +-
>   src/daemon/daemon.conf.in      | 2 +-
>   3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
> index 8bd076d..ff54435 100644
> --- a/man/pulse-daemon.conf.5.xml.in
> +++ b/man/pulse-daemon.conf.5.xml.in
> @@ -463,8 +463,8 @@ USA.
>       </option>
>       <option>
>         <p><opt>default-fragment-size-msec=</opt>The duration of a
> -      single fragment. Defaults to 25ms (i.e. the total buffer is thus
> -      100ms long).</p>
> +      single fragment. Defaults to 5ms (i.e. the total buffer is thus
> +      20ms long).</p>
>       </option>
>
>     </section>
> diff --git a/src/daemon/daemon-conf.c b/src/daemon/daemon-conf.c
> index b7a85aa..6dc15cf 100644
> --- a/src/daemon/daemon-conf.c
> +++ b/src/daemon/daemon-conf.c
> @@ -96,7 +96,7 @@ static const pa_daemon_conf default_conf = {
>       .lock_memory = false,
>       .deferred_volume = true,
>       .default_n_fragments = 4,
> -    .default_fragment_size_msec = 25,
> +    .default_fragment_size_msec = 5,

I've never understood why 4 fragments would be needed. Use 2 fragments 
and 10ms periods...

>       .deferred_volume_safety_margin_usec = 8000,
>       .deferred_volume_extra_delay_usec = 0,
>       .default_sample_spec = { .format = PA_SAMPLE_S16NE, .rate = 44100, .channels = 2 },
> diff --git a/src/daemon/daemon.conf.in b/src/daemon/daemon.conf.in
> index 5b20130..3d758cd 100644
> --- a/src/daemon/daemon.conf.in
> +++ b/src/daemon/daemon.conf.in
> @@ -85,7 +85,7 @@ ifelse(@HAVE_SYS_RESOURCE_H@, 1, [dnl
>   ; default-channel-map = front-left,front-right
>
>   ; default-fragments = 4
> -; default-fragment-size-msec = 25
> +; default-fragment-size-msec = 5
>
>   ; enable-deferred-volume = yes
>   ; deferred-volume-safety-margin-usec = 8000
>