On Fri, Apr 16, 2021 at 11:09:05AM +0300, Vladimir Sementsov-Ogievskiy wrote: > Split out part, that we want to reuse for nbd_open().
Split out the part that we want to reuse for nbd_open(). > > Signed-off-by: Vladimir Sementsov-Ogievskiy <[email protected]> > --- > block/nbd.c | 79 +++++++++++++++++++++++++++-------------------------- > 1 file changed, 41 insertions(+), 38 deletions(-) > > diff --git a/block/nbd.c b/block/nbd.c > index 15b5921725..59971bfba8 100644 > --- a/block/nbd.c > +++ b/block/nbd.c > @@ -361,11 +361,49 @@ static int nbd_handle_updated_info(BlockDriverState > *bs, Error **errp) > return 0; > } > > -static coroutine_fn void nbd_reconnect_attempt(BDRVNBDState *s) > +static int nbd_co_do_establish_connection(BlockDriverState *bs, Error **errp) Given the _co_ in the name, don't you need a coroutine_fn marker? Otherwise looks sane. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
