[Qemu-devel] [PATCH 0/2] qemu-thread: allow cur_mon be per thread

Peter Xu posted 2 patches 6 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180410124913.10832-1-peterx@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test s390x passed
include/monitor/monitor.h   |  2 +-
include/qemu/thread-win32.h |  1 +
monitor.c                   |  2 +-
stubs/monitor.c             |  2 +-
tests/test-util-sockets.c   |  2 +-
util/qemu-thread-posix.c    | 39 +++++++++++++++++++--------------------
util/qemu-thread-win32.c    |  6 ++++++
7 files changed, 30 insertions(+), 24 deletions(-)
[Qemu-devel] [PATCH 0/2] qemu-thread: allow cur_mon be per thread
Posted by Peter Xu 6 years ago
This should be for 2.13.  But I'd like to get early review comments
too if there is any.

Now cur_mon is still only be accessed by the main thread.  So we don't
even need per-thread cur_mon.  However after more commands become OOB
compatible, cur_mon can be accessed by more than main thread now.  The
major user should be the monitor IOThread.  This series tries to let
cur_mon be per-thread, so that we can be well-prepared.

The first patch is a cleanup.  The second patch does the idea.

Any early review comments would be welcomed.  Thanks,

Peter Xu (2):
  qemu-thread: always keep the posix wrapper layer
  qemu-thread: let cur_mon be per-thread

 include/monitor/monitor.h   |  2 +-
 include/qemu/thread-win32.h |  1 +
 monitor.c                   |  2 +-
 stubs/monitor.c             |  2 +-
 tests/test-util-sockets.c   |  2 +-
 util/qemu-thread-posix.c    | 39 +++++++++++++++++++--------------------
 util/qemu-thread-win32.c    |  6 ++++++
 7 files changed, 30 insertions(+), 24 deletions(-)

-- 
2.14.3