Re: [PATCH v5 09/17] arm64: pgtable: move pagetable_dtor() to __tlb_remove_table()
Will Deacon <[email protected]>
| Newsgroups | org.kernel.vger.linux-hexagon,dev.linux.lists.loongarch,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-riscv,org.infradead.lists.linux-um,org.kernel.vger.linux-arch,org.kernel.vger.linux-csky,org.kernel.vger.linux-kernel,org.kernel.vger.linux-m68k,org.kernel.vger.linux-mips,org.kernel.vger.linux-openrisc,org.kernel.vger.linux-s390,org.kernel.vger.linux-sh,org.kernel.vger.sparclinux,org.kvack.linux-mm,org.ozlabs.lists.linuxppc-dev |
|---|---|
| Message-ID | <20250113162931.GB14101@willie-the-truck> |
On Wed, Jan 08, 2025 at 02:57:25PM +0800, Qi Zheng wrote: > Move pagetable_dtor() to __tlb_remove_table(), so that ptlock and page > table pages can be freed together (regardless of whether RCU is used). > This prevents the use-after-free problem where the ptlock is freed > immediately but the page table pages is freed later via RCU. > > Page tables shouldn't have swap cache, so use pagetable_free() instead of > free_page_and_swap_cache() to free page table pages. > > Signed-off-by: Qi Zheng <[email protected]> > Suggested-by: Peter Zijlstra (Intel) <[email protected]> > Reviewed-by: Kevin Brodsky <[email protected]> > Cc: [email protected] > --- > arch/arm64/include/asm/tlb.h | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) Acked-by: Will Deacon <[email protected]> Will