[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/3] 9pfs: add cleanup operation for handle b
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/3] 9pfs: add cleanup operation for handle backend driver |
Date: |
Tue, 15 Nov 2016 11:22:03 +0100 |
On Mon, 14 Nov 2016 21:13:52 -0500
Li Qiang <address@hidden> wrote:
> In the init operation of handle backend dirver, it allocates a
> handle_data struct and opens a mount file. We should free these
> resources when the 9pfs device is unrealized. This is what this
> patch does.
>
> Signed-off-by: Li Qiang <address@hidden>
> ---
Reviewed-by: Greg Kurz <address@hidden>
> hw/9pfs/9p-handle.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/hw/9pfs/9p-handle.c b/hw/9pfs/9p-handle.c
> index 3d77594..9b50f40 100644
> --- a/hw/9pfs/9p-handle.c
> +++ b/hw/9pfs/9p-handle.c
> @@ -649,6 +649,13 @@ out:
> return ret;
> }
>
> +static void handle_cleanup(FsContext *ctx)
> +{
> + struct handle_data *data = ctx->private;
> + close(data->mountfd);
> + g_free(data);
> +}
> +
> static int handle_parse_opts(QemuOpts *opts, struct FsDriverEntry *fse)
> {
> const char *sec_model = qemu_opt_get(opts, "security_model");
> @@ -671,6 +678,7 @@ static int handle_parse_opts(QemuOpts *opts, struct
> FsDriverEntry *fse)
> FileOperations handle_ops = {
> .parse_opts = handle_parse_opts,
> .init = handle_init,
> + .cleanup = handle_cleanup,
> .lstat = handle_lstat,
> .readlink = handle_readlink,
> .close = handle_close,