[libvirt] [PATCH] save domain status after modifing vcpupin

We should save domain status after modifing vcpupin. If not, we will get wrong vcpupin information after rebooting libvirtd. --- src/qemu/qemu_driver.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 8fcc266..768e0f2 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3124,6 +3124,8 @@ qemudDomainPinVcpuFlags(virDomainPtr dom, } } + if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0) + goto cleanup; } if (flags & VIR_DOMAIN_AFFECT_CONFIG) { -- 1.7.1

On 28.06.2011 05:43, Wen Congyang wrote:
We should save domain status after modifing vcpupin. If not, we will get wrong vcpupin information after rebooting libvirtd.
--- src/qemu/qemu_driver.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 8fcc266..768e0f2 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3124,6 +3124,8 @@ qemudDomainPinVcpuFlags(virDomainPtr dom, } }
+ if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0) + goto cleanup; }
if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
ACK. Michal

At 06/28/2011 06:36 PM, Michal Privoznik Write:
On 28.06.2011 05:43, Wen Congyang wrote:
We should save domain status after modifing vcpupin. If not, we will get wrong vcpupin information after rebooting libvirtd.
--- src/qemu/qemu_driver.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 8fcc266..768e0f2 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3124,6 +3124,8 @@ qemudDomainPinVcpuFlags(virDomainPtr dom, } }
+ if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0) + goto cleanup; }
if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
ACK.
Thanks. Pushed.
Michal
participants (2)
-
Michal Privoznik
-
Wen Congyang