aboutsummaryrefslogtreecommitdiff
path: root/kernel
diff options
context:
space:
mode:
authorMarcel Braun <marcel.braun@ost.ch>2026-05-15 11:42:33 +0200
committerLukas Oesch <lukasoesch20@gmail.com>2026-05-16 11:56:07 +0200
commit963c926c68aac4606d80743aca8e7b052eee7efe (patch)
treec678534c823cf708cf4b840a4dfe6ddf12a49c60 /kernel
parentfe0b38db0f8848da8cf28bb883e01bbbb889dd0a (diff)
downloadkernel-963c926c68aac4606d80743aca8e7b052eee7efe.tar.xz
kernel-963c926c68aac4606d80743aca8e7b052eee7efe.zip
Rename mount_table method from find_exact_mount to find_mount
Diffstat (limited to 'kernel')
-rw-r--r--kernel/include/kernel/filesystem/mount_table.hpp2
-rw-r--r--kernel/src/filesystem/mount_table.cpp4
-rw-r--r--kernel/src/filesystem/mount_table.tests.cpp10
-rw-r--r--kernel/src/filesystem/vfs.cpp6
4 files changed, 11 insertions, 11 deletions
diff --git a/kernel/include/kernel/filesystem/mount_table.hpp b/kernel/include/kernel/filesystem/mount_table.hpp
index 59b9503..8bebfe2 100644
--- a/kernel/include/kernel/filesystem/mount_table.hpp
+++ b/kernel/include/kernel/filesystem/mount_table.hpp
@@ -43,7 +43,7 @@ namespace kernel::filesystem
@param path The path to match against the mount paths in the table.
@return A pointer to the mount with the exact matching path, or a null pointer if no mount matches the path.
*/
- [[nodiscard]] auto find_exact_mount(std::string_view path) const -> kstd::shared_ptr<mount>;
+ [[nodiscard]] auto find_mount(std::string_view path) const -> kstd::shared_ptr<mount>;
private:
[[nodiscard]] auto has_child_mounts(kstd::shared_ptr<mount> const & parent_mount) const -> bool;
diff --git a/kernel/src/filesystem/mount_table.cpp b/kernel/src/filesystem/mount_table.cpp
index 4b61fb0..78ac727 100644
--- a/kernel/src/filesystem/mount_table.cpp
+++ b/kernel/src/filesystem/mount_table.cpp
@@ -31,7 +31,7 @@ namespace kernel::filesystem
auto mount_table::remove_mount(std::string_view path) -> operation_result
{
// TODO BA-FS26 check wheter something is open in this mount
- // TODO BA-FS26 nearly the same code is in find_exact_mount -> refactor to avoid code duplication
+ // TODO BA-FS26 nearly the same code is in find_mount -> refactor to avoid code duplication
auto mount_range =
std::ranges::find_last_if(m_mounts, [&](auto const & mount) { return mount->get_mount_path() == path; });
auto mount_it = mount_range.begin();
@@ -52,7 +52,7 @@ namespace kernel::filesystem
return operation_result::removed;
}
- auto mount_table::find_exact_mount(std::string_view path) const -> kstd::shared_ptr<mount>
+ auto mount_table::find_mount(std::string_view path) const -> kstd::shared_ptr<mount>
{
auto mount_range =
std::ranges::find_last_if(m_mounts, [&](auto const & mount) { return mount->get_mount_path() == path; });
diff --git a/kernel/src/filesystem/mount_table.tests.cpp b/kernel/src/filesystem/mount_table.tests.cpp
index 4ae8711..f22b25e 100644
--- a/kernel/src/filesystem/mount_table.tests.cpp
+++ b/kernel/src/filesystem/mount_table.tests.cpp
@@ -58,14 +58,14 @@ SCENARIO("Adding, finding and removing mounts in the mount table", "[filesystem]
THEN("finding mounts by exact valid path returns the correct mount")
{
- REQUIRE(table.find_exact_mount("/") == mount1);
- REQUIRE(table.find_exact_mount("/mnt") == mount2);
+ REQUIRE(table.find_mount("/") == mount1);
+ REQUIRE(table.find_mount("/mnt") == mount2);
}
THEN("finding mounts by exact invalid path returns null")
{
- REQUIRE(table.find_exact_mount("/nonexistent") == nullptr);
- REQUIRE(table.find_exact_mount("/mnt/file") == nullptr);
+ REQUIRE(table.find_mount("/nonexistent") == nullptr);
+ REQUIRE(table.find_mount("/mnt/file") == nullptr);
}
THEN("removing a mount that has no child mounts succeeds")
@@ -103,7 +103,7 @@ SCENARIO("Adding, finding and removing mounts in the mount table", "[filesystem]
THEN("finding mounts by exact valid path returns the correct mount")
{
- REQUIRE(table.find_exact_mount("/") == mount2);
+ REQUIRE(table.find_mount("/") == mount2);
}
THEN("removing the topmost mount with the same path succeeds")
diff --git a/kernel/src/filesystem/vfs.cpp b/kernel/src/filesystem/vfs.cpp
index c395e74..9b0440d 100644
--- a/kernel/src/filesystem/vfs.cpp
+++ b/kernel/src/filesystem/vfs.cpp
@@ -171,7 +171,7 @@ namespace kernel::filesystem
return {nullptr, nullptr};
}
- auto current_mount = m_mount_table.find_exact_mount("/");
+ auto current_mount = m_mount_table.find_mount("/");
if (!current_mount)
{
kapi::system::panic("[FILESYSTEM] no root mount found.");
@@ -232,7 +232,7 @@ namespace kernel::filesystem
}
else if (next_dentry->has_flag(dentry::dentry_flags::is_mount_point))
{
- current_mount = m_mount_table.find_exact_mount(next_dentry->get_absolute_path().view());
+ current_mount = m_mount_table.find_mount(next_dentry->get_absolute_path().view());
if (!current_mount)
{
kapi::system::panic("[FILESYSTEM] mount for dentry with mounted flag not found.");
@@ -260,7 +260,7 @@ namespace kernel::filesystem
if (path::is_valid_absolute_path(symbolic_link_path))
{
- current_mount = m_mount_table.find_exact_mount("/");
+ current_mount = m_mount_table.find_mount("/");
current_dentry = current_mount->get_root_dentry();
}
continue;