[libvirt] [PATCH v2] qemu: fix return value issue in qemuDomainSetMemoryParameters
ajia
ajia at redhat.com
Mon Aug 1 15:34:03 UTC 2011
On 08/01/2011 11:23 PM, Eric Blake wrote:
> On 08/01/2011 09:06 AM, Alex Jia wrote:
>> whether or not previous return value is -1, the following codes will be
>> executed for a inactive guest in qemuDomainSetMemoryParameters:
>> ret = virDomainSaveConfig(driver->configDir, persistentDef);
>> and if everything is okay, 'ret' is assigned to 0, the previous 'ret'
>> will be overwritten, this patch will fix this issue.
>>
>> * src/qemu/qemu_driver.c: avoid return value is overwritten when set
>> min_guarante value to a inactive guest.
>>
>> * how to reproduce?
>> % virsh memtune ${guestname} --min_guarante 1024
>> % echo $?
>>
>> Note: guest must be inactive, in fact, 'min_guarante' hasn't been implemented
>> in memory tunable, and I can get the error when check actual libvirtd.log,
>> however, virsh hasn't raised any error information, and return value is 0.
>>
>> Signed-off-by: Alex Jia<ajia at redhat.com>
>> ---
>> src/qemu/qemu_driver.c | 3 ++-
>> 1 files changed, 2 insertions(+), 1 deletions(-)
> ACK and pushed with one nit fixed:
>
>> if (flags& VIR_DOMAIN_AFFECT_CONFIG) {
>> - ret = virDomainSaveConfig(driver->configDir, persistentDef);
>> + if(virDomainSaveConfig(driver->configDir, persistentDef)< 0)
> space between 'if' and '('.
>
Eric, thanks.
Alex
More information about the libvir-list
mailing list