Re: [PATCH v4 04/15] userfaultfd: introduce mfill_get_vma() and mfill_put_vma()
From: Harry Yoo (Oracle)
Date: Fri Apr 03 2026 - 08:55:26 EST
On Thu, Apr 02, 2026 at 07:11:45AM +0300, Mike Rapoport wrote:
> From: "Mike Rapoport (Microsoft)" <rppt@xxxxxxxxxx>
>
> Split the code that finds, locks and verifies VMA from mfill_atomic() into
> a helper function.
>
> This function will be used later during refactoring of
> mfill_atomic_pte_copy().
>
> Add a counterpart mfill_put_vma() helper that unlocks the VMA and releases
> map_changing_lock.
>
> [avagin@xxxxxxxxxx: fix lock leak in mfill_get_vma()]
> Link: https://lkml.kernel.org/r/20260316173829.1126728-1-avagin@xxxxxxxxxx
> Signed-off-by: Mike Rapoport (Microsoft) <rppt@xxxxxxxxxx>
> Signed-off-by: Andrei Vagin <avagin@xxxxxxxxxx>
> ---
Looks good to me,
Reviewed-by: Harry Yoo (Oracle) <harry@xxxxxxxxxx>
--
Cheers,
Harry / Hyeonggon