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

[Viktor Mitin <viktor.mitin.19@xxxxxxxxx>]

Hi Julien,

On Tue, Jul 9, 2019 at 3:29 PM Julien Grall <julien.grall@xxxxxxx> wrote:
>
> 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.
>

You are right. Please ignore this patch.
I will add extra 0x0 to dom0less documentation instead.

Thanks

> Regards,
>
> --
> Julien Grall

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