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
next prev parent 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).