[libvirt] [PATCH v2] qemu: fix return value issue in qemuDomainSetMemoryParameters
Eric Blake
eblake at redhat.com
Mon Aug 1 15:23:08 UTC 2011
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 Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
More information about the libvir-list
mailing list