From ee4c61f7313fedd23d01c69ea5036fa38ef6248a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matteo=20Gm=C3=BCr?= <matteo.gmuer1@ost.ch>
Date: Mon, 12 May 2025 08:50:12 +0000
Subject: Adjust user heap to lazy allocate heap

---
 arch/x86_64/src/memory/heap/global_heap_allocator.cpp |  2 +-
 arch/x86_64/src/memory/heap/linked_list_allocator.cpp |  4 +---
 arch/x86_64/src/memory/heap/memory_block.cpp          |  4 ++--
 arch/x86_64/src/memory/heap/user_heap_allocator.cpp   | 14 --------------
 arch/x86_64/src/memory/main.cpp                       |  4 ----
 5 files changed, 4 insertions(+), 24 deletions(-)

(limited to 'arch/x86_64/src/memory')

diff --git a/arch/x86_64/src/memory/heap/global_heap_allocator.cpp b/arch/x86_64/src/memory/heap/global_heap_allocator.cpp
index 23e2458..e6c268a 100644
--- a/arch/x86_64/src/memory/heap/global_heap_allocator.cpp
+++ b/arch/x86_64/src/memory/heap/global_heap_allocator.cpp
@@ -41,7 +41,7 @@ namespace teachos::arch::memory::heap
     }
 
     [[gnu::section(".user_data")]]
-    static user_heap_allocator user_allocator{USER_HEAP_START, USER_HEAP_START + USER_HEAP_SIZE};
+    static user_heap_allocator user_allocator{};
     user_allocator_instance = &user_allocator;
   }
 
diff --git a/arch/x86_64/src/memory/heap/linked_list_allocator.cpp b/arch/x86_64/src/memory/heap/linked_list_allocator.cpp
index 5101ab2..63a6111 100644
--- a/arch/x86_64/src/memory/heap/linked_list_allocator.cpp
+++ b/arch/x86_64/src/memory/heap/linked_list_allocator.cpp
@@ -8,9 +8,7 @@
 namespace teachos::arch::memory::heap
 {
   linked_list_allocator::linked_list_allocator(std::size_t heap_start, std::size_t heap_end)
-      : heap_start(heap_start)
-      , heap_end(heap_end)
-      , first(nullptr)
+      : first(nullptr)
       , mutex{stl::mutex{}}
   {
     auto const heap_size = heap_end - heap_start;
diff --git a/arch/x86_64/src/memory/heap/memory_block.cpp b/arch/x86_64/src/memory/heap/memory_block.cpp
index 9f1fea8..6ee675a 100644
--- a/arch/x86_64/src/memory/heap/memory_block.cpp
+++ b/arch/x86_64/src/memory/heap/memory_block.cpp
@@ -7,10 +7,10 @@ namespace teachos::arch::memory::heap
   memory_block::memory_block(std::size_t size, memory_block * next)
   {
     // TODO: Figure out why this memset fails
-    memset(static_cast<void *>(this), 0, size);
+    // memset(static_cast<void *>(this), 0, size);
     this->size = size;
     this->next = next;
   }
 
-  memory_block::~memory_block() { memset(static_cast<void *>(this), 0, sizeof(memory_block)); }
+  memory_block::~memory_block() { /*memset(static_cast<void *>(this), 0, sizeof(memory_block));*/ }
 }  // namespace teachos::arch::memory::heap
diff --git a/arch/x86_64/src/memory/heap/user_heap_allocator.cpp b/arch/x86_64/src/memory/heap/user_heap_allocator.cpp
index ce8b0fa..9cb6c17 100644
--- a/arch/x86_64/src/memory/heap/user_heap_allocator.cpp
+++ b/arch/x86_64/src/memory/heap/user_heap_allocator.cpp
@@ -8,20 +8,6 @@
 
 namespace teachos::arch::memory::heap
 {
-  user_heap_allocator::user_heap_allocator(std::size_t heap_start, std::size_t heap_end)
-      : heap_start(heap_start)
-      , heap_end(heap_end)
-      , first(nullptr)
-      , mutex{stl::mutex{}}
-  {
-    auto const heap_size = heap_end - heap_start;
-    exception_handling::assert(
-        heap_size > min_allocatable_size(),
-        "[Linked List Allocator] Total heap size can not be smaller than minimum of 16 bytes to hold "
-        "atleast one memory hole entry");
-    // first = new (reinterpret_cast<void *>(heap_start)) memory_block(heap_size, nullptr);
-  }
-
   auto user_heap_allocator::allocate(std::size_t size) -> void *
   {
     // Add size of size_t to the total allocated size, because we add a header that includes the size of the allocated
diff --git a/arch/x86_64/src/memory/main.cpp b/arch/x86_64/src/memory/main.cpp
index 595cb0d..5e671ac 100644
--- a/arch/x86_64/src/memory/main.cpp
+++ b/arch/x86_64/src/memory/main.cpp
@@ -69,11 +69,7 @@ namespace teachos::arch::memory
     video::vga::text::write("Kernel remapping successful", video::vga::text::common_attributes::green_on_black);
     video::vga::text::newline();
 
-    // Remap kernel heap
     remap_heap(heap::KERNEL_HEAP_START, heap::KERNEL_HEAP_SIZE);
-    // Remap user heap
-    remap_heap(heap::USER_HEAP_SIZE, heap::USER_HEAP_SIZE, paging::entry::USER_ACCESSIBLE);
-
     video::vga::text::write("Heap remapping successful", video::vga::text::common_attributes::green_on_black);
     video::vga::text::newline();
   }
-- 
cgit v1.2.3