[v13] Introducing the Maple Tree

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

Posted by Liam Howlett 3 years, 7 months ago

From: "Liam R. Howlett" <Liam.Howlett@Oracle.com>

Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
---
 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

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

Posted by Davidlohr Bueso 3 years, 7 months ago

On Mon, 22 Aug 2022, Liam Howlett wrote:

>From: "Liam R. Howlett" <Liam.Howlett@Oracle.com>

-ENOCHANGELOG

>
>Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
>Acked-by: Vlastimil Babka <vbabka@suse.cz>

... but looks good:

Reviewed-by: Davidlohr Bueso <dave@stgolabs.net>

>---
> 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
>