From 8bb382decf74297290622bc4b97452f43fe7cebb Mon Sep 17 00:00:00 2001 From: Augustin Cavalier Date: Thu, 26 Dec 2024 13:46:45 -0500 Subject: [PATCH] kernel/vm: Drop vm_page_num_available_pages. Nothing uses it, and it isn't correct anyway: vm_page_num_free_pages() now includes cached pages, too, while vm_available_memory() includes not just available memory pages but also swap. --- headers/private/kernel/vm/vm_page.h | 1 - src/system/kernel/vm/vm_page.cpp | 14 -------------- 2 files changed, 15 deletions(-) diff --git a/headers/private/kernel/vm/vm_page.h b/headers/private/kernel/vm/vm_page.h index 464d254ce0..00e892d6f0 100644 --- a/headers/private/kernel/vm/vm_page.h +++ b/headers/private/kernel/vm/vm_page.h @@ -51,7 +51,6 @@ void vm_page_requeue(struct vm_page *page, bool tail); // get some data about the number of pages in the system page_num_t vm_page_num_pages(void); page_num_t vm_page_num_free_pages(void); -page_num_t vm_page_num_available_pages(void); page_num_t vm_page_num_unused_pages(void); void vm_page_get_stats(system_info *info); phys_addr_t vm_page_max_address(); diff --git a/src/system/kernel/vm/vm_page.cpp b/src/system/kernel/vm/vm_page.cpp index 0e03c935d2..19154d9aa6 100644 --- a/src/system/kernel/vm/vm_page.cpp +++ b/src/system/kernel/vm/vm_page.cpp @@ -4167,20 +4167,6 @@ vm_page_num_pages(void) } -/*! There is a subtle distinction between the page counts returned by - this function and vm_page_num_free_pages(): - The latter returns the number of pages that are completely uncommitted, - whereas this one returns the number of pages that are available for - use by being reclaimed as well (IOW it factors in things like cache pages - as available). -*/ -page_num_t -vm_page_num_available_pages(void) -{ - return vm_available_memory() / B_PAGE_SIZE; -} - - page_num_t vm_page_num_free_pages(void) {