Re: [PATCH] xen/arm: mm: Encode existing constraints of the memory layout

[Julien Grall <julien@xxxxxxx>]

On 03/03/2022 16:04, Bertrand Marquis wrote:
> Hi Julien,
Hi Bertrand,

> > On 28 Feb 2022, at 10:06, Julien Grall <julien@xxxxxxx> wrote:
> >
> > From: Julien Grall <jgrall@xxxxxxxxxx>
> >
> > The boot code expects the regions XEN_VIRT_START, FIXMAP_ADDR(0),
> > BOOT_FDT_VIRT_START to use the same 0th (arm64 only) and 1st slot.
> >
> > Add some BUILD_BUG_ON() to confirm that. This is helpful if one wants
> > to re-order the memory layout.
> Very good idea :-)
I actually hit this issue when trying to re-order the memory layout for 
testing a patch.
> > Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
>
> Just a small NIT after if you want to do it on commit...
I have update the comment and committed the patch.

Thanks!

Cheers,

--
Julien Grall