On Wed 29-11-17 10:22:34, Michal Hocko wrote:
> What about this on top. I haven't tested this yet though.
> ---
We will need to drop surplus_huge_pages_node handling from the free path
obviously as well
diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index 1be43563e226..756833f9ef8b 100644
--- a/mm/hugetlb.c
+++ b/mm/hugetlb.c
@@ -1312,8 +1312,6 @@ void free_huge_page(struct page *page)
list_del(&page->lru);
ClearPageHugeTemporary(page);
update_and_free_page(h, page);
- if (h->surplus_huge_pages_node[nid])
- h->surplus_huge_pages_node[nid]--;
} else if (h->surplus_huge_pages_node[nid]) {
/* remove the page from active list */
list_del(&page->lru);
--
Michal Hocko
SUSE Labs
