qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "andrzej zaborowski" <balrogg@gmail.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] USB Webcam Redirection
Date: Fri, 5 Oct 2007 01:01:42 +0200	[thread overview]
Message-ID: <fb249edb0710041601x43d64f55r29af71160a8335bf@mail.gmail.com> (raw)
In-Reply-To: <64F9B87B6B770947A9F8391472E032160232DC17@ehost011-8.exch011.intermedia.net>

Hi,

On 24/09/2007, Arnon Gilboa <arnon.gilboa@qumranet.com> wrote:
> Attached is an updated patch for supporting isochronous transfers in
> redirected host USB devices. The initial goal was supporting USB 1.1 Webcam.
> Tested on WinXP guest with several Webcams. Works on USB 1.1 Webcams, as
> well as most USB 2.0 Webcams (backward compatibility) on low resolutions.
> Some jitter is visible in the video stream, and it will be fixed.
>
> Notice USE_ASYNCIO, which defines whether to use signal based async io or
> polling for receiving urbs. Currently it is disabled, so polling is used,
> but it does not seem to affect the performance because it uses the
> non-blocking USBDEVFS_REAPURBNDELAY ioctl. In order to use the signal based
> async io, the patch to usb-uhci.c should be applied.
>
> The patch includes parts of previous patches posted in Qemu-devel:
> usb_host_update_interfaces (from
> qemu-0.9.0-usb-multi-configs.patch),
> usb_linux_update_endp_table (qemu-usb-host-async.patch) as well as some
> other lines of code.
>
> I am currently working on the ehci emulation for fully supporting USB 2.0
> isochronous devices.
>
> Waiting for your comments,

I pushed both patches into CVS, but I haven't tested it with any
devices with ISO endpoints yet, so please do. I made some identifiers
static and also removed the first qemu_free to avoid double freeing
in:

    ret = usb_linux_update_endp_table(dev);
    if (ret) {
        qemu_free(dev);
        goto fail;
    }
...
fail:
    if (dev)
        qemu_free(dev);

Regards

  reply	other threads:[~2007-10-04 23:01 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-24 15:35 [Qemu-devel] [PATCH] USB Webcam Redirection Arnon Gilboa
2007-10-04 23:01 ` andrzej zaborowski [this message]
2007-10-07 14:18   ` [Qemu-devel] [PATCH] usb-linux iso: use pipe instead of bh Arnon Gilboa

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=fb249edb0710041601x43d64f55r29af71160a8335bf@mail.gmail.com \
    --to=balrogg@gmail.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 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).