Re: [Xen-devel] [PATCH] xen/arm: change DomU memory property size from u64 to u32

[Julien Grall <julien.grall@xxxxxxx>]

Hi Viktor,

On 7/9/19 8:49 AM, Viktor Mitin wrote:
> While checking xen dom0less documentation it has been found
> that domU memory property requires extra 0x0 in case of arm64.
And this matches the binding docs/misc/arm/device-tree/booting.txt which 
requires a 64-bit value.
> There is no need to keep memory size in u64, 32 bits is
> enough for domU memory size.
Definitely not. The memory is given in KB so 32-bits only allows to 
cover 4TB. While I agree that none of the DomU created by Xen will be 
bigger than a few gigabytes today, user exposed interfaces should always 
cater for the future.
In any case, the interface changes you suggest is going to break user 
properly using the bindings.
Regards,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel