aboutsummaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorEduardo Habkost <ehabkost@redhat.com>2017-05-26 15:11:58 -0300
committerEduardo Habkost <ehabkost@redhat.com>2017-06-05 14:59:09 -0300
commit37bbcd575745d9bea4f840164e75dd2f40e5ee91 (patch)
tree5e6e9a44cfd013d8d09fcdd43c030cdfdaffd4d6 /scripts
parent99861ecbc53c705fff5e4ea774f0da1193e81da8 (diff)
downloadqemu-37bbcd575745d9bea4f840164e75dd2f40e5ee91.zip
qemu-37bbcd575745d9bea4f840164e75dd2f40e5ee91.tar.gz
qemu-37bbcd575745d9bea4f840164e75dd2f40e5ee91.tar.bz2
qemu.py: Don't set _popen=None on error/shutdown
Keep the Popen object around to we can query its exit code later. To keep the existing 'self._popen is None' checks working, add a is_running() method, that will check if the process is still running. Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> Message-Id: <20170526181200.17227-2-ehabkost@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Diffstat (limited to 'scripts')
-rw-r--r--scripts/qemu.py12
1 files changed, 7 insertions, 5 deletions
diff --git a/scripts/qemu.py b/scripts/qemu.py
index 6d1b623..16934f1 100644
--- a/scripts/qemu.py
+++ b/scripts/qemu.py
@@ -85,8 +85,11 @@ class QEMUMachine(object):
return
raise
+ def is_running(self):
+ return self._popen and (self._popen.returncode is None)
+
def get_pid(self):
- if not self._popen:
+ if not self.is_running():
return None
return self._popen.pid
@@ -128,16 +131,16 @@ class QEMUMachine(object):
stderr=subprocess.STDOUT, shell=False)
self._post_launch()
except:
- if self._popen:
+ if self.is_running():
self._popen.kill()
+ self._popen.wait()
self._load_io_log()
self._post_shutdown()
- self._popen = None
raise
def shutdown(self):
'''Terminate the VM and clean up'''
- if not self._popen is None:
+ if self.is_running():
try:
self._qmp.cmd('quit')
self._qmp.close()
@@ -149,7 +152,6 @@ class QEMUMachine(object):
sys.stderr.write('qemu received signal %i: %s\n' % (-exitcode, ' '.join(self._args)))
self._load_io_log()
self._post_shutdown()
- self._popen = None
underscore_to_dash = string.maketrans('_', '-')
def qmp(self, cmd, conv_keys=True, **args):