[libvirt] [PATCH] qemu: Fix warnings in event handlers
Cole Robinson
crobinso at redhat.com
Wed Mar 9 13:53:26 UTC 2011
On 03/09/2011 08:20 AM, Jiri Denemark wrote:
> Some qemu monitor event handlers were issuing inadequate warning when
> virDomainSaveStatus() failed. They copied the message from I/O error
> handler without customizing it to provide better information on why
> virDomainSaveStatus() was called.
> ---
> src/qemu/qemu_process.c | 15 ++++++++++-----
> 1 files changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index ee1b4c4..4414a7c 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -339,15 +339,18 @@ qemuProcessHandleStop(qemuMonitorPtr mon ATTRIBUTE_UNUSED,
>
> virDomainObjLock(vm);
> if (vm->state == VIR_DOMAIN_RUNNING) {
> - VIR_DEBUG("Transitioned guest %s to paused state due to unknown event", vm->def->name);
> + VIR_DEBUG("Transitioned guest %s to paused state due to unknown event",
> + vm->def->name);
>
> vm->state = VIR_DOMAIN_PAUSED;
> event = virDomainEventNewFromObj(vm,
> VIR_DOMAIN_EVENT_SUSPENDED,
> VIR_DOMAIN_EVENT_SUSPENDED_PAUSED);
>
> - if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0)
> - VIR_WARN("Unable to save status on vm %s after IO error", vm->def->name);
> + if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0) {
> + VIR_WARN("Unable to save status on vm %s after state change",
> + vm->def->name);
> + }
> }
> virDomainObjUnlock(vm);
>
> @@ -412,8 +415,10 @@ qemuProcessHandleWatchdog(qemuMonitorPtr mon ATTRIBUTE_UNUSED,
> VIR_DOMAIN_EVENT_SUSPENDED,
> VIR_DOMAIN_EVENT_SUSPENDED_WATCHDOG);
>
> - if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0)
> - VIR_WARN("Unable to save status on vm %s after IO error", vm->def->name);
> + if (virDomainSaveStatus(driver->caps, driver->stateDir, vm) < 0) {
> + VIR_WARN("Unable to save status on vm %s after watchdog event",
> + vm->def->name);
> + }
> }
>
> if (vm->def->watchdog->action == VIR_DOMAIN_WATCHDOG_ACTION_DUMP) {
ACK
- Cole
More information about the libvir-list
mailing list