Re: [PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-03-20 Thread Daniel P . Berrangé
On Fri, Mar 17, 2023 at 10:36:37PM +0300, Vladimir Sementsov-Ogievskiy wrote: > Hi! > > By my investigation, this commit (bd4c0ef409140bd1be393407c04005ac077d4574) > breaks long qmp output again. > > ./test.py 1000 /tmp/sock > > - works, but if use default behavior (socketpair) we get: > >

Re: [PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-03-17 Thread Vladimir Sementsov-Ogievskiy
Hi! By my investigation, this commit (bd4c0ef409140bd1be393407c04005ac077d4574) breaks long qmp output again. Simple test: $ cd python $ cat test.py #!/usr/bin/env python3 import sys from qemu.machine import QEMUMachine monitor_address = sys.argv[2] if len(sys.argv) > 2 else None vm =

Re: [PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-01-23 Thread John Snow
On Wed, Jan 18, 2023 at 3:03 AM Marc-André Lureau wrote: > > Hi > > On Wed, Jan 18, 2023 at 2:36 AM John Snow wrote: > > > > On Wed, Jan 11, 2023 at 3:01 AM wrote: > > > > > > From: Marc-André Lureau > > > > > > When no monitor address is given, establish the QMP communication through > > > a

Re: [PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-01-18 Thread Marc-André Lureau
Hi On Wed, Jan 18, 2023 at 2:36 AM John Snow wrote: > > On Wed, Jan 11, 2023 at 3:01 AM wrote: > > > > From: Marc-André Lureau > > > > When no monitor address is given, establish the QMP communication through > > a socketpair() (API is also supported on Windows since Python 3.5) > > > >

Re: [PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-01-17 Thread John Snow
On Wed, Jan 11, 2023 at 3:01 AM wrote: > > From: Marc-André Lureau > > When no monitor address is given, establish the QMP communication through > a socketpair() (API is also supported on Windows since Python 3.5) > > Signed-off-by: Marc-André Lureau > Reviewed-by: Daniel P. Berrangé > --- >

[PATCH v3 3/3] python/qemu/machine: use socketpair() for QMP by default

2023-01-11 Thread marcandre . lureau
From: Marc-André Lureau When no monitor address is given, establish the QMP communication through a socketpair() (API is also supported on Windows since Python 3.5) Signed-off-by: Marc-André Lureau Reviewed-by: Daniel P. Berrangé --- python/qemu/machine/machine.py | 24