[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/5] add usb redirection filter support



On Sun, Aug 19, 2012 at 11:42:43PM +0800, Guannan Ren wrote:
> 
> BZ RFE https://bugzilla.redhat.com/show_bug.cgi?id=795929
> qemu support:
> http://git.qemu.org/?p=qemu.git;a=commitdiff;h=6af165892cf900291046f1d25f95416f379504c2
> 
> Since qemu has have the code to support USB redirection filter. This set of
> patches try to support it from libvirt.
> The XML format is like this:
>  <devices>
>    ...
>    <redirdev bus='usb' type='spicevmc'>
>      <address type='usb' bus='0' port='4'/>
>    </redirdev>
>    <redirfilter>
>      <usbdev class='0x08' vendor='0x1234' product='0xbeef' \
>              version='2.00' allow='yes'/>
>      <usbdev class='-1' vendor='-1' product='-1' version='-1' allow='no'/>
>    </redirfilter>
>    ...
>  </devices>

I don't really see the point in this being done on the libvirt side.
The <redirdev> code is allowing redirection of a USB device from a
client (eg SPICE) app, to the remote QEMU instance and from there to
the guest.

With this architecture, IMHO filtering of USB devices is something
that belongs in the client app, not in QEMU which is a broker inbetween
the client & guest OS.

So NACK to this whole series


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]