diff options
| author | John Snow <jsnow@redhat.com> | 2022-02-25 15:59:47 -0500 |
|---|---|---|
| committer | John Snow <jsnow@redhat.com> | 2022-03-07 14:36:41 -0500 |
| commit | 673856f9d889dc50b6a1a7964df960c4f00c7c93 (patch) | |
| tree | f2286866c6e5c624df1c8ca80ff78f7052161b34 /python/qemu | |
| parent | 481607c7d35de2bc4d9bec7f4734036fc467f330 (diff) | |
| download | focaccia-qemu-673856f9d889dc50b6a1a7964df960c4f00c7c93.tar.gz focaccia-qemu-673856f9d889dc50b6a1a7964df960c4f00c7c93.zip | |
python/aqmp: fix race condition in legacy.py
legacy.py provides a synchronous model. iotests frequently uses this paradigm: - create QMP client object - start QEMU process - await connection from QEMU process In the switch from sync to async QMP, the QMP client object stopped calling bind() and listen() during the QMP object creation step, which creates a race condition if the QEMU process dials in too quickly. With refactoring out of the way, restore the former behavior of calling bind() and listen() during __init__() to fix this race condition. Signed-off-by: John Snow <jsnow@redhat.com> Acked-by: Kevin Wolf <kwolf@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Message-id: 20220225205948.3693480-10-jsnow@redhat.com [Expanded commit message. --js] Signed-off-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'python/qemu')
| -rw-r--r-- | python/qemu/aqmp/legacy.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/python/qemu/aqmp/legacy.py b/python/qemu/aqmp/legacy.py index dca1e76ed4..cb50e60564 100644 --- a/python/qemu/aqmp/legacy.py +++ b/python/qemu/aqmp/legacy.py @@ -57,7 +57,7 @@ class QEMUMonitorProtocol(qemu.qmp.QEMUMonitorProtocol): self._timeout: Optional[float] = None if server: - self._aqmp._bind_hack(address) # pylint: disable=protected-access + self._sync(self._aqmp.start_server(address)) _T = TypeVar('_T') @@ -90,10 +90,7 @@ class QEMUMonitorProtocol(qemu.qmp.QEMUMonitorProtocol): self._aqmp.await_greeting = True self._aqmp.negotiate = True - self._sync( - self._aqmp.start_server_and_accept(self._address), - timeout - ) + self._sync(self._aqmp.accept(), timeout) ret = self._get_greeting() assert ret is not None |