From 2ae7a868b867289b9591d662972f559d412315c3 Mon Sep 17 00:00:00 2001
From: Lukas Oesch <lukasoesch20@gmail.com>
Date: Tue, 3 Mar 2026 14:11:49 +0100
Subject: refactoring read and write block calculations

---
 .../devices/src/storage/RAMDisk/RAMDiskDevice.cpp  | 44 +++++++++++-----------
 1 file changed, 23 insertions(+), 21 deletions(-)

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

diff --git a/kernel/devices/src/storage/RAMDisk/RAMDiskDevice.cpp b/kernel/devices/src/storage/RAMDisk/RAMDiskDevice.cpp
index 627641c..f3b8799 100644
--- a/kernel/devices/src/storage/RAMDisk/RAMDiskDevice.cpp
+++ b/kernel/devices/src/storage/RAMDisk/RAMDiskDevice.cpp
@@ -11,12 +11,17 @@
 
 namespace devices::storage::ram_disk
 {
+  namespace
+  {
+    constexpr size_t RAM_DISK_BLOCK_SIZE = 512uz;  // TODO BA-FS26 really correct / good??
+  }  // namespace
+
   ram_disk_device::ram_disk_device()  // TODO BA-FS26 remove when kstd::vector is available
-      : block_device(0, 0)
+      : block_device(0, 0, RAM_DISK_BLOCK_SIZE)
   {}
 
   ram_disk_device::ram_disk_device(kapi::boot_modules::boot_module const & module, size_t major, size_t minor)
-      : block_device(major, minor)
+      : block_device(major, minor, RAM_DISK_BLOCK_SIZE)
       , m_boot_module(module)
   {}
 
@@ -27,21 +32,17 @@ namespace devices::storage::ram_disk
       kapi::system::panic("[RAM DISK DEVICE] read_block called with null buffer.");
     }
 
-    size_t const offset = block_index * block_size;
-    size_t const limit = m_boot_module.size;
+    auto const info = calculate_transfer(block_index);
 
-    size_t const available = (offset < limit) ? (limit - offset) : 0;
-    size_t const to_copy = (available < block_size) ? available : block_size;
-
-    if (to_copy > 0)
+    if (info.to_transfer > 0)
     {
-      auto const source = static_cast<std::byte const *>(m_boot_module.start_address) + offset;
-      kstd::libc::memcpy(buffer, source, to_copy);
+      auto const src = static_cast<std::byte const *>(m_boot_module.start_address) + info.offset;
+      kstd::libc::memcpy(buffer, src, info.to_transfer);
     }
 
-    if (to_copy < block_size)
+    if (info.remainder > 0)
     {
-      kstd::libc::memset(static_cast<std::byte *>(buffer) + to_copy, std::byte{0}, block_size - to_copy);
+      kstd::libc::memset(static_cast<std::byte *>(buffer) + info.to_transfer, 0, info.remainder);
     }
   }
 
@@ -52,16 +53,17 @@ namespace devices::storage::ram_disk
       kapi::system::panic("[RAM DISK DEVICE] write_block called with null buffer.");
     }
 
-    size_t const offset = block_index * block_size;
-    size_t const limit = m_boot_module.size;
-
-    if (offset >= limit)
-      return;
+    auto const info = calculate_transfer(block_index);
 
-    size_t const available = limit - offset;
-    size_t const to_write = (available < block_size) ? available : block_size;
+    if (info.to_transfer > 0)
+    {
+      auto const dest = static_cast<std::byte *>(m_boot_module.start_address) + info.offset;
+      kstd::libc::memcpy(dest, buffer, info.to_transfer);
+    }
+  }
 
-    auto const destination = static_cast<std::byte *>(m_boot_module.start_address) + offset;
-    kstd::libc::memcpy(destination, buffer, to_write);
+  auto ram_disk_device::size() const -> size_t
+  {
+    return m_boot_module.size;
   }
 }  // namespace devices::storage::ram_disk
\ No newline at end of file
-- 
cgit v1.2.3