[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.