On Thu, 24 Mar 2011 16:49:29 +0800 Wen Congyang <wency@xxxxxxxxxxxxxx> wrote: > > Sometimes, qemuDomainAssignPCIAddress() failed, and it does not report error. > > qemuDomainAssignPCIAddress() calls qemuDomainPCIAddressSetCreate() > that calls virDomainDeviceInfoIterate() > that calls qemuCollectPCIAddress() > that calls virHashAddEntry() > that calls virHashAddOrUpdateEntry() > > When the pci address of two drivers are the same, virHashAddOrUpdateEntry() will fail. > if (found) { > if (is_update) { > if (table->dataFree) > table->dataFree(insert->payload, insert->name); > insert->payload = userdata; > return (0); > } else { > return (-1); <==== we do not report error here. > } > } Then, do you have recomendation ? Just retuns error without any hint is bad behavior. Thanks, -Kame -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list