]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
powerpc/64s: free page table caches at exit_mmap time
authorNicholas Piggin <npiggin@gmail.com>
Wed, 25 Jul 2018 09:54:28 +0000 (19:54 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 7 Aug 2018 11:49:27 +0000 (21:49 +1000)
The kernel page table caches are tied to init_mm, so there is no
more need for them after userspace is finished.

destroy_context() gets called when we drop the last reference for an
mm, which can be much later than the task exit due to other lazy mm
references to it. We can free the page table cache pages on task exit
because they only cache the userspace page tables and kernel threads
should not access user space addresses.

The mapping for kernel threads itself is maintained in init_mm and
page table cache for that is attached to init_mm.

Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Reviewed-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com>
[mpe: Merge change log additions from Aneesh]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/mm/mmu_context_book3s64.c

index 39e9ef0eb78bcd901928520c23fb24e750aa1910..8b24168ea8c436cbea406186138b55c0bbd45d8d 100644 (file)
@@ -221,7 +221,7 @@ static void pmd_frag_destroy(void *pmd_frag)
        }
 }
 
-static void destroy_pagetable_page(struct mm_struct *mm)
+static void destroy_pagetable_cache(struct mm_struct *mm)
 {
        void *frag;
 
@@ -244,13 +244,14 @@ void destroy_context(struct mm_struct *mm)
                WARN_ON(process_tb[mm->context.id].prtb0 != 0);
        else
                subpage_prot_free(mm);
-       destroy_pagetable_page(mm);
        destroy_contexts(&mm->context);
        mm->context.id = MMU_NO_CONTEXT;
 }
 
 void arch_exit_mmap(struct mm_struct *mm)
 {
+       destroy_pagetable_cache(mm);
+
        if (radix_enabled()) {
                /*
                 * Radix doesn't have a valid bit in the process table