[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info |
Date: |
Fri, 11 Oct 2013 05:52:28 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
On 10/11/2013 02:31 AM, Fam Zheng wrote:
> Implement .bdrv_get_specific_info to return the extent information.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> +
> + *spec_info->vmdk = (ImageInfoSpecificVmdk) {
> + .create_type = g_strdup(s->create_type),
> + .cid = s->cid,
> + };
> +
>
> ##
> +# @ImageInfoSpecificVmdk:
> +#
> +# @create_type: The create type of VMDK image
Is it worth making this an enum type rather than an open-coded string?
But that's not a show-stopper to me.
> +#
> +# @cid: Content id of image
> +#
> +# @parent-cid: Parent VMDK image's cid
> +#
> +# @extents: List of extent files
> +#
> +# Since: 1.7
> +##
> +{ 'type': 'ImageInfoSpecificVmdk',
> + 'data': {
> + 'create_type': 'str',
> + 'cid': 'int',
> + 'parent-cid': 'int',
> + 'extents': ['ImageInfo']
> + } }
Both patches look fine from the QMP point of view; I didn't closely
review the matching C code for accuracy though.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature