Kernel: Move DeviceManager to its own file

kernel/kernel/Device.cpp

@@ -1,115 +1,12 @@
 #include <BAN/Time.h>
 #include <kernel/Device.h>
-#include <kernel/LockGuard.h>
-#include <kernel/PCI.h>
-#include <kernel/Process.h>
 #include <kernel/RTC.h>
-#include <kernel/Storage/ATAController.h>
 
 namespace Kernel
 {
 
-	static DeviceManager* s_instance = nullptr;
-
 	Device::Device()
 		: m_create_time({ BAN::to_unix_time(RTC::get_current_time()), 0 })
 	{ }
 
-	void DeviceManager::initialize()
-	{
-		ASSERT(s_instance == nullptr);
-
-		s_instance = new DeviceManager;
-		ASSERT(s_instance != nullptr);
-		
-		for (const auto& pci_device : PCI::get().devices())
-		{
-			switch (pci_device.class_code())
-			{
-				case 0x01:
-				{
-					StorageController* controller = nullptr;
-					switch (pci_device.subclass())
-					{
-						case 0x01:
-							if (auto res = ATAController::create(pci_device); res.is_error())
-								dprintln("{}", res.error());
-							else
-								controller = res.value();
-							break;
-						default:
-							dprintln("unsupported storage device (pci subclass {2H})", pci_device.subclass());
-							break;
-					}
-
-					if (controller)
-					{
-						s_instance->add_device(controller);
-						for (auto* device : controller->devices())
-						{
-							s_instance->add_device(device);
-							for (auto* partition : device->partitions())
-								s_instance->add_device(partition);
-						}
-					}
-					break;
-				}
-				default:
-					break;
-			}
-		}
-
-		MUST(Process::create_kernel(
-			[](void*)
-			{
-				while (true)
-				{
-					DeviceManager::get().update();
-					PIT::sleep(1);
-				}
-			}, nullptr)
-		);
-	}
-
-	DeviceManager& DeviceManager::get()
-	{
-		ASSERT(s_instance);
-		return *s_instance;
-	}
-
-	void DeviceManager::update()
-	{
-		LockGuard _(m_lock);
-		for (Device* device : m_devices)
-			device->update();
-	}
-
-	void DeviceManager::add_device(Device* device)
-	{
-		LockGuard _(m_lock);
-		MUST(m_devices.push_back(device));
-	}
-
-	BAN::ErrorOr<BAN::RefPtr<Inode>> DeviceManager::read_directory_inode(BAN::StringView name)
-	{
-		LockGuard _(m_lock);
-		for (Device* device : m_devices)
-			if (device->name() == name)
-				return BAN::RefPtr<Inode>(device);
-		return BAN::Error::from_errno(ENOENT);
-	}
-
-	BAN::ErrorOr<BAN::Vector<BAN::String>> DeviceManager::read_directory_entries(size_t index)
-	{
-		BAN::Vector<BAN::String> result;
-		if (index > 0)
-			return result;
-
-		LockGuard _(m_lock);
-		TRY(result.reserve(m_devices.size()));
-		for (Device* device : m_devices)
-			TRY(result.emplace_back(device->name()));
-		return result;
-	}
-
 }

kernel/kernel/DeviceManager.cpp

@@ -0,0 +1,110 @@
+#include <kernel/DeviceManager.h>
+#include <kernel/LockGuard.h>
+#include <kernel/PCI.h>
+#include <kernel/Process.h>
+#include <kernel/Storage/ATAController.h>
+
+namespace Kernel
+{
+
+	static DeviceManager* s_instance = nullptr;
+
+	void DeviceManager::initialize()
+	{
+		ASSERT(s_instance == nullptr);
+
+		s_instance = new DeviceManager;
+		ASSERT(s_instance != nullptr);
+		
+		for (const auto& pci_device : PCI::get().devices())
+		{
+			switch (pci_device.class_code())
+			{
+				case 0x01:
+				{
+					StorageController* controller = nullptr;
+					switch (pci_device.subclass())
+					{
+						case 0x01:
+							if (auto res = ATAController::create(pci_device); res.is_error())
+								dprintln("{}", res.error());
+							else
+								controller = res.value();
+							break;
+						default:
+							dprintln("unsupported storage device (pci subclass {2H})", pci_device.subclass());
+							break;
+					}
+
+					if (controller)
+					{
+						s_instance->add_device(controller);
+						for (auto* device : controller->devices())
+						{
+							s_instance->add_device(device);
+							for (auto* partition : device->partitions())
+								s_instance->add_device(partition);
+						}
+					}
+					break;
+				}
+				default:
+					break;
+			}
+		}
+
+		MUST(Process::create_kernel(
+			[](void*)
+			{
+				while (true)
+				{
+					DeviceManager::get().update();
+					PIT::sleep(1);
+				}
+			}, nullptr)
+		);
+	}
+
+	DeviceManager& DeviceManager::get()
+	{
+		ASSERT(s_instance);
+		return *s_instance;
+	}
+
+	void DeviceManager::update()
+	{
+		LockGuard _(m_lock);
+		for (Device* device : m_devices)
+			device->update();
+	}
+
+	void DeviceManager::add_device(Device* device)
+	{
+		LockGuard _(m_lock);
+		MUST(m_devices.push_back(device));
+	}
+
+	BAN::ErrorOr<BAN::RefPtr<Inode>> DeviceManager::read_directory_inode(BAN::StringView name)
+	{
+		LockGuard _(m_lock);
+		for (Device* device : m_devices)
+			if (device->name() == name)
+				return BAN::RefPtr<Inode>(device);
+		return BAN::Error::from_errno(ENOENT);
+	}
+
+	BAN::ErrorOr<BAN::Vector<BAN::String>> DeviceManager::read_directory_entries(size_t index)
+	{
+		BAN::Vector<BAN::String> result;
+		if (index > 0)
+			return result;
+
+		LockGuard _(m_lock);
+		TRY(result.reserve(m_devices.size()));
+		for (Device* device : m_devices)
+			TRY(result.emplace_back(device->name()));
+		return result;
+	}
+
+
+}

kernel/kernel/FS/VirtualFileSystem.cpp

@@ -1,6 +1,6 @@
 #include <BAN/ScopeGuard.h>
 #include <BAN/StringView.h>
-#include <kernel/Device.h>
+#include <kernel/DeviceManager.h>
 #include <kernel/FS/Ext2.h>
 #include <kernel/FS/VirtualFileSystem.h>
 #include <kernel/LockGuard.h>

kernel/kernel/Input/PS2Controller.cpp

@@ -1,6 +1,7 @@
-#include <BAN/Time.h>
 #include <BAN/ScopeGuard.h>
+#include <BAN/Time.h>
 #include <kernel/ACPI.h>
+#include <kernel/DeviceManager.h>
 #include <kernel/IDT.h>
 #include <kernel/Input/PS2Controller.h>
 #include <kernel/Input/PS2Keyboard.h>

kernel/kernel/kernel.cpp

@@ -1,6 +1,7 @@
 #include <kernel/ACPI.h>
 #include <kernel/Arch.h>
 #include <kernel/Debug.h>
+#include <kernel/DeviceManager.h>
 #include <kernel/FS/VirtualFileSystem.h>
 #include <kernel/GDT.h>
 #include <kernel/IDT.h>