All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@linux.intel.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH 5/6] Add support for a USB audio device model
Date: Wed, 27 Oct 2010 10:13:09 -0700	[thread overview]
Message-ID: <4CC85DA5.9090704@linux.intel.com> (raw)
In-Reply-To: <1288195475-3807-6-git-send-email-kraxel@redhat.com>

On 10/27/2010 9:04 AM, Gerd Hoffmann wrote:
> From: H. Peter Anvin<hpa@linux.intel.com>
>
> This brings a usb audio device to qemu.  Output only, fixed at
> 16bit stereo @ 480000 Hz.  Based on a patch from
> H. Peter Anvin<hpa@linux.intel.com>
>
> Usage: add '-device usb-audio' to your qemu command line.
>
> Works sorta ok on a idle machine.  Known issues:
>
>   * Is *very* sensitive to latencies: when the uhci emulation misses one
>     of the usb frame rate wakeups (1k/sec!) you'll loose/delay data from
>     the audio stream, resulting in dropouts.
>   * Also seems to not play very well with the usb tablet (and/or usb hub,
>     to be investigated).  Best try this as the only device on a usb bus.
>   * Burns quite some CPU due to usb polling.
>
> In short:  It brings the qemu usb emulation to its limits.  Enjoy!
>
> Signed-off-by: Gerd Hoffmann<kraxel@redhat.com>

Please don't apply this.  This patch is outdated and has several glaring 
bugs.  I can send a better patch later (I'm in an all-day meeting with 
insanely bad networking.)

	-hpa

  reply	other threads:[~2010-10-27 17:14 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-27 16:04 [Qemu-devel] [PATCH 0/6] The windows 7 audio support patch series Gerd Hoffmann
2010-10-27 16:04 ` [Qemu-devel] [PATCH 1/6] pa: allow processing buffer pieces Gerd Hoffmann
2010-10-27 16:04 ` [Qemu-devel] [PATCH 2/6] pa: process 1/4 buffer max at once Gerd Hoffmann
2010-10-27 16:04 ` [Qemu-devel] [PATCH 3/6] pa: setup buffer attrs Gerd Hoffmann
2010-10-27 16:04 ` [Qemu-devel] [PATCH 4/6] pa: tweak config Gerd Hoffmann
2010-10-27 16:04 ` [Qemu-devel] [PATCH 5/6] Add support for a USB audio device model Gerd Hoffmann
2010-10-27 17:13   ` H. Peter Anvin [this message]
2010-10-28  8:01     ` [Qemu-devel] " Gerd Hoffmann
2010-10-28 15:23       ` H. Peter Anvin
2010-10-28 16:13         ` Gerd Hoffmann
2010-10-28 15:28       ` H. Peter Anvin
2010-10-28 15:29       ` H. Peter Anvin
2010-10-27 16:04 ` [Qemu-devel] [PATCH 6/6] Add Intel HD Audio support to qemu Gerd Hoffmann
2010-10-27 17:46 ` [Qemu-devel] [PATCH 0/6] The windows 7 audio support patch series malc
2010-10-27 17:51   ` malc
2010-10-28  8:47   ` Gerd Hoffmann
2010-10-28  9:46     ` malc
2010-10-28 10:06       ` malc
2010-10-28 10:06       ` Gerd Hoffmann
2010-10-28 22:21       ` Gerd Hoffmann

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=4CC85DA5.9090704@linux.intel.com \
    --to=hpa@linux.intel.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.