[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 00/23] QMP: out-of-band (OOB) execution suppo
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v7 00/23] QMP: out-of-band (OOB) execution support |
Date: |
Thu, 25 Jan 2018 10:01:13 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
* Peter Xu (address@hidden) wrote:
> On Wed, Jan 24, 2018 at 01:39:34PM +0800, Peter Xu wrote:
> > This version should have addressed all comments in previous one, also
> > fixed another race condition after I addressed all the comments (a new
> > race condition introduced by addressing the comments...). For some
> > more details of the race condition, please see the last entry of
> > change log, and please refer to patch 9 for the code change.
> >
> > I removed RFC tag from this version. Please review. Thanks.
> >
> > v7:
> > - add some r-bs, and remove some.
> > - remove the chardev fix since already queued by Paolo
> > - use local var in qemu_chr_fe_add_watch [Stefan]
> > - move doc patch to front, mention it in some patches [Eric]
> > - Quite a few of English fixes [Eric]
> > - fix unlock missing in handle_qmp_command [Stefan]
> > - squash some patches according to the review comments
> > - don't break gdbserver usage on HMP non-interactive mode by fixing up
> > the suspend/resume logic [Fam, Stefan]
> > - move the qemu_chr_fe_set_handlers() call in monitor_init() into a
> > bottom half to avoid race between the call itself and
> > iothread. [Stefan]
> > - spent quite a lot of time debugging another assertion failure in
> > io_watch_poll_finalize() after above change is made (ouch! I really
> > hoped we always have the latest glib): when QEMU inits chardevs in
> > chardev_init_func() it's possible that QEMU registers the chardev
> > handlers there, even before CharBackend is connected to that chardev
> > in monitor_init(). Then, when we reach monitor_init() we must make
> > sure we unregister that old one first, or there can have one orphan
> > GSource still in default gcontext (note that this can really happen
> > when we start to use QEMUBH to setup chardev frontends, which is
> > above change).
>
> Online repository updated for v7:
>
> https://github.com/xzpeter/qemu/tree/monitor-oob
This version seems to have fixed the problem I had where after an error
I couldn't reconnect (the one I mentioned on the 12th).
Dave
> Thanks,
>
> --
> Peter Xu
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-devel] [PATCH v7 15/23] qmp: add new event "command-dropped", (continued)
- [Qemu-devel] [PATCH v7 15/23] qmp: add new event "command-dropped", Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 16/23] monitor: send event when command queue full, Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 17/23] qapi: introduce new cmd option "allow-oob", Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 18/23] qmp: support out-of-band (oob) execution, Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 19/23] qmp: isolate responses into io thread, Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 20/23] monitor: enable IO thread for (qmp & !mux) typed, Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 21/23] qmp: add command "x-oob-test", Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 22/23] tests: qmp-test: verify command batching, Peter Xu, 2018/01/24
- [Qemu-devel] [PATCH v7 23/23] tests: qmp-test: add oob test, Peter Xu, 2018/01/24
- Re: [Qemu-devel] [PATCH v7 00/23] QMP: out-of-band (OOB) execution support, Peter Xu, 2018/01/24
- Re: [Qemu-devel] [PATCH v7 00/23] QMP: out-of-band (OOB) execution support,
Dr. David Alan Gilbert <=