From: Gerd Hoffmann <kraxel@redhat.com>
To: Programmingkid <programmingkidx@gmail.com>
Cc: Stefan Hajnoczi <stefanha@gmail.com>,
"H. Peter Anvin" <hpa@linux.intel.com>,
qemu-devel qemu-devel <qemu-devel@nongnu.org>,
spice-devel <spice-devel@freedesktop.org>
Subject: Re: [Qemu-devel] [PATCH] hw/usb/dev-audio.c: make USB audio card sound perfect
Date: Fri, 30 Oct 2015 11:59:29 +0100 [thread overview]
Message-ID: <1446202769.1144.25.camel@redhat.com> (raw)
In-Reply-To: <4D46C0B4-B015-4C55-8126-CCF80999C0ED@gmail.com>
Hi,
[ context for spice folks: patch was added to qemu increasing usb-audio
default buffer size ]
> > What bothers me is that you have no qualms about making latency on
> > everyone's system worse.
>
> How do you know it makes sound on other people's systems worse? If you have
> actually done any testing, I would like to see the results.
It's real. With that change we *do* actually trade latency for better
sound quality.
You probably wouldn't notice with pure music playback.
Higher chance is with video playback. lip sync issues might show up,
although you probably still have to watch carefully to actually notice.
Anything sending audio both ways and expecting it to run with low
latency (VoIP phone, music jam as mentioned by stefan) is affected even
more.
And we *do* actually just paper over the root cause. Problem is the
real root cause can is very hard to track down. It can be pretty much
anywhere in qemu, and even outside qemu.
One known issue actually is in spice-server (added spice-devel because
of that). It does audio processing in the qemu iothread (instead of a
separate thread like it is done for the display channel). If you turn
off audio compression in spice sound quality suddenly becomes better. I
think this this happens because the latency spikes caused by audio
compression go away. Happens with intel-hda and windows guests (which
use very small audio buffers). Not (yet) investigated in detail though.
cheers,
Gerd
next prev parent reply other threads:[~2015-10-30 10:59 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-22 23:32 [Qemu-devel] [PATCH] hw/usb/dev-audio.c: make USB audio card sound perfect Programmingkid
2015-10-16 12:15 ` Stefan Hajnoczi
2015-10-16 13:54 ` Programmingkid
2015-10-16 14:00 ` Peter Maydell
2015-10-16 14:06 ` Programmingkid
2015-10-26 11:00 ` Stefan Hajnoczi
2015-10-26 12:24 ` Gerd Hoffmann
2015-10-26 16:05 ` Programmingkid
2015-10-28 10:58 ` Stefan Hajnoczi
2015-10-28 12:07 ` Gerd Hoffmann
2015-10-28 13:59 ` Programmingkid
2015-10-29 15:08 ` Stefan Hajnoczi
2015-10-29 16:02 ` Programmingkid
2015-10-30 10:59 ` Gerd Hoffmann [this message]
2015-11-02 13:07 ` Stefan Hajnoczi
2015-11-02 13:49 ` Gerd Hoffmann
2015-10-26 15:16 ` Programmingkid
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1446202769.1144.25.camel@redhat.com \
--to=kraxel@redhat.com \
--cc=hpa@linux.intel.com \
--cc=programmingkidx@gmail.com \
--cc=qemu-devel@nongnu.org \
--cc=spice-devel@freedesktop.org \
--cc=stefanha@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).