mm/hugetlb: fix nodes huge page allocation when there are surplus pages
authorXueshi Hu <xueshi.hu@smartx.com>
Tue, 29 Aug 2023 03:33:43 +0000 (11:33 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Wed, 4 Oct 2023 17:32:19 +0000 (10:32 -0700)
commitb72b3c9c34c825c81d205241c5f822fc7835923f
treedcf01dd4c6ddc5b5fef0ab849c49ed64490791f9
parentd8f5f7e445f02eb10dee1a0a992146314cf460f8
mm/hugetlb: fix nodes huge page allocation when there are surplus pages

In set_nr_huge_pages(), local variable "count" is used to record
persistent_huge_pages(), but when it cames to nodes huge page allocation,
the semantics changes to nr_huge_pages.  When there exists surplus huge
pages and using the interface under
/sys/devices/system/node/node*/hugepages to change huge page pool size,
this difference can result in the allocation of an unexpected number of
huge pages.

Steps to reproduce the bug:

Starting with:

  Node 0          Node 1    Total
HugePages_Total             0.00            0.00     0.00
HugePages_Free              0.00            0.00     0.00
HugePages_Surp              0.00            0.00     0.00

create 100 huge pages in Node 0 and consume it, then set Node 0 's
nr_hugepages to 0.

yields:

  Node 0          Node 1    Total
HugePages_Total           200.00            0.00   200.00
HugePages_Free              0.00            0.00     0.00
HugePages_Surp            200.00            0.00   200.00

write 100 to Node 1's nr_hugepages

echo 100 > /sys/devices/system/node/node1/\
hugepages/hugepages-2048kB/nr_hugepages

gets:

  Node 0          Node 1    Total
HugePages_Total           200.00          400.00   600.00
HugePages_Free              0.00          400.00   400.00
HugePages_Surp            200.00            0.00   200.00

Kernel is expected to create only 100 huge pages and it gives 200.

Link: https://lkml.kernel.org/r/20230829033343.467779-1-xueshi.hu@smartx.com
Fixes: 9a30523066cd ("hugetlb: add per node hstate attributes")
Signed-off-by: Xueshi Hu <xueshi.hu@smartx.com>
Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: Lee Schermerhorn <lee.schermerhorn@hp.com>
Cc: Mel Gorman <mel@csn.ul.ie>
Cc: Muchun Song <muchun.song@linux.dev>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/hugetlb.c