From 5801be615a50bf465a9663b7f75cafbcf0870f5c Mon Sep 17 00:00:00 2001
From: Lukas Oesch <lukasoesch20@gmail.com>
Date: Tue, 17 Mar 2026 11:49:13 +0100
Subject: use kstd::vector instead of std::array and replace plain-pointers
 with kstd::shared_ptr

---
 kernel/devices/src/storage/storage_management.cpp | 25 ++++++++++++-----------
 1 file changed, 13 insertions(+), 12 deletions(-)

(limited to 'kernel/devices/src/storage/storage_management.cpp')

diff --git a/kernel/devices/src/storage/storage_management.cpp b/kernel/devices/src/storage/storage_management.cpp
index e1f1bcc..00449fb 100644
--- a/kernel/devices/src/storage/storage_management.cpp
+++ b/kernel/devices/src/storage/storage_management.cpp
@@ -7,8 +7,10 @@
 #include "devices/storage/ram_disk/ram_disk_controller.hpp"
 #include "devices/storage/storage_controller.hpp"
 
+#include <kstd/memory>
+#include <kstd/vector>
+
 #include <algorithm>
-#include <array>
 #include <cstddef>
 #include <optional>
 
@@ -21,8 +23,6 @@ namespace devices::storage
     constinit size_t static next_free_major = START_MAJOR;
 
     constinit auto static active_storage_management = std::optional<storage_management>{};
-    // TODO BA-FS26 remove again, when dynamic memory available
-    constinit auto static active_ram_disk_controller = std::optional<ram_disk::ram_disk_controller>{};
   }  // namespace
 
   auto storage_management::init() -> void
@@ -33,8 +33,9 @@ namespace devices::storage
     }
     active_storage_management.emplace(storage_management{});
 
-    active_ram_disk_controller.emplace(&kapi::boot_modules::get_boot_module_registry());
-    active_storage_management->add_controller(&active_ram_disk_controller.value());
+    auto current_ram_disk_controller =
+        kstd::make_shared<ram_disk::ram_disk_controller>(&kapi::boot_modules::get_boot_module_registry());
+    active_storage_management->add_controller(current_ram_disk_controller);
 
     std::ranges::for_each(active_storage_management->m_controllers, [](auto controller) { controller->probe(); });
   }
@@ -49,22 +50,22 @@ namespace devices::storage
     return *active_storage_management;
   }
 
-  auto storage_management::add_controller(storage_controller * controller) -> void
+  auto storage_management::add_controller(kstd::shared_ptr<storage_controller> controller) -> void
   {
     controller->set_ids(next_free_major++, MINORS_PER_DEVICE);
-    m_controllers.at(0) = controller;  // TODO BA-FS26 use push_back from kstd:vector
+    m_controllers.push_back(controller);
   }
 
-  auto storage_management::all_controllers() const -> std::array<storage_controller *, 1> const &
+  auto storage_management::all_controllers() const -> kstd::vector<kstd::shared_ptr<storage_controller>> const &
   {
     return m_controllers;
   }
 
-  auto storage_management::device_by_major_minor(size_t major, size_t minor) -> device *
+  auto storage_management::device_by_major_minor(size_t major, size_t minor) -> kstd::shared_ptr<device>
   {
-    device * found = nullptr;
+    kstd::shared_ptr<device> found = nullptr;
 
-    std::ranges::find_if(m_controllers, [&](auto const controller) {
+    std::ranges::find_if(m_controllers, [&](auto const & controller) {
       if (controller != nullptr && controller->major() == major)
       {
         found = controller->device_by_minor(minor);
@@ -76,7 +77,7 @@ namespace devices::storage
     return found;
   }
 
-  auto storage_management::determine_boot_device() -> device *
+  auto storage_management::determine_boot_device() -> kstd::shared_ptr<device>
   {
     return device_by_major_minor(START_MAJOR, 0);
   }
-- 
cgit v1.2.3