On 03/15/2011 06:19 PM, Chris Wright wrote: > * Eric Blake (ebl...@redhat.com) wrote: >> @@ -829,21 +829,19 @@ int qemuDomainAttachHostPciDevice(struct qemud_driver >> *driver, >> if (qemuDomainPCIAddressEnsureAddr(priv->pciaddrs, &hostdev->info) >> < 0) >> goto error; >> if (qemuCapsGet(qemuCaps, QEMU_CAPS_PCI_CONFIGFD)) { >> - configfd = qemuOpenPCIConfig(hostdev); >> + if (priv->monConfig->type != VIR_DOMAIN_CHR_TYPE_UNIX) { >> + qemuReportError(VIR_ERR_CONFIG_UNSUPPORTED, >> + _("pci configfd file cannot be attached: " >> + "qemu is not using a unix socket >> monitor")); > > Should that be in common sendfd code?
Yeah, factoring that into a common location _would_ be nicer. I'll whip up a patch. -- Eric Blake ebl...@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list