Re: [PATCH v4] mm/rmap: synchronize lock and unlock target in anon_vma_clone

On Mon, Aug 17, 2026 at 08:43:30AM +0100, Lorenzo Stoakes (ARM) wrote:
> On Mon, Aug 17, 2026 at 02:25:11PM +0900, Eric Kim wrote:
> > ---
> > v2:
> > - Clarify the commit message to explain that src->anon_vma and
> >   active_anon_vma refer to the same anon_vma.
> > v3:
> > - Carry over review tags from v2.
> > v4:
> > - Clarify ctive_anon_vma and src->anon_vma refer to the same
> >   anon_vma.
>
> Extremely petty nit - and you don't have to do anything this is just for the
> future:
>
> - Prefer a reverse order, so latest revision at the top.
> - Nice to have lore links to previous revisions.

Oh and also - nice to attribute the changes to the person who requested them,
e.g. 'Clarify xxx as per Barry Song', etc. thanks! :)

--
Cheers, Lorenzo