[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v20 4/5] block/gluster: using new qapi schema
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v20 4/5] block/gluster: using new qapi schema |
Date: |
Tue, 19 Jul 2016 19:37:57 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
One more thing...
Prasanna Kumar Kalever <address@hidden> writes:
> this patch adds 'GlusterServer' related schema in qapi/block-core.json
>
> Signed-off-by: Prasanna Kumar Kalever <address@hidden>
> ---
> block/gluster.c | 115
> +++++++++++++++++++++++++++++----------------------
> qapi/block-core.json | 68 +++++++++++++++++++++++++++---
> 2 files changed, 128 insertions(+), 55 deletions(-)
>
> diff --git a/block/gluster.c b/block/gluster.c
> index 8a54ad4..c4ca59e 100644
> --- a/block/gluster.c
> +++ b/block/gluster.c
[...]
> @@ -628,7 +641,9 @@ static int qemu_gluster_create(const char *filename,
> }
> out:
> g_free(tmp);
> - qemu_gluster_gconf_free(gconf);
> + if (gconf) {
> + qapi_free_BlockdevOptionsGluster(gconf);
> + }
qapi_free_FOO(NULL) is safe. Let's drop the conditional. Could be done
on commit, or as a follow-up cleanup.
> if (glfs) {
> glfs_fini(glfs);
> }
[...]
R-by stands.
- [Qemu-devel] [PATCH v20 0/5] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2016/07/19
- [Qemu-devel] [PATCH v20 1/5] block/gluster: rename [server, volname, image] -> [host, volume, path], Prasanna Kumar Kalever, 2016/07/19
- [Qemu-devel] [PATCH v20 2/5] block/gluster: code cleanup, Prasanna Kumar Kalever, 2016/07/19
- [Qemu-devel] [PATCH v20 3/5] block/gluster: deprecate rdma support, Prasanna Kumar Kalever, 2016/07/19
- [Qemu-devel] [PATCH v20 4/5] block/gluster: using new qapi schema, Prasanna Kumar Kalever, 2016/07/19
- [Qemu-devel] [PATCH v20 5/5] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2016/07/19
- Re: [Qemu-devel] [PATCH v20 0/5] block/gluster: add support for multiple gluster servers, Jeff Cody, 2016/07/19