Re: [Qemu-devel] [PATCH v7 12/23] monitor: let suspend_cnt be thread safe

2018-02-21 Thread Stefan Hajnoczi
Nevermind, I see the next patch adds QMP support. Stefan signature.asc Description: PGP signature

Re: [Qemu-devel] [PATCH v7 12/23] monitor: let suspend_cnt be thread safe

2018-02-21 Thread Stefan Hajnoczi
On Wed, Jan 24, 2018 at 01:39:46PM +0800, Peter Xu wrote: > Monitor code now can be run in more than one thread. Let it be thread > safe when accessing suspend_cnt counter. > > Reviewed-by: Eric Blake > Signed-off-by: Peter Xu > --- > monitor.c | 15

Re: [Qemu-devel] [PATCH v7 12/23] monitor: let suspend_cnt be thread safe

2018-02-21 Thread Stefan Hajnoczi
On Wed, Jan 24, 2018 at 01:39:46PM +0800, Peter Xu wrote: > @@ -3996,7 +3996,7 @@ int monitor_suspend(Monitor *mon) > { > if (!mon->rs) > return -ENOTTY; Does this mean that QMP monitors like -qmp unix:/tmp/foo,server,nowait will not use suspend_cnt? I thought the point was to use

[Qemu-devel] [PATCH v7 12/23] monitor: let suspend_cnt be thread safe

2018-01-23 Thread Peter Xu
Monitor code now can be run in more than one thread. Let it be thread safe when accessing suspend_cnt counter. Reviewed-by: Eric Blake Signed-off-by: Peter Xu --- monitor.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git