]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
uprobe: collapse THP pmd after removing all uprobes
authorSong Liu <songliubraving@fb.com>
Mon, 23 Sep 2019 22:38:33 +0000 (15:38 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 24 Sep 2019 22:54:11 +0000 (15:54 -0700)
After all uprobes are removed from the huge page (with PTE pgtable), it is
possible to collapse the pmd and benefit from THP again.  This patch does
the collapse by calling collapse_pte_mapped_thp().

Link: http://lkml.kernel.org/r/20190815164525.1848545-7-songliubraving@fb.com
Signed-off-by: Song Liu <songliubraving@fb.com>
Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Reported-by: kbuild test robot <lkp@intel.com>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
kernel/events/uprobes.c

index 27b596f14463a397073d4541474fdaf1075e4c49..94d38a39d72ecaf3ad0cbee26f5aa9fa1e26fb0e 100644 (file)
@@ -26,6 +26,7 @@
 #include <linux/percpu-rwsem.h>
 #include <linux/task_work.h>
 #include <linux/shmem_fs.h>
+#include <linux/khugepaged.h>
 
 #include <linux/uprobes.h>
 
@@ -472,6 +473,7 @@ int uprobe_write_opcode(struct arch_uprobe *auprobe, struct mm_struct *mm,
        struct page *old_page, *new_page;
        struct vm_area_struct *vma;
        int ret, is_register, ref_ctr_updated = 0;
+       bool orig_page_huge = false;
 
        is_register = is_swbp_insn(&opcode);
        uprobe = container_of(auprobe, struct uprobe, arch);
@@ -529,6 +531,9 @@ int uprobe_write_opcode(struct arch_uprobe *auprobe, struct mm_struct *mm,
                                /* let go new_page */
                                put_page(new_page);
                                new_page = NULL;
+
+                               if (PageCompound(orig_page))
+                                       orig_page_huge = true;
                        }
                        put_page(orig_page);
                }
@@ -547,6 +552,10 @@ int uprobe_write_opcode(struct arch_uprobe *auprobe, struct mm_struct *mm,
        if (ret && is_register && ref_ctr_updated)
                update_ref_ctr(uprobe, mm, -1);
 
+       /* try collapse pmd for compound page */
+       if (!ret && orig_page_huge)
+               collapse_pte_mapped_thp(mm, vaddr);
+
        return ret;
 }