fix(linker): fix broken linker symbols

This commit addresses issues introduced on linker symbols when
MEM_NON_UNIFIED and .datanocopy were added.

- Add _data_(vma/lma)_end symbol to get the address of the data
end address used during the data copy routine of non-unified
platforms
- Move _image_load_end to the original location right before the
vm_images, allowing .datanocopy to be correctly mapped in bao's
first continuous region
- Force _dmem_phys_beg calculation to be related to the type of
memory protection (mpu/mmu) of the platform and not the memory
type (unified/non-unified)

Signed-off-by: Daniel Oliveira <[email protected]>

Author: danielRep

src/arch/armv8/aarch32/boot.S

@@ -159,7 +159,7 @@ _set_master_cpu:
     /* Copy data from RX to RWX */
     ldr r7, =_data_lma_start        // LMA start
     ldr r11, =_data_vma_start       // VMA start
-    ldr r12, =_image_load_end       // LMA end
+    ldr r12, =_data_lma_end         // LMA end
     bl copy_data
 
     ldr r1, =_data_vma_start

src/linker.ld

@@ -56,7 +56,7 @@ SECTIONS
         _ipi_cpumsg_handlers_id_end = .;
 	}
 
-    _image_load_end = .;
+    _data_vma_end = .;
 
 	.datanocopy : ALIGN(PAGE_SIZE) {
 		/**
@@ -71,9 +71,12 @@ SECTIONS
 	    . = ALIGN(PAGE_SIZE);
 	}
 
+	_image_load_end = .;
+
 #ifdef MEM_NON_UNIFIED
 	/* Save the current location counter (VMA) and switch to LMA for .vm_images */
 	_vma_before_vm_images = .;
+	_data_lma_end =  ALIGN((_data_lma_start + (_data_vma_end - _data_vma_start)), PAGE_SIZE);
     _image_load_end = ALIGN((_data_lma_start + (_image_load_end - _data_vma_start)), PAGE_SIZE);
 	. = _image_load_end;
 #endif
@@ -122,8 +125,9 @@ SECTIONS
 	. = ALIGN(PAGE_SIZE);
 	_image_end = ABSOLUTE(.);
 
+#ifdef MEM_PROT_MMU
 	_dmem_phys_beg = ABSOLUTE(.) + extra_allocated_phys_mem;
-#ifdef MEM_NON_UNIFIED
+#else
 	_dmem_phys_beg = ABSOLUTE(.);
 #endif