[libvirt] [PATCH v3 07/11] qemu: remove liveness check from qemuDomainObjExitMonitor

Nikolay Shirokovskiy nshirokovskiy at virtuozzo.com
Thu Nov 9 13:18:53 UTC 2017


This check is useless now at least. We enter/exit monitor in
the context of some job. Domain can become inactive only in the context
of another job which calls qemuProcessStop. But we do not allow
concurrency during calling qemu commands anywhere thus domain
can not become inactive.
---
 src/qemu/qemu_domain.c | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 782bd43..3747b25 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -4615,22 +4615,11 @@ void qemuDomainObjEnterMonitor(virQEMUDriverPtr driver,
  *
  * Should be paired with an earlier qemuDomainObjEnterMonitor() call
  *
- * Returns -1 if the domain is no longer alive after exiting the monitor.
- * In that case, the caller should be careful when using obj's data,
- * e.g. the live definition in vm->def has been freed by qemuProcessStop
- * and replaced by the persistent definition, so pointers stolen
- * from the live definition could no longer be valid.
  */
 int qemuDomainObjExitMonitor(virQEMUDriverPtr driver,
                              virDomainObjPtr obj)
 {
     qemuDomainObjExitMonitorInternal(driver, obj);
-    if (!virDomainObjIsActive(obj)) {
-        if (!virGetLastError())
-            virReportError(VIR_ERR_OPERATION_FAILED, "%s",
-                           _("domain is no longer running"));
-        return -1;
-    }
     return 0;
 }
 
-- 
1.8.3.1




More information about the libvir-list mailing list