[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] make latency configurable
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] make latency configurable |
Date: |
Thu, 14 Mar 2019 09:02:36 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.1 |
On 3/14/19 8:38 AM, Martin Schrodt wrote:
> Signed-off-by: Martin Schrodt <address@hidden>
The subject line says "what" (good) but doesn't state everything the
patch is doing (you are not only making it configurable, but you are
changing the default). The commit body should say "why" (missing), and
mention all the related things being done to satisfy the "what". Some of
that information is already in your cover letter, and merely need to be
repeated here (the fact that the previous default of 10ms is too short,
but that 15ms worked for you).
> ---
> audio/paaudio.c | 18 +++++++-----------
> qapi/audio.json | 5 ++++-
> 2 files changed, 11 insertions(+), 12 deletions(-)
>
> +++ b/qapi/audio.json
> @@ -206,12 +206,15 @@
> #
> # @name: name of the sink/source to use
> #
> +# @latency: latency you want PulseAudio to archieve in microseconds
s/archieve/achieve/
> +# (default 15000)
> # Since: 4.0
> ##
> { 'struct': 'AudiodevPaPerDirectionOptions',
> 'base': 'AudiodevPerDirectionOptions',
> 'data': {
> - '*name': 'str' } }
> + '*name': 'str',
> + '*latency': 'uint32' } }
>
> ##
> # @AudiodevPaOptions:
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH 0/3] Fixes for PulseAudio driver, no-reply, 2019/03/14