Kevin Wolf <kw...@redhat.com> 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 <kw...@redhat.com> > Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> > --- > qapi/block-core.json | 3 ++- > blockdev.c | 6 +++--- > 2 files changed, 5 insertions(+), 4 deletions(-) > > diff --git a/qapi/block-core.json b/qapi/block-core.json > index 7ff5e5edaf..1dbb2a9901 100644 > --- a/qapi/block-core.json > +++ b/qapi/block-core.json > @@ -1341,7 +1341,8 @@ > { 'command': 'block_resize', > 'data': { '*device': 'str', > '*node-name': 'str', > - 'size': 'int' } } > + 'size': 'int' }, > + 'coroutine': true } > > ## > # @NewImageMode: > 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?