Re: [PATCH v2 1/3] xen/riscv: introduce setup_initial_pages

[Julien Grall <julien@xxxxxxx>]

Hi,

On 22/03/2023 08:19, Jan Beulich wrote:
> On 21.03.2023 18:58, Julien Grall wrote:
> > Also, where do you guarantee that Xen will be loaded at a 2MB aligned
> > address? (For a fact I know that UEFI is only guarantee 4KB alignment).
> I don't think this is true. We rely on UEFI honoring larger alignment on
> x86, and I believe the PE loader code is uniform across architectures.
> Of course you need to specify large enough alignment in the PE header's
> SectionAlignment field (right now in the Arm64 binary it's only 4k).
Hmmm... I have always been told this was ignored. Hence why we need to 
deal with 4KB. In any case, the problem stay the same. We didn't seem to 
have documented the expected alignment.
So in theory anyone can load RISC-V at any address. My suggestion would 
be to create docs/misc/riscv/booting.txt and document how one is meant 
to load Xen.
Cheers,

--
Julien Grall