[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monito
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read |
Date: |
Wed, 03 May 2017 11:55:34 +0000 |
Hi
On Wed, May 3, 2017 at 3:36 PM Daniel P. Berrange <address@hidden>
wrote:
> On Wed, May 03, 2017 at 01:29:57PM +0200, Markus Armbruster wrote:
>
> > The only difference to before that I can see is that we can read ahead.
> > That's a feature.
> >
> > Looks safe to me. Opinions?
>
> Yes, I concur, it looks safe for QMP.
>
> I might suggest putting an assert(!qmp) in monitor_suspend() to guarantee
> no one accidentally introduces usage of the suspend feature in QMP in
> future.
>
fwiw, in the qapi-async series, I added a bunch of related assert:
https://github.com/elmarco/qemu/commit/48d0691fef7602b652b8e2a2a8c0c6665f8e7c14
--
Marc-André Lureau
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, (continued)
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Markus Armbruster, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Denis V. Lunev, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Markus Armbruster, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Dr. David Alan Gilbert, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Daniel P. Berrange, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Dr. David Alan Gilbert, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Denis V. Lunev, 2017/05/02
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Markus Armbruster, 2017/05/03
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Daniel P. Berrange, 2017/05/03
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read,
Marc-André Lureau <=
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Denis V. Lunev, 2017/05/03
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Denis V. Lunev, 2017/05/03
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Markus Armbruster, 2017/05/10
- Re: [Qemu-devel] [PATCH 1/1] monitor: increase amount of data for monitor to read, Denis V. Lunev, 2017/05/10