[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 1/3] vmdk: Fix comment regarding max l1_size
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v2 1/3] vmdk: Fix comment regarding max l1_size coverage |
Date: |
Wed, 19 Jun 2019 19:10:04 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 05.06.19 14:17, Sam Eiderman wrote:
> Commit b0651b8c246d ("vmdk: Move l1_size check into vmdk_add_extent")
> extended the l1_size check from VMDK4 to VMDK3 but did not update the
> default coverage in the moved comment.
>
> The previous vmdk4 calculation:
>
> (512 * 1024 * 1024) * 512(l2 entries) * 65536(grain) = 16PB
>
> The added vmdk3 calculation:
>
> (512 * 1024 * 1024) * 4096(l2 entries) * 512(grain) = 1PB
>
> Adding the calculation of vmdk3 to the comment.
>
> In any case, VMware does not offer virtual disks more than 2TB for
> vmdk4/vmdk3 or 64TB for the new undocumented seSparse format which is
> not implemented yet in qemu.
>
> Reviewed-by: Karl Heubaum <address@hidden>
> Reviewed-by: Eyal Moscovici <address@hidden>
> Reviewed-by: Liran Alon <address@hidden>
> Reviewed-by: Arbel Moshe <address@hidden>
> Signed-off-by: Sam Eiderman <address@hidden>
> ---
> block/vmdk.c | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature