On Tue, Jan 31, 2023 at 10:54:22AM -0800, Suren Baghdasaryan wrote:
> On Tue, Jan 31, 2023 at 12:32 AM Hyeonggon Yoo <42.hye...@gmail.com> wrote:
> >
> > On Thu, Jan 26, 2023 at 11:37:49AM -0800, Suren Baghdasaryan wrote:
> > > Replace direct modifications to vma->vm_flags with calls to modifier
>
On Tue, Jan 31, 2023 at 3:12 PM Andrew Morton wrote:
>
> On Tue, 31 Jan 2023 13:08:19 -0800 Suren Baghdasaryan
> wrote:
>
> > On Tue, Jan 31, 2023 at 12:54 PM Andrew Morton
> > wrote:
> > >
> > > On Tue, 31 Jan 2023 10:54:22 -0800 Suren Baghdasaryan
> > > wrote:
> > >
> > > > > > -
On Tue, 31 Jan 2023 13:08:19 -0800 Suren Baghdasaryan wrote:
> On Tue, Jan 31, 2023 at 12:54 PM Andrew Morton
> wrote:
> >
> > On Tue, 31 Jan 2023 10:54:22 -0800 Suren Baghdasaryan
> > wrote:
> >
> > > > > - vma->vm_flags &= ~VM_MAYWRITE;
> > > > > + vm_flags_clear(vma,
On Tue, Jan 31, 2023 at 12:54 PM Andrew Morton
wrote:
>
> On Tue, 31 Jan 2023 10:54:22 -0800 Suren Baghdasaryan
> wrote:
>
> > > > - vma->vm_flags &= ~VM_MAYWRITE;
> > > > + vm_flags_clear(vma, VM_MAYSHARE);
> > > > }
> > >
> > > I think it should be:
> > >
On Tue, 31 Jan 2023 10:54:22 -0800 Suren Baghdasaryan wrote:
> > > - vma->vm_flags &= ~VM_MAYWRITE;
> > > + vm_flags_clear(vma, VM_MAYSHARE);
> > > }
> >
> > I think it should be:
> > s/VM_MAYSHARE/VM_MAYWRITE/
>
I added the fixup. Much better than resendi
On Tue, Jan 31, 2023 at 12:32 AM Hyeonggon Yoo <42.hye...@gmail.com> wrote:
>
> On Thu, Jan 26, 2023 at 11:37:49AM -0800, Suren Baghdasaryan wrote:
> > Replace direct modifications to vma->vm_flags with calls to modifier
> > functions to be able to track flag changes and to keep vma locking
> > cor
On Thu, Jan 26, 2023 at 11:37:49AM -0800, Suren Baghdasaryan wrote:
> Replace direct modifications to vma->vm_flags with calls to modifier
> functions to be able to track flag changes and to keep vma locking
> correctness.
>
> Signed-off-by: Suren Baghdasaryan
> Acked-by: Michal Hocko
> Acked-by
* Suren Baghdasaryan [230126 14:38]:
> Replace direct modifications to vma->vm_flags with calls to modifier
> functions to be able to track flag changes and to keep vma locking
> correctness.
>
That was long, but it all looks good!
Reviewed-by: Liam R. Howlett
> Signed-off-by: Suren Baghdasar
Replace direct modifications to vma->vm_flags with calls to modifier
functions to be able to track flag changes and to keep vma locking
correctness.
Signed-off-by: Suren Baghdasaryan
Acked-by: Michal Hocko
Acked-by: Mel Gorman
Acked-by: Mike Rapoport (IBM)
Acked-by: Sebastian Reichel
---
arc