Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Liam R. Howlett
* Jeff Xu [240821 12:33]: > On Wed, Aug 21, 2024 at 9:24 AM Pedro Falcato wrote: > > > > On Wed, Aug 21, 2024 at 5:16 PM Jeff Xu wrote: > > > > > > On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato > > > wrote: > > > > > > > > We were doing an extra mmap tree traversal just to check if the entire

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Lorenzo Stoakes
On Wed, Aug 21, 2024 at 09:33:06AM GMT, Jeff Xu wrote: > On Wed, Aug 21, 2024 at 9:24 AM Pedro Falcato wrote: > > > > On Wed, Aug 21, 2024 at 5:16 PM Jeff Xu wrote: > > > > > > On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato > > > wrote: > > > > > > > > We were doing an extra mmap tree traversal

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Lorenzo Stoakes
On Wed, Aug 21, 2024 at 09:15:52AM GMT, Jeff Xu wrote: > On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato wrote: > > > > We were doing an extra mmap tree traversal just to check if the entire > > range is modifiable. This can be done when we iterate through the VMAs > > instead. > > > > Signed-off-by

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Jeff Xu
On Wed, Aug 21, 2024 at 9:24 AM Pedro Falcato wrote: > > On Wed, Aug 21, 2024 at 5:16 PM Jeff Xu wrote: > > > > On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato > > wrote: > > > > > > We were doing an extra mmap tree traversal just to check if the entire > > > range is modifiable. This can be done

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Pedro Falcato
On Wed, Aug 21, 2024 at 5:16 PM Jeff Xu wrote: > > On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato wrote: > > > > We were doing an extra mmap tree traversal just to check if the entire > > range is modifiable. This can be done when we iterate through the VMAs > > instead. > > > > Signed-off-by: Ped

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-21 Thread Jeff Xu
On Fri, Aug 16, 2024 at 5:18 PM Pedro Falcato wrote: > > We were doing an extra mmap tree traversal just to check if the entire > range is modifiable. This can be done when we iterate through the VMAs > instead. > > Signed-off-by: Pedro Falcato > --- > mm/mmap.c | 11 +-- > mm/vma.c | 1

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-20 Thread Lorenzo Stoakes
On Sat, Aug 17, 2024 at 01:18:29AM GMT, Pedro Falcato wrote: > We were doing an extra mmap tree traversal just to check if the entire > range is modifiable. This can be done when we iterate through the VMAs > instead. > > Signed-off-by: Pedro Falcato > --- > mm/mmap.c | 11 +-- > mm/vma.c

Re: [PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-19 Thread Liam R. Howlett
* Pedro Falcato [240816 20:18]: > We were doing an extra mmap tree traversal just to check if the entire > range is modifiable. This can be done when we iterate through the VMAs > instead. > > Signed-off-by: Pedro Falcato Although this has the side effect of potentially splitting the first vma

[PATCH v3 2/7] mm/munmap: Replace can_modify_mm with can_modify_vma

2024-08-16 Thread Pedro Falcato
We were doing an extra mmap tree traversal just to check if the entire range is modifiable. This can be done when we iterate through the VMAs instead. Signed-off-by: Pedro Falcato --- mm/mmap.c | 11 +-- mm/vma.c | 19 --- 2 files changed, 13 insertions(+), 17 deletions(