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

The page queues no longer count added dummy pages. This fixes a problem

Browse Source 
in steal_pages() which uses the number of pages in the inactive queue as
return criterion. It would thus return, if only marker pages were in the
queue and no page could be stolen. This led to a busy loop in
vm_page_allocate_page(). The whole system would become unusable when the
thread in question was the heap grower, since it would starve everyone
else due to its high priority.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@26710 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Ingo Weinhold 2008-07-31 23:54:14 +00:00
parent 5087b42fd6
commit dbc6635d1e
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/system/kernel/vm/vm_page.cpp
View File

@ -236,7 +236,8 @@ dequeue_page(page_queue *queue)
page->queue_next->queue_prev = NULL;
queue->head = page->queue_next;
queue->count--;
if (page->type != PAGE_TYPE_DUMMY)
queue->count--;
#ifdef DEBUG_PAGE_QUEUE
if (page->queue != queue) {
@ -270,7 +271,8 @@ enqueue_page(page_queue *queue, vm_page *page)
page->queue_next = NULL;
if (queue->head == NULL)
queue->head = page;
queue->count++;
if (page->type != PAGE_TYPE_DUMMY)
queue->count++;
#ifdef DEBUG_PAGE_QUEUE
page->queue = queue;
@ -296,7 +298,8 @@ enqueue_page_to_head(page_queue *queue, vm_page *page)
page->queue_prev = NULL;
if (queue->tail == NULL)
queue->tail = page;
queue->count++;
if (page->type != PAGE_TYPE_DUMMY)
queue->count++;
#ifdef DEBUG_PAGE_QUEUE
page->queue = queue;
@ -324,7 +327,8 @@ remove_page_from_queue(page_queue *queue, vm_page *page)
else
queue->head = page->queue_next;
queue->count--;
if (page->type != PAGE_TYPE_DUMMY)
queue->count--;
#ifdef DEBUG_PAGE_QUEUE
page->queue = NULL;
@ -370,7 +374,8 @@ insert_page_after(page_queue *queue, vm_page *before, vm_page *page)
if (queue->tail == before)
queue->tail = page;
queue->count++;
if (page->type != PAGE_TYPE_DUMMY)
queue->count++;
#ifdef DEBUG_PAGE_QUEUE
page->queue = queue;