[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine |
Date: |
Fri, 17 Jan 2020 10:13:52 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
Am 17.01.2020 um 09:13 hat Markus Armbruster geschrieben:
> Kevin Wolf <address@hidden> writes:
>
> > Am 16.01.2020 um 10:45 hat Markus Armbruster geschrieben:
> >> Kevin Wolf <address@hidden> writes:
> >> > block_resize is safe to run in a coroutine, so use it as an example for
> >> > the new 'coroutine': true annotation in the QAPI schema.
> >> >
> >> > Signed-off-by: Kevin Wolf <address@hidden>
> >> > Reviewed-by: Marc-André Lureau <address@hidden>
> >
> >> > diff --git a/blockdev.c b/blockdev.c
> >> > index 8e029e9c01..b5e5d1e072 100644
> >> > --- a/blockdev.c
> >> > +++ b/blockdev.c
> >> > @@ -3161,9 +3161,9 @@ void hmp_drive_del(Monitor *mon, const QDict
> >> > *qdict)
> >> > aio_context_release(aio_context);
> >> > }
> >> >
> >> > -void qmp_block_resize(bool has_device, const char *device,
> >> > - bool has_node_name, const char *node_name,
> >> > - int64_t size, Error **errp)
> >> > +void coroutine_fn qmp_block_resize(bool has_device, const char *device,
> >> > + bool has_node_name, const char
> >> > *node_name,
> >> > + int64_t size, Error **errp)
> >> > {
> >> > Error *local_err = NULL;
> >> > BlockBackend *blk = NULL;
> >>
> >> Pardon my ignorant question: what exactly makes a function a
> >> coroutine_fn?
> >
> > When Stefan requested adding the coroutine_fn marker, it seemed to make
> > sense to me because the QMP dispatcher will always call it from
> > coroutine context now, and being always run in coroutine context makes a
> > function a coroutine_fn.
> >
> > However, it's also called from hmp_block_resize(), so at least for now
> > coroutine_fn is actually wrong.
>
> With the coroutine_fn dropped:
> Reviewed-by: Markus Armbruster <address@hidden>
>
> Any plans to make more QMP commands 'coroutine': true?
No immediate plans from me. Myself, I was interested in block_resize
because without the conversion, Max wanted me to jump through some hoops
with a bdrv_truncate() fix to make sure that it wouldn't block the guest
during block_resize.
Of course, apart from that, there is Marc-André's long-standing
screendump bug that will finally have all of the required
infrastructure (and I think I saw a patch on the list already).
Kevin
- Re: [PATCH v3 2/4] vl: Initialise main loop earlier, (continued)
- [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Kevin Wolf, 2020/01/15
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Markus Armbruster, 2020/01/16
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Kevin Wolf, 2020/01/16
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Markus Armbruster, 2020/01/16
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Kevin Wolf, 2020/01/16
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Markus Armbruster, 2020/01/17
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Kevin Wolf, 2020/01/17
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Markus Armbruster, 2020/01/17
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine, Markus Armbruster, 2020/01/17
- Re: [PATCH v3 4/4] block: Mark 'block_resize' as coroutine,
Kevin Wolf <=
[PATCH v3 3/4] qmp: Move dispatcher to a coroutine, Kevin Wolf, 2020/01/15