RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-24 13:47:34 +00:00
Code Issues Activity Actions

Kernel: Always hold space lock while calculating memory statistics

Browse source 
And put the locker at the top of the functions for clarity.
This commit is contained in:
Andreas Kling 2021-02-08 22:18:26 +01:00
parent 8bda30edd2
commit 9ca42c4c0e
1 changed files with 10 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

22
Kernel/VM/Space.cpp
View file

@ -222,11 +222,11 @@ void Space::remove_all_regions(Badge<Process>)
size_t Space::amount_dirty_private() const size_t Space::amount_dirty_private() const
{ {
ScopedSpinLock lock(m_lock);
// FIXME: This gets a bit more complicated for Regions sharing the same underlying VMObject. // FIXME: This gets a bit more complicated for Regions sharing the same underlying VMObject.
// The main issue I'm thinking of is when the VMObject has physical pages that none of the Regions are mapping. // The main issue I'm thinking of is when the VMObject has physical pages that none of the Regions are mapping.
// That's probably a situation that needs to be looked at in general. // That's probably a situation that needs to be looked at in general.
size_t amount = 0; size_t amount = 0;
ScopedSpinLock lock(m_lock);
for (auto& region : m_regions) { for (auto& region : m_regions) {
if (!region.is_shared()) if (!region.is_shared())
amount += region.amount_dirty(); amount += region.amount_dirty();
@ -236,13 +236,11 @@ size_t Space::amount_dirty_private() const
size_t Space::amount_clean_inode() const size_t Space::amount_clean_inode() const
{ {
ScopedSpinLock lock(m_lock);
HashTable<const InodeVMObject*> vmobjects; HashTable<const InodeVMObject*> vmobjects;
{ for (auto& region : m_regions) {
ScopedSpinLock lock(m_lock); if (region.vmobject().is_inode())
for (auto& region : m_regions) { vmobjects.set(&static_cast<const InodeVMObject&>(region.vmobject()));
if (region.vmobject().is_inode())
vmobjects.set(&static_cast<const InodeVMObject&>(region.vmobject()));
}
} }
size_t amount = 0; size_t amount = 0;
for (auto& vmobject : vmobjects) for (auto& vmobject : vmobjects)
@ -252,8 +250,8 @@ size_t Space::amount_clean_inode() const
size_t Space::amount_virtual() const size_t Space::amount_virtual() const
{ {
size_t amount = 0;
ScopedSpinLock lock(m_lock); ScopedSpinLock lock(m_lock);
size_t amount = 0;
for (auto& region : m_regions) { for (auto& region : m_regions) {
amount += region.size(); amount += region.size();
} }
@ -262,9 +260,9 @@ size_t Space::amount_virtual() const
size_t Space::amount_resident() const size_t Space::amount_resident() const
{ {
ScopedSpinLock lock(m_lock);
// FIXME: This will double count if multiple regions use the same physical page. // FIXME: This will double count if multiple regions use the same physical page.
size_t amount = 0; size_t amount = 0;
ScopedSpinLock lock(m_lock);
for (auto& region : m_regions) { for (auto& region : m_regions) {
amount += region.amount_resident(); amount += region.amount_resident();
} }
@ -273,12 +271,12 @@ size_t Space::amount_resident() const
size_t Space::amount_shared() const size_t Space::amount_shared() const
{ {
ScopedSpinLock lock(m_lock);
// FIXME: This will double count if multiple regions use the same physical page. // FIXME: This will double count if multiple regions use the same physical page.
// FIXME: It doesn't work at the moment, since it relies on PhysicalPage ref counts, // FIXME: It doesn't work at the moment, since it relies on PhysicalPage ref counts,
// and each PhysicalPage is only reffed by its VMObject. This needs to be refactored // and each PhysicalPage is only reffed by its VMObject. This needs to be refactored
// so that every Region contributes +1 ref to each of its PhysicalPages. // so that every Region contributes +1 ref to each of its PhysicalPages.
size_t amount = 0; size_t amount = 0;
ScopedSpinLock lock(m_lock);
for (auto& region : m_regions) { for (auto& region : m_regions) {
amount += region.amount_shared(); amount += region.amount_shared();
} }
@ -287,8 +285,8 @@ size_t Space::amount_shared() const
size_t Space::amount_purgeable_volatile() const size_t Space::amount_purgeable_volatile() const
{ {
size_t amount = 0;
ScopedSpinLock lock(m_lock); ScopedSpinLock lock(m_lock);
size_t amount = 0;
for (auto& region : m_regions) { for (auto& region : m_regions) {
if (region.vmobject().is_anonymous() && static_cast<const AnonymousVMObject&>(region.vmobject()).is_any_volatile()) if (region.vmobject().is_anonymous() && static_cast<const AnonymousVMObject&>(region.vmobject()).is_any_volatile())
amount += region.amount_resident(); amount += region.amount_resident();
@ -298,8 +296,8 @@ size_t Space::amount_purgeable_volatile() const
size_t Space::amount_purgeable_nonvolatile() const size_t Space::amount_purgeable_nonvolatile() const
{ {
size_t amount = 0;
ScopedSpinLock lock(m_lock); ScopedSpinLock lock(m_lock);
size_t amount = 0;
for (auto& region : m_regions) { for (auto& region : m_regions) {
if (region.vmobject().is_anonymous() && !static_cast<const AnonymousVMObject&>(region.vmobject()).is_any_volatile()) if (region.vmobject().is_anonymous() && !static_cast<const AnonymousVMObject&>(region.vmobject()).is_any_volatile())
amount += region.amount_resident(); amount += region.amount_resident();