[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH 04/34] vmdk: Use bdrv_open_image()
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH 04/34] vmdk: Use bdrv_open_image() |
Date: |
Tue, 12 May 2015 15:12:38 -0400 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, May 08, 2015 at 07:21:36PM +0200, Kevin Wolf wrote:
> Besides standardising on a single interface for opening child nodes,
> this patch allows the user to specify options to individual extent
> nodes. Overriding file names isn't possible with this yet, so it's of
> limited usefulness, but still a step forward.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> block/vmdk.c | 34 +++++++++++++++++++++-------------
> 1 file changed, 21 insertions(+), 13 deletions(-)
>
> diff --git a/block/vmdk.c b/block/vmdk.c
> index b66745d..641e026 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -543,7 +543,7 @@ static int vmdk_open_vmfs_sparse(BlockDriverState *bs,
> }
>
> static int vmdk_open_desc_file(BlockDriverState *bs, int flags, char *buf,
> - Error **errp);
> + QDict *options, Error **errp);
>
> static char *vmdk_read_desc(BlockDriverState *file, uint64_t desc_offset,
> Error **errp)
> @@ -582,7 +582,7 @@ static char *vmdk_read_desc(BlockDriverState *file,
> uint64_t desc_offset,
>
> static int vmdk_open_vmdk4(BlockDriverState *bs,
> BlockDriverState *file,
> - int flags, Error **errp)
> + int flags, QDict *options, Error **errp)
> {
> int ret;
> uint32_t magic;
> @@ -606,7 +606,7 @@ static int vmdk_open_vmdk4(BlockDriverState *bs,
> if (!buf) {
> return -EINVAL;
> }
> - ret = vmdk_open_desc_file(bs, flags, buf, errp);
> + ret = vmdk_open_desc_file(bs, flags, buf, options, errp);
> g_free(buf);
> return ret;
> }
> @@ -763,7 +763,7 @@ static int vmdk_parse_description(const char *desc, const
> char *opt_name,
> /* Open an extent file and append to bs array */
> static int vmdk_open_sparse(BlockDriverState *bs,
> BlockDriverState *file, int flags,
> - char *buf, Error **errp)
> + char *buf, QDict *options, Error **errp)
> {
> uint32_t magic;
>
> @@ -773,7 +773,7 @@ static int vmdk_open_sparse(BlockDriverState *bs,
> return vmdk_open_vmfs_sparse(bs, file, flags, errp);
> break;
> case VMDK4_MAGIC:
> - return vmdk_open_vmdk4(bs, file, flags, errp);
> + return vmdk_open_vmdk4(bs, file, flags, options, errp);
> break;
> default:
> error_setg(errp, "Image not in VMDK format");
> @@ -783,7 +783,8 @@ static int vmdk_open_sparse(BlockDriverState *bs,
> }
>
> static int vmdk_parse_extents(const char *desc, BlockDriverState *bs,
> - const char *desc_file_path, Error **errp)
> + const char *desc_file_path, QDict *options,
> + Error **errp)
> {
> int ret;
> int matches;
> @@ -797,6 +798,7 @@ static int vmdk_parse_extents(const char *desc,
> BlockDriverState *bs,
> BlockDriverState *extent_file;
> BDRVVmdkState *s = bs->opaque;
> VmdkExtent *extent;
> + char extent_opt_prefix[32];
>
> while (*p) {
> /* parse extent line in one of below formats:
> @@ -846,8 +848,13 @@ static int vmdk_parse_extents(const char *desc,
> BlockDriverState *bs,
> extent_path = g_malloc0(PATH_MAX);
> path_combine(extent_path, PATH_MAX, desc_file_path, fname);
> extent_file = NULL;
> - ret = bdrv_open(&extent_file, extent_path, NULL, NULL,
> - bs->open_flags | BDRV_O_PROTOCOL, NULL, errp);
> +
> + ret = snprintf(extent_opt_prefix, 32, "extents.%d", s->num_extents);
> + assert(ret < 32);
> +
> + ret = bdrv_open_image(&extent_file, extent_path,
> + options, extent_opt_prefix,
> + bs->open_flags | BDRV_O_PROTOCOL, false, errp);
> g_free(extent_path);
> if (ret) {
> return ret;
> @@ -870,7 +877,8 @@ static int vmdk_parse_extents(const char *desc,
> BlockDriverState *bs,
> if (!buf) {
> ret = -EINVAL;
> } else {
> - ret = vmdk_open_sparse(bs, extent_file, bs->open_flags, buf,
> errp);
> + ret = vmdk_open_sparse(bs, extent_file, bs->open_flags, buf,
> + options, errp);
> }
> g_free(buf);
> if (ret) {
> @@ -898,7 +906,7 @@ next_line:
> }
>
> static int vmdk_open_desc_file(BlockDriverState *bs, int flags, char *buf,
> - Error **errp)
> + QDict *options, Error **errp)
> {
> int ret;
> char ct[128];
> @@ -920,7 +928,7 @@ static int vmdk_open_desc_file(BlockDriverState *bs, int
> flags, char *buf,
> }
> s->create_type = g_strdup(ct);
> s->desc_offset = 0;
> - ret = vmdk_parse_extents(buf, bs, bs->file->exact_filename, errp);
> + ret = vmdk_parse_extents(buf, bs, bs->file->exact_filename, options,
> errp);
> exit:
> return ret;
> }
> @@ -942,11 +950,11 @@ static int vmdk_open(BlockDriverState *bs, QDict
> *options, int flags,
> switch (magic) {
> case VMDK3_MAGIC:
> case VMDK4_MAGIC:
> - ret = vmdk_open_sparse(bs, bs->file, flags, buf, errp);
> + ret = vmdk_open_sparse(bs, bs->file, flags, buf, options, errp);
> s->desc_offset = 0x200;
> break;
> default:
> - ret = vmdk_open_desc_file(bs, flags, buf, errp);
> + ret = vmdk_open_desc_file(bs, flags, buf, options, errp);
> break;
> }
> if (ret) {
> --
> 1.8.3.1
>
>
Reviewed-by: Jeff Cody <address@hidden>
- Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries(), (continued)
- [Qemu-devel] [PATCH 02/34] qdict: Add qdict_{set,copy}_default(), Kevin Wolf, 2015/05/08
- [Qemu-devel] [PATCH 04/34] vmdk: Use bdrv_open_image(), Kevin Wolf, 2015/05/08
- [Qemu-devel] [PATCH 05/34] block: Use macro for cache option names, Kevin Wolf, 2015/05/08
- Re: [Qemu-devel] [PATCH 05/34] block: Use macro for cache option names, Max Reitz, 2015/05/11
- Re: [Qemu-devel] [Qemu-block] [PATCH 05/34] block: Use macro for cache option names, Jeff Cody, 2015/05/12
- Re: [Qemu-devel] [Qemu-block] [PATCH 05/34] block: Use macro for cache option names, Alberto Garcia, 2015/05/20
- [Qemu-devel] [PATCH 06/34] block: Use QemuOpts in bdrv_open_common(), Kevin Wolf, 2015/05/08