On 14/10/2024 20:11, Luca Fancellu wrote:
Hi Ayan,

Hi Luca,

Sorry I missed something.


/*
  * Maps the various sections of Xen (described in xen.lds.S) as different MPU
  * regions.
@@ -68,10 +92,11 @@
  * Inputs:
  *   lr : Address to return to.
  *
- * Clobbers x0 - x5
+ * Clobbers x0 - x6
  *
  */
FUNC(enable_boot_cpu_mm)
+    mov   x6, lr

     /* Check if the number of regions exceeded the count specified in 
MPUIR_EL2 */
     mrs   x5, MPUIR_EL2
@@ -113,7 +138,7 @@ FUNC(enable_boot_cpu_mm)
     beq 5f
     prepare_xen_region x0, x1, x2, x3, x4, x5
bl    enable_mpu
-5:
+5:  mov   lr, x6
Shall these changes to enable_boot_cpu_mm be part of the previous commit?

This is why I had to save and restore the LR.

- Ayan


Reply via email to