Adjust bump allocator comment

3 files changed, 4 insertions, 3 deletions

diff --git a/arch/x86_64/include/arch/memory/heap/bump_allocator.hpp b/arch/x86_64/include/arch/memory/heap/bump_allocator.hpp
index 5b581ba..d31783d 100644
--- a/arch/x86_64/include/arch/memory/heap/bump_allocator.hpp
+++ b/arch/x86_64/include/arch/memory/heap/bump_allocator.hpp
@@ -6,7 +6,8 @@
 namespace teachos::arch::memory::heap
 {
   /**
-   * @brief Simple heap allocator, which leaks all allocated memory, because it does not really deallocate anything.
+   * @brief Simple heap allocator, which allocates linearly and leaks all allocated memory, because it does not really
+   * deallocate anything.
    */
   struct bump_allocator
   {
diff --git a/arch/x86_64/src/kernel/main.cpp b/arch/x86_64/src/kernel/main.cpp
index f4eb033..13526f4 100644
--- a/arch/x86_64/src/kernel/main.cpp
+++ b/arch/x86_64/src/kernel/main.cpp
@@ -31,5 +31,7 @@ namespace teachos::arch::kernel
     // allocator.allocate_frame still works?
     // TODO: Fix unmapping old level 4 page table and turn it into guard page, use Stack Probes for stack allocation if
     // possible.
+
+    // TODO: Align up and down for the bump allocator. https://os.phil-opp.com/kernel-heap/#a-bump-allocator
   }
 }  // namespace teachos::arch::kernel
diff --git a/arch/x86_64/src/memory/heap/bump_allocator.cpp b/arch/x86_64/src/memory/heap/bump_allocator.cpp
index 486ece8..1ab8ea9 100644
--- a/arch/x86_64/src/memory/heap/bump_allocator.cpp
+++ b/arch/x86_64/src/memory/heap/bump_allocator.cpp
@@ -6,8 +6,6 @@ namespace teachos::arch::memory::heap
 {
   auto bump_allocator::allocate(std::size_t size) -> void *
   {
-    // Uses some sort of alignment orignally:
-    // https://github.com/phil-opp/blog_os/blob/7f6576c9dc34e360b81236c54c25c7827fd6a2df/src/memory/heap_allocator.rs#L24
     auto alloc_start = next;
     auto alloc_end = next + size;