Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

vm_cache_resize() did not care about busy pages - now it does.

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@22353 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Axel Dörfler 2007-09-28 21:20:57 +00:00
parent 747c1927fd
commit 5036d66c8f
1 changed files with 22 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/system/kernel/vm/vm_cache.cpp
View File

@ -482,6 +482,9 @@ vm_cache_set_minimal_commitment_locked(vm_cache *cache, off_t commitment)
The cache lock must be held when you call it.
Since removed pages don't belong to the cache any longer, they are not
written back before they will be removed.
Note, this function way temporarily release the cache lock in case it
has to wait for busy pages.
*/
status_t
vm_cache_resize(vm_cache *cache, off_t newSize)
@ -499,17 +502,33 @@ vm_cache_resize(vm_cache *cache, off_t newSize)
newPageCount = (uint32)((newSize + B_PAGE_SIZE - 1) >> PAGE_SHIFT);
if (newPageCount < oldPageCount) {
// we need to remove all pages in the cache outside of the new virtual size
vm_page *page, *next;
// we need to remove all pages in the cache outside of the new virtual
// size
vm_page *page = cache->page_list, *next;
for (page = cache->page_list; page; page = next) {
while (page != NULL) {
next = page->cache_next;
if (page->cache_offset >= newPageCount) {
if (page->state == PAGE_STATE_BUSY) {
// wait for page to become unbusy
ConditionVariableEntry<vm_page> entry;
entry.Add(page);
mutex_unlock(&cache->lock);
entry.Wait();
mutex_lock(&cache->lock);
// restart from the start of the list
page = cache->page_list;
continue;
}
// remove the page and put it into the free queue
vm_cache_remove_page(cache, page);
vm_page_set_state(page, PAGE_STATE_FREE);
}
page = next;
}
}