Commitb36f5b0

hnaz

authored and

torvalds

committed

thp: mprotect: pass vma down to page table walkers

Flushing the tlb for huge pmds requires the vma's anon_vma, so pass alongthe vma instead of the mm, we can always get the latter when we need it.Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>Signed-off-by: Andrea Arcangeli <aarcange@redhat.com>Reviewed-by: Rik van Riel <riel@redhat.com>Signed-off-by: Andrew Morton <akpm@linux-foundation.org>Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

1 parentc489f12 commitb36f5b0Copy full SHA for b36f5b0

File tree

1 file changed

-6

lines changed

mm
- mprotect.c

1 file changed

-6

lines changed

`‎mm/mprotect.c‎`

Lines changed: 9 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ static void change_pte_range(struct mm_struct mm, pmd_t pmd,`
`78`	`78`	`pte_unmap_unlock(pte-1,ptl);`
`79`	`79`	`}`
`80`	`80`
`81`		`-staticinlinevoidchange_pmd_range(structmm_structmm,pud_tpud,`
	`81`	`+staticinlinevoidchange_pmd_range(structvm_area_structvma,pud_tpud,`
`82`	`82`	`unsigned longaddr,unsigned longend,pgprot_tnewprot,`
`83`	`83`	`intdirty_accountable)`
`84`	`84`	`{`
`@@ -88,14 +88,15 @@ static inline void change_pmd_range(struct mm_struct mm, pud_t pud,`
`88`	`88`	`pmd=pmd_offset(pud,addr);`
`89`	`89`	`do {`
`90`	`90`	`next=pmd_addr_end(addr,end);`
`91`		`-split_huge_page_pmd(mm,pmd);`
	`91`	`+split_huge_page_pmd(vma->vm_mm,pmd);`
`92`	`92`	`if (pmd_none_or_clear_bad(pmd))`
`93`	`93`	`continue;`
`94`		`-change_pte_range(mm,pmd,addr,next,newprot,dirty_accountable);`
	`94`	`+change_pte_range(vma->vm_mm,pmd,addr,next,newprot,`
	`95`	`+dirty_accountable);`
`95`	`96`	`}while (pmd++,addr=next,addr!=end);`
`96`	`97`	`}`
`97`	`98`
`98`		`-staticinlinevoidchange_pud_range(structmm_structmm,pgd_tpgd,`
	`99`	`+staticinlinevoidchange_pud_range(structvm_area_structvma,pgd_tpgd,`
`99`	`100`	`unsigned longaddr,unsigned longend,pgprot_tnewprot,`
`100`	`101`	`intdirty_accountable)`
`101`	`102`	`{`
`@@ -107,7 +108,8 @@ static inline void change_pud_range(struct mm_struct mm, pgd_t pgd,`
`107`	`108`	`next=pud_addr_end(addr,end);`
`108`	`109`	`if (pud_none_or_clear_bad(pud))`
`109`	`110`	`continue;`
`110`		`-change_pmd_range(mm,pud,addr,next,newprot,dirty_accountable);`
	`111`	`+change_pmd_range(vma,pud,addr,next,newprot,`
	`112`	`+dirty_accountable);`
`111`	`113`	`}while (pud++,addr=next,addr!=end);`
`112`	`114`	`}`
`113`	`115`
`@@ -127,7 +129,8 @@ static void change_protection(struct vm_area_struct *vma,`
`127`	`129`	`next=pgd_addr_end(addr,end);`
`128`	`130`	`if (pgd_none_or_clear_bad(pgd))`
`129`	`131`	`continue;`
`130`		`-change_pud_range(mm,pgd,addr,next,newprot,dirty_accountable);`
	`132`	`+change_pud_range(vma,pgd,addr,next,newprot,`
	`133`	`+dirty_accountable);`
`131`	`134`	`}while (pgd++,addr=next,addr!=end);`
`132`	`135`	`flush_tlb_range(vma,start,end);`
`133`	`136`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb36f5b0

File tree

1 file changed

1 file changed

`‎mm/mprotect.c‎`

0 commit comments