Re: [Qemu-devel] [PATCH v5 12/12] BlockDriver: add bdrv_get_allocated_file_size() operation

[Stefan Hajnoczi <stefanha@gmail.com>]

On Tue, Jun 28, 2011 at 2:32 AM, Fam Zheng <famcool@gmail.com> wrote:
Please use "block:" as the commit message tag instead of BlockDriver.
Usually the easiest way to find out which tag to use it by doing
git-log(1) on the main file you have modified and looking at previous
commit messages.

> +static int64_t vmdk_get_allocated_file_size(BlockDriverState *bs)
> +{
> +    int i;
> +    int64_t ret = 0;
> +    int64_t r;
> +    BDRVVmdkState *s = bs->opaque;
> +
> +    ret = bdrv_get_allocated_file_size(bs->file);
> +    if (ret < 0) {
> +        return ret;
> +    }
> +    for (i = 0; i < s->num_extents; i++) {
> +        r = bdrv_get_allocated_file_size(s->extents[i].file);
> +        if (r < 0) {
> +            return r;
> +        }
> +        ret += r;
> +    }
> +    return ret;
> +}

Does this count bs->file twice for images without a separate descriptor file?

Stefan