[libvirt] [PATCH] qemu: qxl devices don't support multifunction yet
Daniel P. Berrange
berrange at redhat.com
Mon Sep 19 17:32:33 UTC 2011
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 at 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 :|
More information about the libvir-list
mailing list