[libvirt] [PATCH v3 2/5] Make address assignment in qemuDomainValidateDevicePCISlotsChipsets optional

Martin Kletzander mkletzan at redhat.com
Tue Aug 16 11:26:03 UTC 2016

On Sun, Aug 14, 2016 at 04:14:54AM +0200, Tomasz Flendrich wrote:
>With a parameter, we can decide whether we want
>qemuDomainValidateDevicePCISlotsChipsets to assign new addresses, or
>simply verify the existing ones and reserve slots for potential future
>The PCI address set will be recalculated instead of being cached, and
>during recalculation we want to forbid assignment of new addresses.

It looks like the new parameter just errors out if address that needs to
be assigned is not...  It might seem better to just assign the address
then.  It is explicitly saying that the part of the code *should not*
get executed.

If that helps in future patches, I'm not totally against it, but add
more information in the error message so that in case the error occurs,
we know where it came from.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160816/3255c4ff/attachment-0001.sig>

More information about the libvir-list mailing list