kernel/vm: Rename _FreePageRange "discarded" argument to "freedPages".

Clarifies things. No functional change.
This commit is contained in:
Augustin Cavalier
2024-12-16 22:06:06 -05:00
parent 47dbff983a
commit d4d55c1fbb
2 changed files with 6 additions and 6 deletions
+1 -1
View File
@@ -220,7 +220,7 @@ private:
void _RemoveConsumer(VMCache* consumer); void _RemoveConsumer(VMCache* consumer);
bool _FreePageRange(VMCachePagesTree::Iterator it, bool _FreePageRange(VMCachePagesTree::Iterator it,
page_num_t* toPage, page_num_t* discarded); page_num_t* toPage, page_num_t* freedPages);
private: private:
int32 fRefCount; int32 fRefCount;
+5 -5
View File
@@ -1106,7 +1106,7 @@ VMCache::SetMinimalCommitment(off_t commitment, int priority)
bool bool
VMCache::_FreePageRange(VMCachePagesTree::Iterator it, VMCache::_FreePageRange(VMCachePagesTree::Iterator it,
page_num_t* toPage = NULL, page_num_t* discarded = NULL) page_num_t* toPage = NULL, page_num_t* freedPages = NULL)
{ {
for (vm_page* page = it.Next(); for (vm_page* page = it.Next();
page != NULL && (toPage == NULL || page->cache_offset < *toPage); page != NULL && (toPage == NULL || page->cache_offset < *toPage);
@@ -1118,8 +1118,8 @@ VMCache::_FreePageRange(VMCachePagesTree::Iterator it,
// as we might cause a deadlock this way // as we might cause a deadlock this way
page->busy_writing = false; page->busy_writing = false;
// this will notify the writer to free the page // this will notify the writer to free the page
if (discarded != NULL) if (freedPages != NULL)
(*discarded)++; (*freedPages)++;
continue; continue;
} }
@@ -1140,8 +1140,8 @@ VMCache::_FreePageRange(VMCachePagesTree::Iterator it,
// removing the current node is safe. // removing the current node is safe.
vm_page_free(this, page); vm_page_free(this, page);
if (discarded != NULL) if (freedPages != NULL)
(*discarded)++; (*freedPages)++;
} }
return false; return false;