[RFC PATCH 0/2] xen/arm: Remove recursion from dt_unreserved_regions

Luca Fancellu posted 2 patches 1 week, 2 days ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20240425131119.2299629-1-luca.fancellu@arm.com
xen/arch/arm/arm32/mmu/mm.c             |  29 +----
xen/arch/arm/bootfdt.c                  |  51 +++++---
xen/arch/arm/domain_build.c             |   3 +-
xen/arch/arm/include/asm/setup.h        |  13 +-
xen/arch/arm/include/asm/static-shmem.h |   7 +-
xen/arch/arm/kernel.c                   |   2 +-
xen/arch/arm/setup.c                    | 158 +++++++++++++-----------
7 files changed, 135 insertions(+), 128 deletions(-)
[RFC PATCH 0/2] xen/arm: Remove recursion from dt_unreserved_regions
Posted by Luca Fancellu 1 week, 2 days ago
Hi, this is an RFC that is exploiting the new 'struct membank' interface
introduced here 2209c1e35b479dff8ed3d3161001ffdefa0a704e ("xen/arm: Introduce a
generic way to access memory bank structures") to start removing recursion
from some function, in this serie the dt_unreserved_regions is reworked for
that reason.

This is an RFC to understand if the proposed approach can be accepted.

Another function can benefit from this approach, consider_modules in
arm32/mmu/mm.c, but it might require to rework also the 'struct bootmodules'
to adhere to the new interface and have just a loop that can go through all the
different structures.

Comments are welcome.

Luca Fancellu (2):
  xen/arm: Add DT reserve map regions to bootinfo.reserved_mem
  xen/arm: Rework dt_unreserved_regions to avoid recursion

 xen/arch/arm/arm32/mmu/mm.c             |  29 +----
 xen/arch/arm/bootfdt.c                  |  51 +++++---
 xen/arch/arm/domain_build.c             |   3 +-
 xen/arch/arm/include/asm/setup.h        |  13 +-
 xen/arch/arm/include/asm/static-shmem.h |   7 +-
 xen/arch/arm/kernel.c                   |   2 +-
 xen/arch/arm/setup.c                    | 158 +++++++++++++-----------
 7 files changed, 135 insertions(+), 128 deletions(-)

-- 
2.34.1