Re: [PATCH v6 13/15] xen/arm: make consider_modules() available for xen relocation

[Julien Grall <julien@xxxxxxx>]

(+ Ayan + Henry)

Hi Carlo,

On 29/01/2024 17:18, Carlo Nonato wrote:
> Cache coloring must physically relocate Xen in order to color the hypervisor
> and consider_modules() is a key function that is needed to find a new
> available physical address.
>
> 672d67f339c0 ("xen/arm: Split MMU-specific setup_mm() and related code out")
> moved consider_modules() under arm32. Move it back to setup.c and make it
> non-static so that it can be used outside.
>
> Signed-off-by: Carlo Nonato <carlo.nonato@xxxxxxxxxxxxxxx>
> ---
> v6:
> - new patch
> ---
>   xen/arch/arm/arm32/mmu/mm.c      | 93 +-------------------------------
>   xen/arch/arm/include/asm/setup.h |  3 ++
>   xen/arch/arm/setup.c             | 92 +++++++++++++++++++++++++++++++
Ayan, Henry, will the function consider_modules() be used for the MPU 
code? If not, then I think the function should live in arm/mmu/setup.c.
Cheers,

--
Julien Grall