remove inode metadata

5 files changed, 56 insertions, 92 deletions

diff --git a/kernel/src/filesystem/device_inode.cpp b/kernel/src/filesystem/device_inode.cpp
index b9018b1..18e3b24 100644
--- a/kernel/src/filesystem/device_inode.cpp
+++ b/kernel/src/filesystem/device_inode.cpp
@@ -1,6 +1,52 @@
 #include "kernel/filesystem/device_inode.hpp"
 
+#include "kapi/system.hpp"
+
+#include "kernel/devices/device.hpp"
+#include "kernel/filesystem/inode.hpp"
+
+#include <kstd/memory>
+
+#include <cstddef>
+
 namespace filesystem
 {
-  // TODO BA-FS26: Implement device inode
-}
\ No newline at end of file
+  device_inode::device_inode(kstd::shared_ptr<devices::device> const & device)
+      : inode(inode_kind::device)
+      , m_device(device)
+  {
+    if (!device)
+    {
+      kapi::system::panic("[FILESYSTEM] device_inode constructed with null device.");
+    }
+  }
+
+  auto device_inode::backing_device() const -> kstd::shared_ptr<devices::device> const &
+  {
+    return m_device;
+  }
+
+  auto device_inode::read(void * /*buffer*/, size_t /*offset*/, size_t /*size*/) const -> size_t
+  {
+    if (!m_device)
+    {
+      kapi::system::panic("[FILESYSTEM] device_inode has null device.");
+    }
+
+    // TODO BA-FS26 use device file?
+    // return m_device->read(buffer, offset, size);
+    return 0;
+  }
+
+  auto device_inode::write(void const * /*buffer*/, size_t /*offset*/, size_t /*size*/) -> size_t
+  {
+    if (!m_device)
+    {
+      kapi::system::panic("[FILESYSTEM] device_inode has null device.");
+    }
+
+    // TODO BA-FS26 use device file?
+    // return m_device->write(buffer, offset, size);
+    return 0;
+  }
+}  // namespace filesystem
\ No newline at end of file
diff --git a/kernel/src/filesystem/ext2/ext2_filesystem.cpp b/kernel/src/filesystem/ext2/ext2_filesystem.cpp
index 408b292..87845d5 100644
--- a/kernel/src/filesystem/ext2/ext2_filesystem.cpp
+++ b/kernel/src/filesystem/ext2/ext2_filesystem.cpp
@@ -2,7 +2,6 @@
 
 #include "kernel/devices/device.hpp"
 #include "kernel/filesystem/inode.hpp"
-#include "kernel/filesystem/inode_metadata.hpp"
 
 #include <kstd/memory>
 
@@ -19,7 +18,7 @@ namespace filesystem::ext2
 
     m_device = device;
     // TODO BA-FS26 load proper root inode from ext2 metadata
-    m_root_inode = inode{inode_kind::directory};
+    // m_root_inode = inode{inode_kind::directory};
 
     // TODO BA-FS26 implement
     return 0;
diff --git a/kernel/src/filesystem/filesystem.cpp b/kernel/src/filesystem/filesystem.cpp
index 86b7940..50b5587 100644
--- a/kernel/src/filesystem/filesystem.cpp
+++ b/kernel/src/filesystem/filesystem.cpp
@@ -2,9 +2,11 @@
 
 #include "kernel/filesystem/inode.hpp"
 
+#include <kstd/memory>
+
 namespace filesystem
 {
-  auto filesystem::root_inode() const -> inode const &
+  auto filesystem::root_inode() const -> kstd::shared_ptr<inode> const &
   {
     return m_root_inode;
   }
diff --git a/kernel/src/filesystem/inode.cpp b/kernel/src/filesystem/inode.cpp
index af73662..de3282f 100644
--- a/kernel/src/filesystem/inode.cpp
+++ b/kernel/src/filesystem/inode.cpp
@@ -1,44 +1,11 @@
 #include "kernel/filesystem/inode.hpp"
 
-#include "kapi/system.hpp"
-
-#include "kernel/devices/device.hpp"
-#include "kernel/filesystem/inode_metadata.hpp"
-
-#include <kstd/memory>
-
-#include <cstddef>
-
 namespace filesystem
 {
   inode::inode(inode_kind kind)
       : m_kind(kind)
   {}
 
-  inode::inode(kstd::shared_ptr<devices::device> const & device)
-      : m_kind(inode_kind::device)
-      , m_device(device)
-  {
-    if (!m_device)
-    {
-      kapi::system::panic("[FILESYSTEM] inode constructed with null device.");
-    }
-  }
-
-  auto inode::metadata() const -> inode_metadata
-  {
-    auto meta = inode_metadata{};
-    meta.kind = m_kind;
-
-    if (is_device())
-    {
-      meta.major = m_device->major();
-      meta.minor = m_device->minor();
-    }
-
-    return meta;
-  }
-
   auto inode::is_directory() const -> bool
   {
     return m_kind == inode_kind::directory;
@@ -53,56 +20,4 @@ namespace filesystem
   {
     return m_kind == inode_kind::device;
   }
-
-  auto inode::is_block_device() const -> bool
-  {
-    return is_device() && m_device->is_block_device();
-  }
-
-  auto inode::major_device() const -> size_t
-  {
-    if (!is_device())
-    {
-      kapi::system::panic("[FILESYSTEM] inode::major_device called on non-device inode.");
-    }
-
-    return m_device->major();
-  }
-
-  auto inode::minor_device() const -> size_t
-  {
-    if (!is_device())
-    {
-      kapi::system::panic("[FILESYSTEM] inode::minor_device called on non-device inode.");
-    }
-
-    return m_device->minor();
-  }
-
-  auto inode::backing_device() const -> kstd::shared_ptr<devices::device> const &
-  {
-    return m_device;
-  }
-
-  auto inode::read(void * /*buffer*/, size_t /*offset*/, size_t /*size*/) const -> size_t
-  {
-    if (is_device())
-    {
-      kapi::system::panic("[FILESYSTEM] inode::read called on device inode. Open it as a device file first.");
-    }
-
-    // TODO BA-FS26
-    return 0;
-  }
-
-  auto inode::write(void const *, size_t, size_t) -> size_t
-  {
-    if (is_device())
-    {
-      kapi::system::panic("[FILESYSTEM] inode::write called on device inode. Open it as a device file first.");
-    }
-
-    kapi::system::panic("[FILESYSTEM] inode::write is not implemented yet");
-    return 0;
-  }
 }  // namespace filesystem
\ No newline at end of file
diff --git a/kernel/src/filesystem/vfs.cpp b/kernel/src/filesystem/vfs.cpp
index 4e0b6bf..188da6d 100644
--- a/kernel/src/filesystem/vfs.cpp
+++ b/kernel/src/filesystem/vfs.cpp
@@ -6,6 +6,7 @@
 #include "kernel/devices/storage/storage_management.hpp"
 #include "kernel/filesystem/custody.hpp"
 #include "kernel/filesystem/device_file.hpp"
+#include "kernel/filesystem/device_inode.hpp"
 #include "kernel/filesystem/ext2/ext2_filesystem.hpp"
 #include "kernel/filesystem/inode.hpp"
 #include "kernel/filesystem/inode_file.hpp"
@@ -72,7 +73,8 @@ namespace filesystem
       auto node = custody->get_inode();
       if (node->is_device())
       {
-        auto current_device_file = kstd::make_shared<device_file>(node->backing_device());
+        auto device_node = static_cast<device_inode *>(node.get());
+        auto current_device_file = kstd::make_shared<device_file>(device_node->backing_device());
         current_device_file->open();
         return open_file_description{current_device_file};
       }
@@ -92,7 +94,7 @@ namespace filesystem
       kapi::system::panic("[FILESYSTEM] make_device_node called with null device.");
     }
 
-    m_device_nodes.push_back(device_node_entry{device->name(), kstd::make_shared<inode>(device)});
+    m_device_nodes.push_back(device_node_entry{device->name(), kstd::make_shared<device_inode>(device)});
   }
 
   auto vfs::resolve_path(std::string_view path) -> std::optional<custody>