i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

* If steal_pages() stole more than originally asked for, it might have lost those pages in the

Browse Source 
non-reserve case.
* vm_page_allocate_page() could also lose a stolen page in case more free pages were available
  after steal_pages() was called.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@22834 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Axel Dörfler
2007-11-05 11:37:02 +00:00
parent a66c2a2c9e
commit 5ff3d4f2db
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/system/kernel/vm/vm_page.cpp
+3 -5
View File
@@ -974,7 +974,7 @@ steal_pages(vm_page **pages, size_t count, bool reserve)
break; break;
if (steal_page(page, false)) { if (steal_page(page, false)) {
if (reserve) { if (reserve || stolen >= maxCount) {
InterruptsSpinLocker _(sPageLock); InterruptsSpinLocker _(sPageLock);
enqueue_page(&sFreePageQueue, page); enqueue_page(&sFreePageQueue, page);
page->state = PAGE_STATE_FREE; page->state = PAGE_STATE_FREE;
@@ -1374,10 +1374,8 @@ vm_page_allocate_page(int pageState, bool reserved)
size_t stolen = steal_pages(&page, 1, false); size_t stolen = steal_pages(&page, 1, false);
locker.Lock(); locker.Lock();
if (stolen == 0) { if (stolen > 0)
// just try again break;
continue;
}
} }
if (page->cache != NULL) if (page->cache != NULL)