summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKen Chen <kenchen@google.com>2007-07-23 18:44:00 -0700
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-07-24 12:24:59 -0700
commit5ab3ee7b1cd5c91eb2272764f9d7d1fe4749681e (patch)
tree82c1fbc5e3a41651d4c0d45cf13d85760708c184
parentbe1ff386e768ee4fc19bb7da48cee4fc4cb4e75b (diff)
downloadop-kernel-dev-5ab3ee7b1cd5c91eb2272764f9d7d1fe4749681e.zip
op-kernel-dev-5ab3ee7b1cd5c91eb2272764f9d7d1fe4749681e.tar.gz
fix hugetlb page allocation leak
dequeue_huge_page() has a serious memory leak upon hugetlb page allocation. The for loop continues on allocating hugetlb pages out of all allowable zone, where this function is supposedly only dequeue one and only one pages. Fixed it by breaking out of the for loop once a hugetlb page is found. Signed-off-by: Ken Chen <kenchen@google.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--mm/hugetlb.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index f127940..d7ca59d 100644
--- a/mm/hugetlb.c
+++ b/mm/hugetlb.c
@@ -84,6 +84,7 @@ static struct page *dequeue_huge_page(struct vm_area_struct *vma,
list_del(&page->lru);
free_huge_pages--;
free_huge_pages_node[nid]--;
+ break;
}
}
return page;
OpenPOWER on IntegriCloud