diff options
author | Andreas Kling <kling@serenityos.org> | 2022-04-05 12:07:45 +0200 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2022-04-05 12:23:47 +0200 |
commit | cfb61cbd5407681e2712bdde75d05e1831315062 (patch) | |
tree | c0d9061de0234c19f937d8dfea2a3997c5d72422 /Kernel/Memory/MemoryManager.cpp | |
parent | da7ea2556e2dc2b027609feec35204b12693afd4 (diff) | |
download | serenity-cfb61cbd5407681e2712bdde75d05e1831315062.zip |
Kernel: Add RegionTree::find_region_containing(address or range)
Let's encapsulate looking up regions so clients don't have to dig into
RegionTree internals.
Diffstat (limited to 'Kernel/Memory/MemoryManager.cpp')
-rw-r--r-- | Kernel/Memory/MemoryManager.cpp | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/Kernel/Memory/MemoryManager.cpp b/Kernel/Memory/MemoryManager.cpp index 9384fa6a98..2934655b75 100644 --- a/Kernel/Memory/MemoryManager.cpp +++ b/Kernel/Memory/MemoryManager.cpp @@ -658,18 +658,12 @@ UNMAP_AFTER_INIT void MemoryManager::initialize(u32 cpu) } } -Region* MemoryManager::kernel_region_from_vaddr(VirtualAddress vaddr) +Region* MemoryManager::kernel_region_from_vaddr(VirtualAddress address) { - if (is_user_address(vaddr)) + if (is_user_address(address)) return nullptr; - SpinlockLocker lock(s_mm_lock); - SpinlockLocker tree_locker(MM.m_region_tree.get_lock()); - - auto* region = MM.m_region_tree.regions().find_largest_not_above(vaddr.get()); - if (!region || !region->contains(vaddr)) - return nullptr; - return region; + return MM.region_tree().find_region_containing(address); } Region* MemoryManager::find_user_region_from_vaddr_no_lock(AddressSpace& space, VirtualAddress vaddr) |