Re: [PATCH v2] libxl: memory size in kb requires 64 bit variable

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1531 bytes --]

On Fri, 2016-07-22 at 08:53 +0200, Juergen Gross wrote:
> libxl_set_memory_target() and several other interface functions of
> libxl use a 32 bit sized parameter for a memory size value in kBytes.
> This limits the maximum size to be passed in such a parameter
> depending on signedness of the parameter to 2TB or 4TB.
> 
> Correct this by using 64 bit types.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>
>
Reviewed-by: Dario Faggioli <dario.faggioli@citrix.com>

Entirely out of folklore, it seems to me that this function...

> @@ -5057,8 +5057,28 @@ out:
>      return rc;
>  }
>  
> +static int libxl__memkb_32to64(libxl_ctx *ctx, int rc,
> +                               uint64_t val64, uint32_t *ptr32)
> +{
> +    GC_INIT(ctx);
> +
> +    if (rc)
> +        goto out;
> +
> +    *ptr32 = val64;
> +    if (*ptr32 == val64)
> +        goto out;
> +
> +    LOGE(ERROR, "memory size %"PRIu64" too large for 32 bit
> value\n", val64);
> +    rc = ERROR_FAIL;
> +
> +out:
> +    GC_FREE;
> +    return rc;
> +}
> +
...does more 64to32 then 32to64, but maybe that's me, and, anyway, it's
certainly not a big deal.

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel