diff options
author | Liu Xiang <[email protected]> | 2020-12-14 19:12:05 -0800 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2020-12-15 12:13:44 -0800 |
commit | 0a4f3d1bb91cac4efdd780373638b6a1a4c24c51 (patch) | |
tree | ce7178171289186ab535a0043e2e299e3eaf7f85 | |
parent | 336e6b53d99ae32ee35c1a7d3d0f99db22e1ff51 (diff) |
mm: hugetlb: fix type of delta parameter and related local variables in gather_surplus_pages()
On 64-bit machine, delta variable in hugetlb_acct_memory() may be larger
than 0xffffffff, but gather_surplus_pages() can only use the low 32-bit
value now. So we need to fix type of delta parameter and related local
variables in gather_surplus_pages().
Link: https://lkml.kernel.org/r/[email protected]
Reported-by: Ma Chenggong <[email protected]>
Signed-off-by: Liu Xiang <[email protected]>
Signed-off-by: Pan Jiagen <[email protected]>
Reviewed-by: Mike Kravetz <[email protected]>
Cc: Liu Xiang <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
-rw-r--r-- | mm/hugetlb.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 987d4b57ab0d..70ecc1538499 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -1944,13 +1944,14 @@ struct page *alloc_huge_page_vma(struct hstate *h, struct vm_area_struct *vma, * Increase the hugetlb pool such that it can accommodate a reservation * of size 'delta'. */ -static int gather_surplus_pages(struct hstate *h, int delta) +static int gather_surplus_pages(struct hstate *h, long delta) __must_hold(&hugetlb_lock) { struct list_head surplus_list; struct page *page, *tmp; - int ret, i; - int needed, allocated; + int ret; + long i; + long needed, allocated; bool alloc_ok = true; needed = (h->resv_huge_pages + delta) - h->free_huge_pages; |