Re: [PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-15 Thread Stefan Hajnoczi
On Tue, Jan 14, 2020 at 07:27:31PM +0100, Kevin Wolf wrote: > Some QMP command handlers can block the main loop for a relatively long > time, for example because they perform some I/O. This is quite nasty. > Allowing such handlers to run in a coroutine where they can yield (and > therefore release

Re: [PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-15 Thread Kevin Wolf
Am 14.01.2020 um 21:41 hat no-re...@patchew.org geschrieben: > Patchew URL: https://patchew.org/QEMU/20200114182735.5553-1-kw...@redhat.com/ > > Hi, > > This series failed the docker-quick@centos7 build test. Please find the > testing commands and > their output below. If you have Docker

Re: [PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-14 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200114182735.5553-1-kw...@redhat.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN ===

Re: [PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-14 Thread Kevin Wolf
Am 14.01.2020 um 19:45 hat Daniel P. Berrangé geschrieben: > On Tue, Jan 14, 2020 at 07:27:31PM +0100, Kevin Wolf wrote: > > Some QMP command handlers can block the main loop for a relatively long > > time, for example because they perform some I/O. This is quite nasty. > > Allowing such handlers

Re: [PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-14 Thread Daniel P . Berrangé
On Tue, Jan 14, 2020 at 07:27:31PM +0100, Kevin Wolf wrote: > Some QMP command handlers can block the main loop for a relatively long > time, for example because they perform some I/O. This is quite nasty. > Allowing such handlers to run in a coroutine where they can yield (and > therefore release

[PATCH v2 0/4] qmp: Optionally run handlers in coroutines

2020-01-14 Thread Kevin Wolf
Some QMP command handlers can block the main loop for a relatively long time, for example because they perform some I/O. This is quite nasty. Allowing such handlers to run in a coroutine where they can yield (and therefore release the BQL) while waiting for an event such as I/O completion solves