From 40c91984152a238351529d2611fe627040f6a618 Mon Sep 17 00:00:00 2001 From: Felix Morgner Date: Fri, 24 Jul 2026 19:33:03 +0200 Subject: kapi: rename boot_module to module --- arch/x86_64/kapi/boot_modules.cpp | 5 ++--- kapi/kapi/boot_modules.hpp | 6 +++--- kapi/kapi/boot_modules/boot_module.hpp | 25 ------------------------- kapi/kapi/boot_modules/bus.hpp | 10 +++++----- kapi/kapi/boot_modules/module.hpp | 25 +++++++++++++++++++++++++ kapi/kapi/boot_modules/registry.hpp | 6 +++--- kernel/kapi/boot_module/bus.cpp | 4 ++-- kernel/kapi/boot_module/registry.cpp | 2 +- kernel/kernel/drivers/storage/ram_disk.cpp | 4 ++-- 9 files changed, 43 insertions(+), 44 deletions(-) delete mode 100644 kapi/kapi/boot_modules/boot_module.hpp create mode 100644 kapi/kapi/boot_modules/module.hpp diff --git a/arch/x86_64/kapi/boot_modules.cpp b/arch/x86_64/kapi/boot_modules.cpp index 361a155a..136eb30d 100644 --- a/arch/x86_64/kapi/boot_modules.cpp +++ b/arch/x86_64/kapi/boot_modules.cpp @@ -4,10 +4,9 @@ #include #include -#include +#include #include #include -#include #include @@ -28,7 +27,7 @@ namespace kapi::boot_modules auto modules = boot::bootstrap_information.mbi->modules(); std::ranges::for_each(modules, [](auto const & module) { - registry::get().add(kapi::boot_modules::boot_module{ + registry::get().add(kapi::boot_modules::module{ .name = module.string(), .start_address = memory::linear_address{module.start_address + std::bit_cast(&arch::boot::TEACHOS_VMA)}, diff --git a/kapi/kapi/boot_modules.hpp b/kapi/kapi/boot_modules.hpp index 49a83ef5..7f47c628 100644 --- a/kapi/kapi/boot_modules.hpp +++ b/kapi/kapi/boot_modules.hpp @@ -1,9 +1,9 @@ #ifndef TEACHOS_KAPI_BOOT_MODULES_HPP #define TEACHOS_KAPI_BOOT_MODULES_HPP -#include // IWYU pragma: export -#include // IWYU pragma: export -#include // IWYU pragma: export +#include // IWYU pragma: export +#include // IWYU pragma: export +#include // IWYU pragma: export namespace kapi::boot_modules { diff --git a/kapi/kapi/boot_modules/boot_module.hpp b/kapi/kapi/boot_modules/boot_module.hpp deleted file mode 100644 index 85de7ab0..00000000 --- a/kapi/kapi/boot_modules/boot_module.hpp +++ /dev/null @@ -1,25 +0,0 @@ -#ifndef TEACHOS_KAPI_BOOT_MODULES_BOOT_MODULE_HPP -#define TEACHOS_KAPI_BOOT_MODULES_BOOT_MODULE_HPP - -// IWYU pragma: private, include - -#include - -#include -#include - -namespace kapi::boot_modules -{ - // ! The boot module struct - // ! - // ! The boot module struct represents a module loaded by the bootloader, and contains information about it, such as - // ! its name, virtual start address, and size. - struct boot_module - { - std::string_view name{}; - memory::linear_address start_address{}; - std::size_t size{}; - }; -} // namespace kapi::boot_modules - -#endif \ No newline at end of file diff --git a/kapi/kapi/boot_modules/bus.hpp b/kapi/kapi/boot_modules/bus.hpp index 7e3c773c..2c52677e 100644 --- a/kapi/kapi/boot_modules/bus.hpp +++ b/kapi/kapi/boot_modules/bus.hpp @@ -3,7 +3,7 @@ // IWYU pragma: private, include -#include +#include #include #include @@ -24,7 +24,7 @@ namespace kapi::boot_modules [[nodiscard]] auto virtual command_line() const -> std::string_view = 0; - [[nodiscard]] auto virtual module() const -> boot_module const & = 0; + [[nodiscard]] auto virtual module() const -> module const & = 0; }; struct boot_module_driver_identification @@ -38,17 +38,17 @@ namespace kapi::boot_modules struct boot_module_device final : kapi::devices::device, boot_module_identification { - boot_module_device(std::size_t index, boot_module const & module); + boot_module_device(std::size_t index, struct module const & module); [[nodiscard]] auto command_line() const -> std::string_view override; - [[nodiscard]] auto module() const -> boot_module const & override; + [[nodiscard]] auto module() const -> struct module const & override; protected: auto query_interface(kapi::devices::interface interface) -> void * override; private: - boot_module m_module; + struct module m_module; }; struct boot_module_bus final : kapi::devices::bus, kapi::devices::bus_protocol diff --git a/kapi/kapi/boot_modules/module.hpp b/kapi/kapi/boot_modules/module.hpp new file mode 100644 index 00000000..14898132 --- /dev/null +++ b/kapi/kapi/boot_modules/module.hpp @@ -0,0 +1,25 @@ +#ifndef TEACHOS_KAPI_BOOT_MODULES_MODULE_HPP +#define TEACHOS_KAPI_BOOT_MODULES_MODULE_HPP + +// IWYU pragma: private, include + +#include + +#include +#include + +namespace kapi::boot_modules +{ + // ! The boot module struct + // ! + // ! The boot module struct represents a module loaded by the bootloader, and contains information about it, such as + // ! its name, virtual start address, and size. + struct module + { + std::string_view name{}; + memory::linear_address start_address{}; + std::size_t size{}; + }; +} // namespace kapi::boot_modules + +#endif \ No newline at end of file diff --git a/kapi/kapi/boot_modules/registry.hpp b/kapi/kapi/boot_modules/registry.hpp index 4c659fa4..33dc2169 100644 --- a/kapi/kapi/boot_modules/registry.hpp +++ b/kapi/kapi/boot_modules/registry.hpp @@ -3,7 +3,7 @@ // IWYU pragma: private, include -#include +#include #include @@ -21,7 +21,7 @@ namespace kapi::boot_modules //! providing access to them for the rest of the kernel. struct registry { - using container = kstd::vector; + using container = kstd::vector; using value_type = container::value_type; using const_reference = container::const_reference; @@ -63,7 +63,7 @@ namespace kapi::boot_modules [[nodiscard]] auto operator[](std::size_t index) const noexcept -> const_reference; - auto add(boot_module module) -> void; + auto add(module module) -> void; private: container m_modules{}; diff --git a/kernel/kapi/boot_module/bus.cpp b/kernel/kapi/boot_module/bus.cpp index 88f2cda3..95264d00 100644 --- a/kernel/kapi/boot_module/bus.cpp +++ b/kernel/kapi/boot_module/bus.cpp @@ -28,7 +28,7 @@ namespace kapi::boot_modules return false; } - boot_module_device::boot_module_device(std::size_t index, boot_module const & module) + boot_module_device::boot_module_device(std::size_t index, struct module const & module) : kapi::devices::device{kstd::format("boot_module{}", index)} , m_module(module) {} @@ -38,7 +38,7 @@ namespace kapi::boot_modules return m_module.name; } - auto boot_module_device::module() const -> boot_modules::boot_module const & + auto boot_module_device::module() const -> boot_modules::module const & { return m_module; } diff --git a/kernel/kapi/boot_module/registry.cpp b/kernel/kapi/boot_module/registry.cpp index 80c76129..9876866b 100644 --- a/kernel/kapi/boot_module/registry.cpp +++ b/kernel/kapi/boot_module/registry.cpp @@ -106,7 +106,7 @@ namespace kapi::boot_modules return m_modules[index]; } - auto registry::add(boot_module module) -> void + auto registry::add(module module) -> void { m_modules.push_back(module); } diff --git a/kernel/kernel/drivers/storage/ram_disk.cpp b/kernel/kernel/drivers/storage/ram_disk.cpp index cd5368cb..db87a48b 100644 --- a/kernel/kernel/drivers/storage/ram_disk.cpp +++ b/kernel/kernel/drivers/storage/ram_disk.cpp @@ -32,7 +32,7 @@ namespace kernel::drivers::storage struct block_node final : kapi::devices::block_device { - explicit block_node(kapi::boot_modules::boot_module const & module) + explicit block_node(kapi::boot_modules::module const & module) : m_module(module) {} @@ -88,7 +88,7 @@ namespace kernel::drivers::storage } private: - kapi::boot_modules::boot_module m_module; + kapi::boot_modules::module m_module; }; } // namespace -- cgit v1.2.3