Re: [Qemu-devel] [Qemu-block] [PATCH v2] rbd: make the code more readable
On Sat, Oct 15, 2016 at 04:26:13PM +0800, Xiubo Li wrote: > Make it a bit clearer and more readable. > > Signed-off-by: Xiubo Li> CC: John Snow > --- > > V2: > - Advice from John Snow. Thanks. > > > block/rbd.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-) > > diff --git a/block/rbd.c b/block/rbd.c > index 0a5840d..d0d4b39 100644 > --- a/block/rbd.c > +++ b/block/rbd.c > @@ -366,45 +366,44 @@ static int qemu_rbd_create(const char *filename, > QemuOpts *opts, Error **errp) > rados_conf_read_file(cluster, NULL); > } else if (conf[0] != '\0' && > qemu_rbd_set_conf(cluster, conf, true, _err) < 0) { > -rados_shutdown(cluster); > error_propagate(errp, local_err); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > if (conf[0] != '\0' && > qemu_rbd_set_conf(cluster, conf, false, _err) < 0) { > -rados_shutdown(cluster); > error_propagate(errp, local_err); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > if (qemu_rbd_set_auth(cluster, secretid, errp) < 0) { > -rados_shutdown(cluster); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > ret = rados_connect(cluster); > if (ret < 0) { > error_setg_errno(errp, -ret, "error connecting"); > -rados_shutdown(cluster); > -return ret; > +goto shutdown; > } > > ret = rados_ioctx_create(cluster, pool, _ctx); > if (ret < 0) { > error_setg_errno(errp, -ret, "error opening pool %s", pool); > -rados_shutdown(cluster); > -return ret; > +goto shutdown; > } > > ret = rbd_create(io_ctx, name, bytes, _order); > -rados_ioctx_destroy(io_ctx); > -rados_shutdown(cluster); > if (ret < 0) { > error_setg_errno(errp, -ret, "error rbd create"); > -return ret; > } > > +rados_ioctx_destroy(io_ctx); > + > +shutdown: > +rados_shutdown(cluster); > return ret; > } > > -- > 1.8.3.1 > > > > Thanks, Applied to my block branch: git://github.com/codyprime/qemu-kvm-jtc.git block -Jeff
Re: [Qemu-devel] [Qemu-block] [PATCH v2] rbd: make the code more readable
Am 15.10.2016 um 10:26 hat Xiubo Li geschrieben: > Make it a bit clearer and more readable. > > Signed-off-by: Xiubo Li> CC: John Snow > --- > > V2: > - Advice from John Snow. Thanks. Copying the official maintainers: $ scripts/get_maintainer.pl -f block/rbd.c Josh Durgin (supporter:RBD) Jeff Cody (supporter:RBD) ... > > block/rbd.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-) > > diff --git a/block/rbd.c b/block/rbd.c > index 0a5840d..d0d4b39 100644 > --- a/block/rbd.c > +++ b/block/rbd.c > @@ -366,45 +366,44 @@ static int qemu_rbd_create(const char *filename, > QemuOpts *opts, Error **errp) > rados_conf_read_file(cluster, NULL); > } else if (conf[0] != '\0' && > qemu_rbd_set_conf(cluster, conf, true, _err) < 0) { > -rados_shutdown(cluster); > error_propagate(errp, local_err); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > if (conf[0] != '\0' && > qemu_rbd_set_conf(cluster, conf, false, _err) < 0) { > -rados_shutdown(cluster); > error_propagate(errp, local_err); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > if (qemu_rbd_set_auth(cluster, secretid, errp) < 0) { > -rados_shutdown(cluster); > -return -EIO; > +ret = -EIO; > +goto shutdown; > } > > ret = rados_connect(cluster); > if (ret < 0) { > error_setg_errno(errp, -ret, "error connecting"); > -rados_shutdown(cluster); > -return ret; > +goto shutdown; > } > > ret = rados_ioctx_create(cluster, pool, _ctx); > if (ret < 0) { > error_setg_errno(errp, -ret, "error opening pool %s", pool); > -rados_shutdown(cluster); > -return ret; > +goto shutdown; > } > > ret = rbd_create(io_ctx, name, bytes, _order); > -rados_ioctx_destroy(io_ctx); > -rados_shutdown(cluster); > if (ret < 0) { > error_setg_errno(errp, -ret, "error rbd create"); > -return ret; > } > > +rados_ioctx_destroy(io_ctx); > + > +shutdown: > +rados_shutdown(cluster); > return ret; > } > > -- > 1.8.3.1 > > > >
Re: [Qemu-devel] [Qemu-block] [PATCH v2] rbd: make the code more readable
On Sat, Oct 15, 2016 at 04:26:13PM +0800, Xiubo Li wrote: > Make it a bit clearer and more readable. > > Signed-off-by: Xiubo Li> CC: John Snow > --- > > V2: > - Advice from John Snow. Thanks. > > > block/rbd.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature