[libvirt] [PATCH] qemu: simplify PCI configfd handling in monitor

Eric Blake eblake at redhat.com
Wed Mar 16 00:44:36 UTC 2011


On 03/15/2011 06:19 PM, Chris Wright wrote:
> * Eric Blake (eblake at 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   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110315/4f41e8b9/attachment-0001.sig>


More information about the libvir-list mailing list