All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Xu <peterx@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Paolo Bonzini" <pbonzini@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Fam Zheng" <famz@redhat.com>,
	peterx@redhat.com, "Eric Blake" <eblake@redhat.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>
Subject: [Qemu-devel] [PATCH 0/2] qemu-thread: allow cur_mon be per thread
Date: Tue, 10 Apr 2018 20:49:11 +0800	[thread overview]
Message-ID: <20180410124913.10832-1-peterx@redhat.com> (raw)

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

             reply	other threads:[~2018-04-10 12:49 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-10 12:49 Peter Xu [this message]
2018-04-10 12:49 ` [Qemu-devel] [PATCH 1/2] qemu-thread: always keep the posix wrapper layer Peter Xu
2018-04-10 13:35   ` Eric Blake
2018-04-11  3:18     ` Peter Xu
2018-04-10 12:49 ` [Qemu-devel] [PATCH 2/2] qemu-thread: let cur_mon be per-thread Peter Xu
2018-04-10 13:54   ` Eric Blake
2018-04-11  3:31     ` Peter Xu
2018-04-11  1:45   ` Stefan Hajnoczi
2018-04-11  3:49     ` Peter Xu
2018-04-11  9:23       ` Paolo Bonzini
2018-04-11  9:35         ` Peter Xu
2018-04-11  9:38           ` Paolo Bonzini
2018-04-11  9:48             ` Peter Xu
2018-04-11 13:06               ` Eric Blake
2018-04-12  5:24                 ` Peter Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180410124913.10832-1-peterx@redhat.com \
    --to=peterx@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=armbru@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=eblake@redhat.com \
    --cc=famz@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.