| Message ID | 20190121075722.7945-6-peterx@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | userfaultfd: write protection support | expand |
On Mon, Jan 21, 2019 at 03:57:03PM +0800, Peter Xu wrote: > From: Shaohua Li <shli@fb.com> > > add helper for writeprotect check. Will use it later. I'd merge this with the commit that actually uses this helper. > Cc: Andrea Arcangeli <aarcange@redhat.com> > Cc: Pavel Emelyanov <xemul@parallels.com> > Cc: Rik van Riel <riel@redhat.com> > Cc: Kirill A. Shutemov <kirill@shutemov.name> > Cc: Mel Gorman <mgorman@suse.de> > Cc: Hugh Dickins <hughd@google.com> > Cc: Johannes Weiner <hannes@cmpxchg.org> > Signed-off-by: Shaohua Li <shli@fb.com> > Signed-off-by: Andrea Arcangeli <aarcange@redhat.com> > Signed-off-by: Peter Xu <peterx@redhat.com> > --- > include/linux/userfaultfd_k.h | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/include/linux/userfaultfd_k.h b/include/linux/userfaultfd_k.h > index 37c9eba75c98..38f748e7186e 100644 > --- a/include/linux/userfaultfd_k.h > +++ b/include/linux/userfaultfd_k.h > @@ -50,6 +50,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) > return vma->vm_flags & VM_UFFD_MISSING; > } > > +static inline bool userfaultfd_wp(struct vm_area_struct *vma) > +{ > + return vma->vm_flags & VM_UFFD_WP; > +} > + > static inline bool userfaultfd_armed(struct vm_area_struct *vma) > { > return vma->vm_flags & (VM_UFFD_MISSING | VM_UFFD_WP); > @@ -94,6 +99,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) > return false; > } > > +static inline bool userfaultfd_wp(struct vm_area_struct *vma) > +{ > + return false; > +} > + > static inline bool userfaultfd_armed(struct vm_area_struct *vma) > { > return false; > -- > 2.17.1 >
On Mon, Jan 21, 2019 at 12:23:12PM +0200, Mike Rapoport wrote: > On Mon, Jan 21, 2019 at 03:57:03PM +0800, Peter Xu wrote: > > From: Shaohua Li <shli@fb.com> > > > > add helper for writeprotect check. Will use it later. > > I'd merge this with the commit that actually uses this helper. Hi, Mike, Yeah actually that's what I'd prefer for most of the time. But I'm trying to avoid doing that because I wanted to keep the credit of the original authors, not only for this single patch, but also for the whole series. Meanwhile, since this work has been there for quite a few years (starting from 2015), IMHO keeping the old patches mostly untouched at least in the RFC stage might also help the reviewers if they have read or prior knowledge of the previous work. And if the patch cannot even stand on itself (this one can; it only introduces new functions) I'll do the merge no matter what. Please correct me if this is not the good way to do. Thanks! > > > Cc: Andrea Arcangeli <aarcange@redhat.com> > > Cc: Pavel Emelyanov <xemul@parallels.com> > > Cc: Rik van Riel <riel@redhat.com> > > Cc: Kirill A. Shutemov <kirill@shutemov.name> > > Cc: Mel Gorman <mgorman@suse.de> > > Cc: Hugh Dickins <hughd@google.com> > > Cc: Johannes Weiner <hannes@cmpxchg.org> > > Signed-off-by: Shaohua Li <shli@fb.com> > > Signed-off-by: Andrea Arcangeli <aarcange@redhat.com> > > Signed-off-by: Peter Xu <peterx@redhat.com> > > --- > > include/linux/userfaultfd_k.h | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/include/linux/userfaultfd_k.h b/include/linux/userfaultfd_k.h > > index 37c9eba75c98..38f748e7186e 100644 > > --- a/include/linux/userfaultfd_k.h > > +++ b/include/linux/userfaultfd_k.h > > @@ -50,6 +50,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) > > return vma->vm_flags & VM_UFFD_MISSING; > > } > > > > +static inline bool userfaultfd_wp(struct vm_area_struct *vma) > > +{ > > + return vma->vm_flags & VM_UFFD_WP; > > +} > > + > > static inline bool userfaultfd_armed(struct vm_area_struct *vma) > > { > > return vma->vm_flags & (VM_UFFD_MISSING | VM_UFFD_WP); > > @@ -94,6 +99,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) > > return false; > > } > > > > +static inline bool userfaultfd_wp(struct vm_area_struct *vma) > > +{ > > + return false; > > +} > > + > > static inline bool userfaultfd_armed(struct vm_area_struct *vma) > > { > > return false; > > -- > > 2.17.1 > > > > -- > Sincerely yours, > Mike. > Regards,
diff --git a/include/linux/userfaultfd_k.h b/include/linux/userfaultfd_k.h index 37c9eba75c98..38f748e7186e 100644 --- a/include/linux/userfaultfd_k.h +++ b/include/linux/userfaultfd_k.h @@ -50,6 +50,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) return vma->vm_flags & VM_UFFD_MISSING; } +static inline bool userfaultfd_wp(struct vm_area_struct *vma) +{ + return vma->vm_flags & VM_UFFD_WP; +} + static inline bool userfaultfd_armed(struct vm_area_struct *vma) { return vma->vm_flags & (VM_UFFD_MISSING | VM_UFFD_WP); @@ -94,6 +99,11 @@ static inline bool userfaultfd_missing(struct vm_area_struct *vma) return false; } +static inline bool userfaultfd_wp(struct vm_area_struct *vma) +{ + return false; +} + static inline bool userfaultfd_armed(struct vm_area_struct *vma) { return false;