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

Ayan Kumar Halder posted 4 patches 3 months ago
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%)
[PATCH v4 0/4] xen: arm: Split MMU code in preparation for MPU work (part 2)
Posted by Ayan Kumar Halder 3 months ago
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%)

-- 
2.25.1
Re: [PATCH v4 0/4] xen: arm: Split MMU code in preparation for MPU work (part 2)
Posted by Julien Grall 3 months ago
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,

-- 
Julien Grall