Bananymous/banan-os
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Kernel: Remove unnecessary usages of PageTableScope

Browse Source 
This should be used as few times as possible since it calls 'cli'
This commit is contained in:
Bananymous
2023-05-29 21:10:48 +03:00
parent 5bb1f2a48c
commit 861966eed6
3 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
kernel/kernel/Memory/FixedWidthAllocator.cpp
View File

@@ -1,5 +1,4 @@
#include <kernel/Memory/FixedWidthAllocator.h>
#include <kernel/Memory/PageTableScope.h>
namespace Kernel
{
@@ -229,7 +228,8 @@ namespace Kernel
if (allocator == nullptr)
return BAN::Error::from_errno(ENOMEM);
PageTableScope _(m_page_table);
m_page_table.lock();
ASSERT(m_page_table.is_page_free(0));
for (node* node = m_used_list; node; node = node->next)
@@ -257,6 +257,8 @@ namespace Kernel
m_page_table.unmap_page(0);
m_page_table.invalidate(0);
m_page_table.unlock();
return allocator;
}

7
kernel/kernel/Memory/GeneralAllocator.cpp
View File

@@ -1,6 +1,4 @@
#include <kernel/Memory/GeneralAllocator.h>
#include <kernel/Memory/PageTableScope.h>
#include <kernel/Process.h>
namespace Kernel
{
@@ -68,7 +66,8 @@ namespace Kernel
if (allocator == nullptr)
return BAN::Error::from_errno(ENOMEM);
PageTableScope _(m_page_table);
m_page_table.lock();
ASSERT(m_page_table.is_page_free(0));
for (auto& allocation : m_allocations)
@@ -100,6 +99,8 @@ namespace Kernel
m_page_table.unmap_page(0);
m_page_table.invalidate(0);
m_page_table.unlock();
return allocator;
}

6
kernel/kernel/Memory/VirtualRange.cpp
View File

@@ -1,5 +1,4 @@
#include <kernel/Memory/Heap.h>
#include <kernel/Memory/PageTableScope.h>
#include <kernel/Memory/VirtualRange.h>
namespace Kernel
@@ -78,7 +77,8 @@ namespace Kernel
{
VirtualRange* result = create(page_table, vaddr(), size(), flags());
PageTableScope _(m_page_table);
m_page_table.lock();
ASSERT(m_page_table.is_page_free(0));
for (size_t i = 0; i < result->m_physical_pages.size(); i++)
{
@@ -89,6 +89,8 @@ namespace Kernel
m_page_table.unmap_page(0);
m_page_table.invalidate(0);
m_page_table.unlock();
return result;
}