[pulseaudio-discuss] [PATCH] protocol-native: fix bytes-to-usec conversion of "on-the-fly" data
Arun Raghavan
arun at arunraghavan.net
Sat Jul 15 05:50:36 UTC 2017
On Tue, 11 Jul 2017, at 11:46 PM, Tanu Kaskinen wrote:
> The on_the_fly_snapshot variable contains the amount of bytes that has
> been sent from the source IO thread to the main thread, but not yet
> pushed to the stream memblockq. The data is in the stream format, but
> the bytes-to-usec conversion used the source format, which caused random
> latency reporting errors.
>
> BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=81075
> ---
> src/pulsecore/protocol-native.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/pulsecore/protocol-native.c
> b/src/pulsecore/protocol-native.c
> index 266b676de..d5d032950 100644
> --- a/src/pulsecore/protocol-native.c
> +++ b/src/pulsecore/protocol-native.c
> @@ -2922,7 +2922,7 @@ static void command_get_record_latency(pa_pdispatch
> *pd, uint32_t command, uint3
> pa_tagstruct_put_usec(reply, s->current_monitor_latency);
> pa_tagstruct_put_usec(reply,
> s->current_source_latency +
> - pa_bytes_to_usec(s->on_the_fly_snapshot,
> &s->source_output->source->sample_spec));
> + pa_bytes_to_usec(s->on_the_fly_snapshot,
> &s->source_output->sample_spec));
> pa_tagstruct_put_boolean(reply,
> pa_source_get_state(s->source_output->source)
> == PA_SOURCE_RUNNING &&
> pa_source_output_get_state(s->source_output)
> == PA_SOURCE_OUTPUT_RUNNING);
> --
Looks good. Let's get this into 11.0, maybe?
-- Arun
More information about the pulseaudio-discuss
mailing list