[PATCH v1 0/3] python/qemu: follow-up changes for ConsoleSocket

Robert Foley posted 3 patches 5 years, 4 months ago
Test checkpatch failed
Test docker-mingw@fedora failed
Test FreeBSD failed
Test docker-quick@centos7 failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200717203041.9867-1-robert.foley@linaro.org
Maintainers: "Philippe Mathieu-Daudé" <philmd@redhat.com>, Cleber Rosa <crosa@redhat.com>, Fam Zheng <fam@euphon.net>, Eduardo Habkost <ehabkost@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>
python/qemu/console_socket.py | 137 +++++++++++++++++++---------------
python/qemu/machine.py        |  14 ++--
python/qemu/pylintrc          |   2 +-
tests/vm/basevm.py            |  15 ++--
4 files changed, 94 insertions(+), 74 deletions(-)
[PATCH v1 0/3] python/qemu: follow-up changes for ConsoleSocket
Posted by Robert Foley 5 years, 4 months ago
For v1, we added a few minor changes, and also added one new patch 
in tests/vm to add a shutdown timeout.  This fixes an issue we saw in 
testing the aarch64 VMs with TCG.

This patch series introduces a few follow-up changes after the introduction of 
ConsoleSocket.

The first patch introduces cleanup changes for pylint and flake8.

The second patch allows machine.py to use a single type for the console_socket,
a ConsoleSocket.
Since machine.py will use ConsoleSocket for both the draining and non-draining
cases, we changed ConsoleSocket to handle the case where it does not drain the
socket at all and essentially behaves like a socket.

Robert Foley (3):
  python/qemu: Cleanup changes to ConsoleSocket
  python/qemu: Change ConsoleSocket to optionally drain socket.
  tests/vm: add shutdown timeout in basevm.py

 python/qemu/console_socket.py | 137 +++++++++++++++++++---------------
 python/qemu/machine.py        |  14 ++--
 python/qemu/pylintrc          |   2 +-
 tests/vm/basevm.py            |  15 ++--
 4 files changed, 94 insertions(+), 74 deletions(-)

-- 
2.17.1