[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 0/4] monitor/hmp: cleanup monitor_event() and suspend_cnt
|
From: |
Paolo Bonzini |
|
Subject: |
[PATCH 0/4] monitor/hmp: cleanup monitor_event() and suspend_cnt |
|
Date: |
Thu, 18 May 2023 12:18:00 +0200 |
monitor_event() is not using monitor_suspend() and monitor_resume()
even though that is the desired outcome of moving out of and back into
the monitor with Ctrl-a c.
While cleaning this up, make the locking of several fields in struct
Monitor less esoteric, i.e. protect them just with mon_lock. This gets
rid of one of the last two cases where qatomic_mb_read()/qatomic_mb_set()
pair is used to achieve a semblance of sequential consistency.
Paolo
Paolo Bonzini (4):
monitor: use QEMU_LOCK_GUARD a bit more
monitor: allow calling monitor_resume under mon_lock
monitor: add more *_locked() functions
monitor: do not use mb_read/mb_set for suspend_cnt
include/monitor/monitor.h | 3 +++
monitor/hmp.c | 41 ++++++++++++++++-------------------
monitor/monitor-internal.h | 3 +--
monitor/monitor.c | 39 ++++++++++++++++++---------------
tests/qemu-iotests/051.out | 4 ++--
tests/qemu-iotests/051.pc.out | 20 ++++++++---------
6 files changed, 57 insertions(+), 53 deletions(-)
--
2.40.1
- [PATCH 0/4] monitor/hmp: cleanup monitor_event() and suspend_cnt,
Paolo Bonzini <=
- [PATCH 1/4] monitor: use QEMU_LOCK_GUARD a bit more, Paolo Bonzini, 2023/05/18
- [PATCH 2/4] monitor: allow calling monitor_resume under mon_lock, Paolo Bonzini, 2023/05/18
- [PATCH 3/4] monitor: add more *_locked() functions, Paolo Bonzini, 2023/05/18
- [PATCH 4/4] monitor: do not use mb_read/mb_set for suspend_cnt, Paolo Bonzini, 2023/05/18
- Re: [PATCH 0/4] monitor/hmp: cleanup monitor_event() and suspend_cnt, Markus Armbruster, 2023/05/25