From 65229a40828bb77e3ecd6085e9d4c0c3b39ce3d9 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 9 Dec 2019 20:06:03 +0100 Subject: [PATCH] Kernel: Move VMObject::for_each_region() to MemoryManager.h It can't be in VMObject.h since it depends on MemoryManager.h --- Kernel/VM/InodeVMObject.cpp | 15 --------------- Kernel/VM/MemoryManager.h | 15 +++++++++++++++ 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/Kernel/VM/InodeVMObject.cpp b/Kernel/VM/InodeVMObject.cpp index 93bcc65bf3..eb8f9d39f6 100644 --- a/Kernel/VM/InodeVMObject.cpp +++ b/Kernel/VM/InodeVMObject.cpp @@ -102,18 +102,3 @@ void InodeVMObject::inode_contents_changed(Badge, off_t offset, ssize_t s region.remap(); }); } - -template -void VMObject::for_each_region(Callback callback) -{ - // FIXME: Figure out a better data structure so we don't have to walk every single region every time an inode changes. - // Perhaps VMObject could have a Vector with all of his mappers? - for (auto& region : MM.m_user_regions) { - if (®ion.vmobject() == this) - callback(region); - } - for (auto& region : MM.m_kernel_regions) { - if (®ion.vmobject() == this) - callback(region); - } -} diff --git a/Kernel/VM/MemoryManager.h b/Kernel/VM/MemoryManager.h index b4c7cac06d..816c7d4fdb 100644 --- a/Kernel/VM/MemoryManager.h +++ b/Kernel/VM/MemoryManager.h @@ -138,3 +138,18 @@ struct ProcessPagingScope { ProcessPagingScope(Process&); ~ProcessPagingScope(); }; + +template +void VMObject::for_each_region(Callback callback) +{ + // FIXME: Figure out a better data structure so we don't have to walk every single region every time an inode changes. + // Perhaps VMObject could have a Vector with all of his mappers? + for (auto& region : MM.m_user_regions) { + if (®ion.vmobject() == this) + callback(region); + } + for (auto& region : MM.m_kernel_regions) { + if (®ion.vmobject() == this) + callback(region); + } +}