Re: [PATCH v2 1/7] mm: remove arch_flush_lazy_mmu_mode()

2025-09-19 Thread Yeoreum Yun
Reviewed-by: Yeoreum Yun On Mon, Sep 08, 2025 at 08:39:25AM +0100, Kevin Brodsky wrote: > This function has only ever been used in arch/x86, so there is no > need for other architectures to implement it. Remove it from > linux/pgtable.h and all architectures besides x86. >

Re: [PATCH v2 2/7] mm: introduce local state for lazy_mmu sections

2025-09-08 Thread Yeoreum Yun
Reviewed-by: Yeoreum Yun On Mon, Sep 08, 2025 at 08:39:26AM +0100, Kevin Brodsky wrote: > arch_{enter,leave}_lazy_mmu_mode() currently have a stateless API > (taking and returning no value). This is proving problematic in > situations where leave() needs to restore some context ba

Re: [PATCH v2 3/7] arm64: mm: fully support nested lazy_mmu sections

2025-09-08 Thread Yeoreum Yun
Reviewed-by: Yeoreum Yun On Mon, Sep 08, 2025 at 08:39:27AM +0100, Kevin Brodsky wrote: > Despite recent efforts to prevent lazy_mmu sections from nesting, it > remains difficult to ensure that it never occurs - and in fact it > does occur on arm64 in certain situations (CONFIG_DEBUG_

Re: [PATCH v2 7/7] mm: update lazy_mmu documentation

2025-09-08 Thread Yeoreum Yun
Reviewed-by: Yeoreum Yun On Mon, Sep 08, 2025 at 08:39:31AM +0100, Kevin Brodsky wrote: > We now support nested lazy_mmu sections on all architectures > implementing the API. Update the API comment accordingly. > > Acked-by: Mike Rapoport (Microsoft) > Signed-off-by

Re: [PATCH 2/7] mm: introduce local state for lazy_mmu sections

2025-09-05 Thread Yeoreum Yun
But here, the usage is like: static int kasan_populate_vmalloc_pte() { ... arch_leave_lazy_mmu_mode(); ... arch_enter_lazy_mmu_mode(); ... } Might be you can call the arch_leave_lazy_mmu_mode() with LAZY_MMU_DEFAULT in here since I think kasan_populate_vmalloc_pte() wouldn't be called nestly. [...] Thanks. -- Sincerely, Yeoreum Yun