libsupport_zynq/ram: split allocators for two cores.

libsupport_zynq/Cargo.toml

@@ -10,6 +10,7 @@ target_zc706 = ["libboard_zynq/target_zc706"]
 target_cora_z7_10 = ["libboard_zynq/target_cora_z7_10"]
 panic_handler = []
 dummy_irq_handler = []
+alloc_core = []
 
 default = ["panic_handler", "dummy_irq_handler"]
 

libsupport_zynq/src/ram.rs

@@ -1,55 +1,100 @@
-use core::alloc::GlobalAlloc;
-use core::ptr::NonNull;
 use alloc::alloc::Layout;
-use linked_list_allocator::Heap;
+use core::alloc::GlobalAlloc;
-use libcortex_a9::mutex::Mutex;
+use core::cell::UnsafeCell;
+use core::ptr::NonNull;
 use libboard_zynq::ddr::DdrRam;
+use libcortex_a9::regs::MPIDR;
+use libregister::RegisterR;
+use linked_list_allocator::Heap;
 
 #[global_allocator]
-static ALLOCATOR: CortexA9Alloc = CortexA9Alloc(Mutex::new(Heap::empty()));
+static mut ALLOCATOR: CortexA9Alloc = CortexA9Alloc(
+    UnsafeCell::new(Heap::empty()),
+    UnsafeCell::new(Heap::empty()),
+);
 
-/// LockedHeap doesn't lock properly
+struct CortexA9Alloc(UnsafeCell<Heap>, UnsafeCell<Heap>);
-struct CortexA9Alloc(Mutex<Heap>);
 
 unsafe impl Sync for CortexA9Alloc {}
 
 unsafe impl GlobalAlloc for CortexA9Alloc {
     unsafe fn alloc(&self, layout: Layout) -> *mut u8 {
-        self.0.lock()
+        if MPIDR.read().cpu_id() == 0 {
+            self.0.get().as_mut()
+        } else {
+            self.1.get().as_mut()
+        }
+        .unwrap()
         .allocate_first_fit(layout)
         .ok()
         .map_or(0 as *mut u8, |allocation| allocation.as_ptr())
     }
 
     unsafe fn dealloc(&self, ptr: *mut u8, layout: Layout) {
-        self.0.lock()
+        if MPIDR.read().cpu_id() == 0 {
+            self.0.get().as_mut()
+        } else {
+            self.1.get().as_mut()
+        }
+        .unwrap()
         .deallocate(NonNull::new_unchecked(ptr), layout)
     }
 }
 
+#[cfg(not(feature = "alloc_core"))]
 pub fn init_alloc_ddr(ddr: &mut DdrRam) {
     unsafe {
-        ALLOCATOR.0.lock()
+        ALLOCATOR
+            .0
+            .get()
+            .as_mut()
+            .unwrap()
             .init(ddr.ptr::<u8>() as usize, ddr.size());
     }
 }
 
+#[cfg(feature = "alloc_core")]
 extern "C" {
-    static __heap_start: usize;
+    static __heap0_start: usize;
-    static __heap_end: usize;
+    static __heap0_end: usize;
+    static __heap1_start: usize;
+    static __heap1_end: usize;
 }
 
-pub fn init_alloc_linker() {
+#[cfg(feature = "alloc_core")]
+pub fn init_alloc_core0() {
     unsafe {
-        let start = &__heap_start as *const usize as usize;
+        let start = &__heap0_start as *const usize as usize;
-        let end = &__heap_end as *const usize as usize;
+        let end = &__heap0_end as *const usize as usize;
-        ALLOCATOR.0.lock()
+        ALLOCATOR.0.get().as_mut().unwrap().init(start, end - start);
-            .init(start, end - start);
     }
 }
 
+#[cfg(feature = "alloc_core")]
+pub fn init_alloc_core1() {
+    unsafe {
+        let start = &__heap1_start as *const usize as usize;
+        let end = &__heap1_end as *const usize as usize;
+        ALLOCATOR.1.get().as_mut().unwrap().init(start, end - start);
+    }
+}
 
 #[alloc_error_handler]
-fn alloc_error(_: core::alloc::Layout) -> ! {
+fn alloc_error(layout: core::alloc::Layout) -> ! {
-    panic!("alloc_error")
+    let id = MPIDR.read().cpu_id();
+    let heap = unsafe {
+        if id == 0 {
+            ALLOCATOR.0.get()
+        } else {
+            ALLOCATOR.1.get()
+        }
+        .as_mut()
+        .unwrap()
+    };
+    panic!(
+        "Core {} alloc_error, layout: {:?}, used memory: {}",
+        id,
+        layout,
+        heap.used()
+    );
 }