 4c44b4a4c8
			
		
	
	
		4c44b4a4c8
		
	
	
	
	
		
			
			The previous commit refactoring iotests.py:
  commit 66613974468fb6e1609fb3eabf55981b1ee436cf
  Author: Daniel P. Berrange <berrange@redhat.com>
  Date:   Wed Jul 20 14:23:10 2016 +0100
    scripts: refactor the VM class in iotests for reuse
was not properly tested and included a number of broken
bits.
 - The 'event_match' method was not moved into qemu.py
 - The 'self._args' list parameter in QEMUMachine needs
   to be copied otherwise modifications will affect the
   global 'qemu_opts' variable in iotests.py
 - The QEMUQtestMachine class methods had inverted
   parameter order for the super() calls
 - The QEMUQtestMachine class forgot to add
   '-machine accel=qtest'
 - The QEMUQtestMachine class constructor needs to set
   a default 'name' value before using it as it may
   be None
 - The QEMUQtestMachine class constructor needs to use
   named parameters when calling the super constructor
   as it is leaving out some positional parameters.
 - The 'qemu_prog' variable should be a string not a
   list in iotests.py
 - The VM classs constructor needs to use named
   parameters when calling the super constructor
   as it is leaving out some positional parameters.
 - The path to the socket-scm-helper needs to be
   passed into the QEMUMachine class
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Message-id: 1469549767-27249-1-git-send-email-berrange@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
		
	
			
		
			
				
	
	
		
			111 lines
		
	
	
		
			3.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			111 lines
		
	
	
		
			3.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
| # QEMU qtest library
 | |
| #
 | |
| # Copyright (C) 2015 Red Hat Inc.
 | |
| #
 | |
| # Authors:
 | |
| #  Fam Zheng <famz@redhat.com>
 | |
| #
 | |
| # This work is licensed under the terms of the GNU GPL, version 2.  See
 | |
| # the COPYING file in the top-level directory.
 | |
| #
 | |
| # Based on qmp.py.
 | |
| #
 | |
| 
 | |
| import errno
 | |
| import socket
 | |
| import string
 | |
| import os
 | |
| import subprocess
 | |
| import qmp.qmp
 | |
| import qemu
 | |
| 
 | |
| class QEMUQtestProtocol(object):
 | |
|     def __init__(self, address, server=False):
 | |
|         """
 | |
|         Create a QEMUQtestProtocol object.
 | |
| 
 | |
|         @param address: QEMU address, can be either a unix socket path (string)
 | |
|                         or a tuple in the form ( address, port ) for a TCP
 | |
|                         connection
 | |
|         @param server: server mode, listens on the socket (bool)
 | |
|         @raise socket.error on socket connection errors
 | |
|         @note No connection is established, this is done by the connect() or
 | |
|               accept() methods
 | |
|         """
 | |
|         self._address = address
 | |
|         self._sock = self._get_sock()
 | |
|         if server:
 | |
|             self._sock.bind(self._address)
 | |
|             self._sock.listen(1)
 | |
| 
 | |
|     def _get_sock(self):
 | |
|         if isinstance(self._address, tuple):
 | |
|             family = socket.AF_INET
 | |
|         else:
 | |
|             family = socket.AF_UNIX
 | |
|         return socket.socket(family, socket.SOCK_STREAM)
 | |
| 
 | |
|     def connect(self):
 | |
|         """
 | |
|         Connect to the qtest socket.
 | |
| 
 | |
|         @raise socket.error on socket connection errors
 | |
|         """
 | |
|         self._sock.connect(self._address)
 | |
| 
 | |
|     def accept(self):
 | |
|         """
 | |
|         Await connection from QEMU.
 | |
| 
 | |
|         @raise socket.error on socket connection errors
 | |
|         """
 | |
|         self._sock, _ = self._sock.accept()
 | |
| 
 | |
|     def cmd(self, qtest_cmd):
 | |
|         """
 | |
|         Send a qtest command on the wire.
 | |
| 
 | |
|         @param qtest_cmd: qtest command text to be sent
 | |
|         """
 | |
|         self._sock.sendall(qtest_cmd + "\n")
 | |
| 
 | |
|     def close(self):
 | |
|         self._sock.close()
 | |
| 
 | |
|     def settimeout(self, timeout):
 | |
|         self._sock.settimeout(timeout)
 | |
| 
 | |
| 
 | |
| class QEMUQtestMachine(qemu.QEMUMachine):
 | |
|     '''A QEMU VM'''
 | |
| 
 | |
|     def __init__(self, binary, args=[], name=None, test_dir="/var/tmp",
 | |
|                  socket_scm_helper=None):
 | |
|         if name is None:
 | |
|             name = "qemu-%d" % os.getpid()
 | |
|         super(QEMUQtestMachine, self).__init__(binary, args, name=name, test_dir=test_dir,
 | |
|                                                socket_scm_helper=socket_scm_helper)
 | |
|         self._qtest_path = os.path.join(test_dir, name + "-qtest.sock")
 | |
| 
 | |
|     def _base_args(self):
 | |
|         args = super(QEMUQtestMachine, self)._base_args()
 | |
|         args.extend(['-qtest', 'unix:path=' + self._qtest_path,
 | |
|                      '-machine', 'accel=qtest'])
 | |
|         return args
 | |
| 
 | |
|     def _pre_launch(self):
 | |
|         super(QEMUQtestMachine, self)._pre_launch()
 | |
|         self._qtest = QEMUQtestProtocol(self._qtest_path, server=True)
 | |
| 
 | |
|     def _post_launch(self):
 | |
|         super(QEMUQtestMachine, self)._post_launch()
 | |
|         self._qtest.accept()
 | |
| 
 | |
|     def _post_shutdown(self):
 | |
|         super(QEMUQtestMachine, self)._post_shutdown()
 | |
|         self._remove_if_exists(self._qtest_path)
 | |
| 
 | |
|     def qtest(self, cmd):
 | |
|         '''Send a qtest command to guest'''
 | |
|         return self._qtest.cmd(cmd)
 |