directsoundsrc: Use latency-time and buffer-time settings
authorNirbheek Chauhan <nirbheek@centricular.com>
Wed, 3 May 2017 17:20:27 +0000 (22:50 +0530)
committerSebastian Dröge <sebastian@centricular.com>
Mon, 8 May 2017 14:13:20 +0000 (16:13 +0200)
commit1a8610f8f8305f20a235aa4fe78ab3fb0c785df4
tree83bda0b8a7cf9a11565cb05f4494701b33a09775
parent6f38a0c6e1e011d829d0230b13f69e48fb0a9c56
directsoundsrc: Use latency-time and buffer-time settings

Earlier, the plugin was ignoring those settings and blindly setting
buffer-time to 2 seconds and latency-time to 200ms, which forced all
pipelines to have a minimum latency of 200ms + sink latency.

The values of segsize and segtotal were also not derived correctly.
Now we obey these values, and you can get close to the previous
behaviour by setting buffer-time and latency-time manually. Note that
they are set in microseconds.

As a consequence, when we haven't received enough data from the
device, we now sleep for a time proportional to the data remaining.
However, Directsound is a deprecated API so it maintains its own
software ringbuffer which updates at arbitrary intervals. Hence we
might have to wait a full segsize to get the last 10% of data. To
avoid tight loops, we clamp our sleep floor at 10ms.

In my testing, this keeps the wakeups not-too-high (proportional to
the latency-time set on the source). Further improvements should be
made by fixing the WASAPI audio source plugin instead of this.
Directsound is deprecated and as the comments explain, it is
impossible to get low latency, decent quality, or good performance
from it.

Based on a patch by Sebastian Dröge <sebastian@centricular.com>

https://bugzilla.gnome.org/show_bug.cgi?id=781249
sys/directsound/gstdirectsoundsrc.c
sys/directsound/gstdirectsoundsrc.h