RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 19:37:35 +00:00
Code Issues Activity Actions

Kernel: Always give back VM to the RangeAllocator when unmapping Region

Browse source 
We were only doing this in Process::deallocate_region(), which meant
that kernel-only Regions never gave back their VM.

With this patch, we can start reusing freed-up address space! :^)
This commit is contained in:
Andreas Kling 2019-08-07 21:55:52 +02:00
parent 37ba2a7b65
commit f5ff796970
2 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Kernel/Process.cpp
View file

@ -142,7 +142,6 @@ bool Process::deallocate_region(Region& region)
InterruptDisabler disabler; InterruptDisabler disabler;
for (int i = 0; i < m_regions.size(); ++i) { for (int i = 0; i < m_regions.size(); ++i) {
if (&m_regions[i] == &region) { if (&m_regions[i] == &region) {
page_directory().range_allocator().deallocate({ region.vaddr(), region.size() });
MM.unmap_region(region); MM.unmap_region(region);
m_regions.remove(i); m_regions.remove(i);
return true; return true;

1
Kernel/VM/MemoryManager.cpp
View file

@ -742,6 +742,7 @@ bool MemoryManager::unmap_region(Region& region)
dbgprintf("MM: >> Unmapped L%x => P%x <<\n", vaddr, physical_page ? physical_page->paddr().get() : 0); dbgprintf("MM: >> Unmapped L%x => P%x <<\n", vaddr, physical_page ? physical_page->paddr().get() : 0);
#endif #endif
} }
region.page_directory()->range_allocator().deallocate({ region.vaddr(), region.size() });
region.release_page_directory(); region.release_page_directory();
return true; return true;
} }