Re: [PATCH 0/3] mm/vma: miss to restore vmi.index on expansion failure

[Lorenzo Stoakes <lorenzo.stoakes@oracle.com>]

On Fri, Oct 25, 2024 at 04:06:39PM -0400, Liam R. Howlett wrote:
> * Wei Yang <richard.weiyang@gmail.com> [241024 23:18]:
> > This serial starts on spotting on miss to restore vmi.index in
> > vma_merge_new_range() when expansion is failed.
> >
> > Patch 2 add a test to assert this.
> >
> > While found build error on old platform, because parameter name missing. Patch
> > 3 is to fix this.
>
> Just to make sure I understand; this series isn't happening now?

Nope.

I already folded the pertinent bit (me mistakenly doing a pointless compare and
mas->index/end write which later gets overwritten) into the mmap-region series
where we just get rid of this code altogether as it's nnnnnecessary and
horrible.

Great spot from Wei but I think this is the better approach.

>
> Thanks,
> Liam
>
> >
> > Wei Yang (3):
> >   mm/vma: miss to restore vmi.index on expansion failure
> >   tools: test vmi.index would be restored on merge failure
> >   tools: fix build error on parameter name omitted
> >
> >  mm/vma.c                         | 10 ++--
> >  tools/testing/vma/vma.c          |  8 +++-
> >  tools/testing/vma/vma_internal.h | 82 ++++++++++++++++----------------
> >  3 files changed, 55 insertions(+), 45 deletions(-)
> >
> > --
> > 2.34.1
> >
>