Re: [PATCH v6 00/25] monitor: add asynchronous command type

2020-01-13 Thread Kevin Wolf
Am 13.01.2020 um 16:58 hat Markus Armbruster geschrieben: > Marc-André Lureau writes: > > > Hi > > > > On Tue, Jan 7, 2020 at 9:17 AM Kevin Wolf wrote: > >> > >> Am 06.01.2020 um 19:21 hat Marc-André Lureau geschrieben: > >> > > What my patch does is moving everything into a coroutine. This is w

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2020-01-13 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Tue, Jan 7, 2020 at 9:17 AM Kevin Wolf wrote: >> >> Am 06.01.2020 um 19:21 hat Marc-André Lureau geschrieben: >> > > What my patch does is moving everything into a coroutine. This is wrong >> > > because not everything can be run in a coroutine, so it needs

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2020-01-07 Thread Marc-André Lureau
Hi On Tue, Jan 7, 2020 at 9:17 AM Kevin Wolf wrote: > > Am 06.01.2020 um 19:21 hat Marc-André Lureau geschrieben: > > > What my patch does is moving everything into a coroutine. This is wrong > > > because not everything can be run in a coroutine, so it needs to be made > > > optional (like you d

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2020-01-06 Thread Kevin Wolf
Am 06.01.2020 um 19:21 hat Marc-André Lureau geschrieben: > > What my patch does is moving everything into a coroutine. This is wrong > > because not everything can be run in a coroutine, so it needs to be made > > optional (like you did with your async flag). > > "everything" is a bit too much ;)

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2020-01-06 Thread Marc-André Lureau
Hi Kevin, On Mon, Dec 16, 2019 at 4:07 PM Kevin Wolf wrote: > > Am 13.12.2019 um 17:28 hat Marc-André Lureau geschrieben: > > On Fri, Dec 13, 2019 at 8:04 PM Kevin Wolf wrote: > > > > > > Am 08.11.2019 um 16:00 hat Marc-André Lureau geschrieben: > > > > The following series implements an interna

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2019-12-16 Thread Kevin Wolf
Am 13.12.2019 um 17:28 hat Marc-André Lureau geschrieben: > On Fri, Dec 13, 2019 at 8:04 PM Kevin Wolf wrote: > > > > Am 08.11.2019 um 16:00 hat Marc-André Lureau geschrieben: > > > The following series implements an internal async command solution > > > instead. By introducing a session context a

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2019-12-13 Thread Marc-André Lureau
Hi On Fri, Dec 13, 2019 at 8:04 PM Kevin Wolf wrote: > > Am 08.11.2019 um 16:00 hat Marc-André Lureau geschrieben: > > The following series implements an internal async command solution > > instead. By introducing a session context and a command return > > handler, QMP handlers can: > > - defer t

Re: [PATCH v6 00/25] monitor: add asynchronous command type

2019-12-13 Thread Kevin Wolf
Am 08.11.2019 um 16:00 hat Marc-André Lureau geschrieben: > The following series implements an internal async command solution > instead. By introducing a session context and a command return > handler, QMP handlers can: > - defer the return, allowing the mainloop to reenter > - return only to the

[PATCH v6 00/25] monitor: add asynchronous command type

2019-11-08 Thread Marc-André Lureau
Hi, HMP and QMP commands are handled synchronously in qemu today. But there are benefits allowing the command handler to re-enter the main loop if the command cannot be handled synchronously, or if it is long-lasting. Some bugs such as rhbz#1230527 are difficult to solve without it. The common so