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 <karl.heub...@oracle.com>
> Reviewed-by: Eyal Moscovici <eyal.moscov...@oracle.com>
> Reviewed-by: Liran Alon <liran.a...@oracle.com>
> Reviewed-by: Arbel Moshe <arbel.mo...@oracle.com>
> Signed-off-by: Sam Eiderman <shmuel.eider...@oracle.com>
> ---
> block/vmdk.c | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)Reviewed-by: Max Reitz <mre...@redhat.com>
signature.asc
Description: OpenPGP digital signature
