[v3] python/qemu/machine: fix potential hang in QMP accept

[PATCH v3 0/3] python/qemu/machine: fix potential hang in QMP accept

marcandre.lureau@redhat.com posted 3 patches 1 year, 3 months ago

Diff against v1 v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230111080101.969151-1-marcandre.lureau@redhat.com

Maintainers: John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>, Beraldo Leal <bleal@redhat.com>

python/qemu/machine/machine.py | 24 ++++++++++++++++--------
python/qemu/qmp/legacy.py      | 18 +++++++++++++++---
python/qemu/qmp/protocol.py    | 25 ++++++++++++++++++++-----
3 files changed, 51 insertions(+), 16 deletions(-)

Expand all Fold all

[PATCH v3 0/3] python/qemu/machine: fix potential hang in QMP accept

Posted by marcandre.lureau@redhat.com 1 year, 3 months ago

From: Marc-André Lureau <marcandre.lureau@redhat.com>

Hi,

As reported earlier by Richard Henderson ("virgl avocado hang" thread), avocado
tests may hang when QEMU exits before the QMP connection is established.

v3:
 - after merge in https://gitlab.com/qemu-project/python-qemu-qmp
 - resend as requested by John Snow

v2:
 - use a socketpair() for QMP (instead of async concurrent code from v1) as
   suggested by Daniel Berrange.
 - should not regress (hopefully)

Marc-André Lureau (3):
  python/qmp/protocol: add open_with_socket()
  python/qmp/legacy: make QEMUMonitorProtocol accept a socket
  python/qemu/machine: use socketpair() for QMP by default

 python/qemu/machine/machine.py | 24 ++++++++++++++++--------
 python/qemu/qmp/legacy.py      | 18 +++++++++++++++---
 python/qemu/qmp/protocol.py    | 25 ++++++++++++++++++++-----
 3 files changed, 51 insertions(+), 16 deletions(-)

-- 
2.39.0