[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/6] monitor: misc fixes
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/6] monitor: misc fixes |
Date: |
Tue, 30 Oct 2018 12:06:51 +0400 |
Hi
On Tue, Oct 30, 2018 at 9:48 AM Peter Xu <address@hidden> wrote:
>
> On Mon, Oct 29, 2018 at 04:57:27PM +0400, Marc-André Lureau wrote:
> > Hi,
> >
> > Here is a small series of fixes for the monitor, mostly related to
> > threading issues.
>
> Hi, Marc-André, Markus,
>
> I'd be glad to know how you think about these monitor series,
> considering that I just noticed it's exactly softfreeze for QEMU 3.1
> today...
>
> I gave a quick look on this series, IMHO patch 1,2,6 could be
> candidate for 3.1 (though I'm not sure for patch 6; it seems even fine
> to me with some comments there but I'd like to see how other people
> think about it too, and whether it's really possible to insert a new
> monitor at that time) and I'm not sure whether the rest can be
Patch 5 & 6 are the result of the discussion of the previous proposed fixed:
https://lists.nongnu.org/archive/html/qemu-devel/2018-08/msg00018.html
They fix a dead-lock on cleanup, so it should be considered for 3.1.
> postponed (say, patch 3,4,5). Meanwhile, I think we need Wolfgang's
Patch 3 & 4 should also be considered, as OOB won't have the expected
behaviour with all chardevs. With non-gcontext feature chardev, they
will behave more or less like non-oob.
> patches to fix the daemonize chaos, and hopefully I also like to know
> your thoughts on my oob enablement series (especially Marc-André,
> since I talked to Markus face to face during the forum after all :).
> I would like it to be in 3.1 as well but I'll see my luck.
I acked the series already (except "monitor: resume the monitor
earlier if needed" which we agreed could be dropped for now), so it's
in Markus hand.
Thanks
- [Qemu-devel] [PATCH v2 1/6] monitor: inline ambiguous helper functions, (continued)
- [Qemu-devel] [PATCH v2 1/6] monitor: inline ambiguous helper functions, Marc-André Lureau, 2018/10/29
- [Qemu-devel] [PATCH v2 2/6] monitor: accept chardev input from iothread, Marc-André Lureau, 2018/10/29
- [Qemu-devel] [PATCH v2 3/6] char: add a QEMU_CHAR_FEATURE_GCONTEXT flag, Marc-André Lureau, 2018/10/29
- [Qemu-devel] [PATCH v2 4/6] monitor: check if chardev can switch gcontext for OOB, Marc-André Lureau, 2018/10/29
- [Qemu-devel] [PATCH v2 6/6] monitor: avoid potential dead-lock when cleaning up, Marc-André Lureau, 2018/10/29
- [Qemu-devel] [PATCH v2 5/6] monitor: prevent inserting new monitors after cleanup, Marc-André Lureau, 2018/10/29
- Re: [Qemu-devel] [PATCH v2 0/6] monitor: misc fixes, Peter Xu, 2018/10/30
- Re: [Qemu-devel] [PATCH v2 0/6] monitor: misc fixes,
Marc-André Lureau <=
- Re: [Qemu-devel] [PATCH v2 0/6] monitor: misc fixes, Markus Armbruster, 2018/10/30