mbox series

[v4,0/4] xen: arm: Split MMU code in preparation for MPU work (part 2)

Message ID 20240821122503.2315844-1-ayan.kumar.halder@amd.com (mailing list archive)
Headers show
Series xen: arm: Split MMU code in preparation for MPU work (part 2) | expand

Message

Ayan Kumar Halder Aug. 21, 2024, 12:24 p.m. UTC 
Hi,

In https://patchew.org/Xen/20231116145032.1651305-1-Henry.Wang@arm.com/, Henry has
reorganized some of the code between the MMU specific and generic files.

In this patch serie, we address the remaining code reorg so that MMU specific
code is cleanly separated and we have added stubs wherever necessary to avoid
introducing if-def.

Changes from :-

v1..v2 - mentioned in individual patches.
v3 - Reordered the patches so that first 3 can be committed (as they have been
reviewed).

Ayan Kumar Halder (4):
  xen: arm: Add a new helper update_boot_mapping()
  xen: arm: Move domain_page.c to MMU specific
  xen: arm: Enclose access to EL2 MMU specific registers under
    CONFIG_MMU
  xen: make VMAP support in MMU system only

 xen/arch/arm/Kconfig                 |  4 +++-
 xen/arch/arm/Makefile                |  1 -
 xen/arch/arm/arm64/mmu/mm.c          | 14 +++++++++++++-
 xen/arch/arm/arm64/smpboot.c         |  6 +++---
 xen/arch/arm/include/asm/arm64/mm.h  |  9 +--------
 xen/arch/arm/mmu/Makefile            |  1 +
 xen/arch/arm/{ => mmu}/domain_page.c |  0
 xen/arch/arm/setup.c                 |  2 ++
 xen/arch/arm/traps.c                 | 10 ++++++++++
 xen/arch/ppc/Kconfig                 |  1 +
 xen/arch/riscv/Kconfig               |  1 +
 xen/arch/x86/Kconfig                 |  1 +
 xen/common/Kconfig                   |  3 +++
 xen/common/Makefile                  |  2 +-
 xen/common/vmap.c                    |  2 --
 xen/include/xen/vmap.h               |  2 ++
 16 files changed, 42 insertions(+), 17 deletions(-)
 rename xen/arch/arm/{ => mmu}/domain_page.c (100%)

Comments

Julien Grall Aug. 22, 2024, 10:37 a.m. UTC | #1 
Hi Ayan,

On 21/08/2024 13:24, Ayan Kumar Halder wrote:
> Hi,
> 
> In https://patchew.org/Xen/20231116145032.1651305-1-Henry.Wang@arm.com/, Henry has
> reorganized some of the code between the MMU specific and generic files.
> 
> In this patch serie, we address the remaining code reorg so that MMU specific
> code is cleanly separated and we have added stubs wherever necessary to avoid
> introducing if-def.
> 
> Changes from :-
> 
> v1..v2 - mentioned in individual patches.
> v3 - Reordered the patches so that first 3 can be committed (as they have been
> reviewed).
> 
> Ayan Kumar Halder (4):
>    xen: arm: Add a new helper update_boot_mapping()
>    xen: arm: Move domain_page.c to MMU specific
>    xen: arm: Enclose access to EL2 MMU specific registers under
>      CONFIG_MMU

I have committed the above 3 patches. AFAICT...

>    xen: make VMAP support in MMU system only

... this patch may require a respin and definitely needs more acks.

Cheers,