[libvirt] [PATCH v2 1/5] domain: Fix unknown flags diagnosis in virDomainGetXMLDesc
John Ferlan
jferlan at redhat.com
Wed Feb 20 01:31:36 UTC 2019
On 2/19/19 6:00 PM, Eric Blake wrote:
> On 2/19/19 2:24 PM, Eric Blake wrote:
>
>>
>>> BTW: For the two callers that pass all 3 maybe it'd be good to change
>>> those to pass VIR_DOMAIN_XML_COMMON_FLAGS. Currently:
>>>
>>> * qemuMigrationCookieXMLFormat
>>> * qemuDomainSaveImageDefineXML
>>
>> Makes sense, although I might split it as a separate patch.
>>
>
> Actually, no, it doesn't make sense. If VIR_DOMAIN_XML_COMMON_FLAGS
> gains a new flag, we don't want these callers to start outputting new
> information merely because they used the convenience macro. (That is,
> the macro is good for virCheckFlags(), to allow a driver to
> automatically support a new public flag by deferring to the common
> domain_conf implementation's support of the new flag; but it is not good
> for internal uses where we are generating specific output without
> regards to flags passed in through the public API).
>
oh right, good thought.
John
More information about the libvir-list
mailing list