On 03/15/2011 06:19 PM, Chris Wright wrote: > * Eric Blake (eblake@xxxxxxxxxx) 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 eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list