Re: [PATCHv6 2/3] libvirt/qemu - support persistent modification of qemu disks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]