| Message ID | 20180725095428.22561-1-npiggin@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 34c604d27590fdc9a2c944be8c50ae1fc80f5f25 |
| Headers | show |
| Series | powerpc/64s: free page table caches at exit_mmap time | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/apply_patch | success | next/apply_patch Successfully applied |
| snowpatch_ozlabs/checkpatch | success | Test checkpatch on branch next |
| snowpatch_ozlabs/build-ppc64le | success | Test build-ppc64le on branch next |
| snowpatch_ozlabs/build-ppc64be | success | Test build-ppc64be on branch next |
| snowpatch_ozlabs/build-ppc64e | success | Test build-ppc64e on branch next |
| snowpatch_ozlabs/build-ppc32 | success | Test build-ppc32 on branch next |
Nicholas Piggin <npiggin@gmail.com> writes: > The kernel page table caches are tied to init_mm, so there is no > more need for them after userspace is finished. > The commit message could be improved with reference to active_mm. something like? destroy_context get called when we drop the last reference for mm which could be much later than the task exit due to other lazy mm reference to it. We could free the page table cache pages on task exit because they only cache the userspace page table and kernel thread should not access the user space address. The mapping for kernel threads itself is maintained in init_mm and page table cache for that is attached to init_mm Reviewed-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> > Signed-off-by: Nicholas Piggin <npiggin@gmail.com> > --- > arch/powerpc/mm/mmu_context_book3s64.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/mm/mmu_context_book3s64.c b/arch/powerpc/mm/mmu_context_book3s64.c > index 3bb5cec03d1f..5738c2db751c 100644 > --- a/arch/powerpc/mm/mmu_context_book3s64.c > +++ b/arch/powerpc/mm/mmu_context_book3s64.c > @@ -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 > -- > 2.17.0
On Wed, 2018-07-25 at 09:54:28 UTC, Nicholas Piggin wrote: > The kernel page table caches are tied to init_mm, so there is no > more need for them after userspace is finished. > > Signed-off-by: Nicholas Piggin <npiggin@gmail.com> > Reviewed-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/34c604d27590fdc9a2c944be8c50ae cheers
diff --git a/arch/powerpc/mm/mmu_context_book3s64.c b/arch/powerpc/mm/mmu_context_book3s64.c index 3bb5cec03d1f..5738c2db751c 100644 --- a/arch/powerpc/mm/mmu_context_book3s64.c +++ b/arch/powerpc/mm/mmu_context_book3s64.c @@ -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
The kernel page table caches are tied to init_mm, so there is no more need for them after userspace is finished. Signed-off-by: Nicholas Piggin <npiggin@gmail.com> --- arch/powerpc/mm/mmu_context_book3s64.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)