Re: [PATCH v4 53/66] mm/madvise: Use vma_find() instead of vma linked list

[Vlastimil Babka]

On 12/1/21 15:30, Liam Howlett wrote:
> From: "Liam R. Howlett" <Liam.Howlett@xxxxxxxxxx>
> 
> Signed-off-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx>

Acked-by: Vlastimil Babka <vbabka@xxxxxxx>

> ---
>  mm/madvise.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mm/madvise.c b/mm/madvise.c
> index 8c927202bbe6..cb6daa98e5b1 100644
> --- a/mm/madvise.c
> +++ b/mm/madvise.c
> @@ -1209,7 +1209,7 @@ int do_madvise(struct mm_struct *mm, unsigned long start, size_t len_in, int beh
>  		if (start >= end)
>  			goto out;
>  		if (prev)
> -			vma = prev->vm_next;
> +			vma = find_vma(mm, prev->vm_end);
>  		else	/* madvise_remove dropped mmap_lock */
>  			vma = find_vma(mm, start);
>  	}