2022-08-22 15:41:50

by Liam R. Howlett

[permalink] [raw]
Subject: [PATCH v13 60/70] mm/msync: use vma_find() instead of vma linked list

From: "Liam R. Howlett" <[email protected]>

Signed-off-by: Liam R. Howlett <[email protected]>
Acked-by: Vlastimil Babka <[email protected]>
---
mm/msync.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/msync.c b/mm/msync.c
index 137d1c104f3e..ac4c9bfea2e7 100644
--- a/mm/msync.c
+++ b/mm/msync.c
@@ -104,7 +104,7 @@ SYSCALL_DEFINE3(msync, unsigned long, start, size_t, len, int, flags)
error = 0;
goto out_unlock;
}
- vma = vma->vm_next;
+ vma = find_vma(mm, vma->vm_end);
}
}
out_unlock:
--
2.35.1


2022-08-31 04:06:31

by Davidlohr Bueso

[permalink] [raw]
Subject: Re: [PATCH v13 60/70] mm/msync: use vma_find() instead of vma linked list

On Mon, 22 Aug 2022, Liam Howlett wrote:

>From: "Liam R. Howlett" <[email protected]>

-ENOCHANGELOG

>
>Signed-off-by: Liam R. Howlett <[email protected]>
>Acked-by: Vlastimil Babka <[email protected]>

... but looks good:

Reviewed-by: Davidlohr Bueso <[email protected]>

>---
> mm/msync.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/mm/msync.c b/mm/msync.c
>index 137d1c104f3e..ac4c9bfea2e7 100644
>--- a/mm/msync.c
>+++ b/mm/msync.c
>@@ -104,7 +104,7 @@ SYSCALL_DEFINE3(msync, unsigned long, start, size_t, len, int, flags)
> error = 0;
> goto out_unlock;
> }
>- vma = vma->vm_next;
>+ vma = find_vma(mm, vma->vm_end);
> }
> }
> out_unlock:
>
>--
>2.35.1
>