Re: [libvirt] [PATCH] qemu: qxl devices don't support multifunction yet

["Daniel P. Berrange" <berrange redhat com>]

On Mon, Sep 19, 2011 at 07:16:22PM +0200, Marc-André Lureau wrote:
> Hi hi
> 
> On Fri, Sep 16, 2011 at 1:38 PM, Marc-André Lureau <mlureau redhat com> wrote:
> >> >
> >> > How do we allow other devices to share the slot? It seems to me that
> >> > qemuDomainPCIAddressSetNextAddr() only allocate whole slot, while
> >> > making sure there is no conflicts on the same slot.
> >>
> >> So, if the user wants to use multi function pci device, he should
> >> specify the
> >> pci address.
> >
> > So adding a check such as:
> >
> > if (!multiFunc && info->addr.pci.function != 0)
> >   return error("The %s device doesn't support multifunction address")
> >
> 
> Wen, does that sound reasonable to you?
> 
> Daniel, did you had time to verify that PCI allocation is per-slot?
> 
> (It would be nice to get this "workaround" for the next release)

IMHO this kind of hack doesn't belong in libvirt. It is fine for distro
vendors to consider as a one off quick-hack for their packages of libvirt,
if they don't have time to fix the real QXL bug, but not for libvirt
upstream releases. QXL/QEMU should really be fixed since that's where the
problem appears to lie.

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 :|