Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit bed2b90

Browse files
Liu Shixingregkh
authored andcommitted
mm/swapfile: skip HugeTLB pages for unuse_vma
commit 7528c4f upstream. I got a bad pud error and lost a 1GB HugeTLB when calling swapoff. The problem can be reproduced by the following steps: 1. Allocate an anonymous 1GB HugeTLB and some other anonymous memory. 2. Swapout the above anonymous memory. 3. run swapoff and we will get a bad pud error in kernel message: mm/pgtable-generic.c:42: bad pud 00000000743d215d(84000001400000e7) We can tell that pud_clear_bad is called by pud_none_or_clear_bad in unuse_pud_range() by ftrace. And therefore the HugeTLB pages will never be freed because we lost it from page table. We can skip HugeTLB pages for unuse_vma to fix it. Link: https://lkml.kernel.org/r/[email protected] Fixes: 0fe6e20 ("hugetlb, rmap: add reverse mapping for hugepage") Signed-off-by: Liu Shixin <[email protected]> Acked-by: Muchun Song <[email protected]> Cc: Naoya Horiguchi <[email protected]> Cc: <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent a0035fc commit bed2b90

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mm/swapfile.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2003,7 +2003,7 @@ static int unuse_mm(struct mm_struct *mm, unsigned int type)
20032003

20042004
mmap_read_lock(mm);
20052005
for_each_vma(vmi, vma) {
2006-
if (vma->anon_vma) {
2006+
if (vma->anon_vma && !is_vm_hugetlb_page(vma)) {
20072007
ret = unuse_vma(vma, type);
20082008
if (ret)
20092009
break;

0 commit comments

Comments
 (0)