[libvirt] [PATCH] [3/4] Implement managed save operations for qemu driver
Laine Stump
laine at laine.org
Fri Apr 2 20:48:36 UTC 2010
On 04/02/2010 04:26 PM, Eric Blake wrote:
> On 04/02/2010 01:56 PM, Daniel Veillard wrote:
>
>
>> +static int
>> +qemuDomainManagedSave(virDomainPtr dom,
>> + unsigned int flags ATTRIBUTE_UNUSED)
>>
> No - for future compability, we NEED to check that flags==0 or fail now.
> Otherwise, a future version, where flags has meaning, will mistakenly
> think that our older version can honor those flags.
>
That makes sense, but there's a lot of precedence to the contrary
already in the code:
find . -name \*.c | xargs grep "flags ATTRIBUTE_UNUSED" | wc
155 832 14106
(some of these are probably false positives, but still...)
Sounds like you're volunteering for a global cleanup ;-)
More information about the libvir-list
mailing list