diff options
author | zhong jiang <zhongjiang@huawei.com> | 2016-08-11 18:32:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-08-11 19:58:13 -0400 |
commit | c1470b33bb6e18cddd361fef339ef225b8339fe7 (patch) | |
tree | 9689ae48f5c43edd37a8f472980817eb23485c17 /mm | |
parent | 85e97be32c6242c98dbbc7a241b4a78c1b93327b (diff) |
mm/hugetlb: fix incorrect hugepages count during mem hotplug
When memory hotplug operates, free hugepages will be freed if the
movable node is offline. Therefore, /proc/sys/vm/nr_hugepages will be
incorrect.
Fix it by reducing max_huge_pages when the node is offlined.
n-horiguchi@ah.jp.nec.com said:
: dissolve_free_huge_page intends to break a hugepage into buddy, and the
: destination hugepage is supposed to be allocated from the pool of the
: destination node, so the system-wide pool size is reduced. So adding
: h->max_huge_pages-- makes sense to me.
Link: http://lkml.kernel.org/r/1470624546-902-1-git-send-email-zhongjiang@huawei.com
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Cc: Mike Kravetz <mike.kravetz@oracle.com>
Acked-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/hugetlb.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mm/hugetlb.c b/mm/hugetlb.c index b9aa1b0b38b0..87e11d8ad536 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c | |||
@@ -1448,6 +1448,7 @@ static void dissolve_free_huge_page(struct page *page) | |||
1448 | list_del(&page->lru); | 1448 | list_del(&page->lru); |
1449 | h->free_huge_pages--; | 1449 | h->free_huge_pages--; |
1450 | h->free_huge_pages_node[nid]--; | 1450 | h->free_huge_pages_node[nid]--; |
1451 | h->max_huge_pages--; | ||
1451 | update_and_free_page(h, page); | 1452 | update_and_free_page(h, page); |
1452 | } | 1453 | } |
1453 | spin_unlock(&hugetlb_lock); | 1454 | spin_unlock(&hugetlb_lock); |