| Message ID | CAG48ez1GM==OnHpS=ghqZNJPn02FCDUEHc7GQmGRMXUD_aKudg@mail.gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/4] mm/gup_benchmark: Take the mmap lock around GUP | expand |
On Tue, Sep 29, 2020 at 06:20:01PM -0700, Jann Horn wrote: > After having cleaned up all GUP callers (except for the atomisp staging > driver, which currently gets mmap locking completely wrong [1]) to always > ensure that they hold the mmap lock when calling into GUP (unless the mm is > not yet globally visible), add an assertion to make sure it stays that way > going forward. > > [1] https://lore.kernel.org/lkml/CAG48ez3tZAb9JVhw4T5e-i=h2_DUZxfNRTDsagSRCVazNXx5qA@mail.gmail.com/ > > Signed-off-by: Jann Horn <jannh@google.com> > --- > mm/gup.c | 2 ++ > 1 file changed, 2 insertions(+) I'm happy to see this, I have observed many cases of missing locking here. Reviewed-by: Jason Gunthorpe <jgg@nvidia.com> Thanks, Jason
On Wed, Sep 30, 2020 at 5:32 AM Jason Gunthorpe <jgg@ziepe.ca> wrote: > On Tue, Sep 29, 2020 at 06:20:01PM -0700, Jann Horn wrote: > > After having cleaned up all GUP callers (except for the atomisp staging > > driver, which currently gets mmap locking completely wrong [1]) to always > > ensure that they hold the mmap lock when calling into GUP (unless the mm is > > not yet globally visible), add an assertion to make sure it stays that way > > going forward. Thanks for doing this, there is a lot of value in ensuring that a function's callers follows the prerequisites. Acked-by: Michel Lespinasse <walken@google.com>
diff --git a/mm/gup.c b/mm/gup.c index f11d39867cf5..3e5d843215b9 100644 --- a/mm/gup.c +++ b/mm/gup.c @@ -1020,6 +1020,8 @@ static long __get_user_pages(struct mm_struct *mm, struct vm_area_struct *vma = NULL; struct follow_page_context ctx = { NULL }; + mmap_assert_locked(mm); + if (!nr_pages) return 0;
After having cleaned up all GUP callers (except for the atomisp staging driver, which currently gets mmap locking completely wrong [1]) to always ensure that they hold the mmap lock when calling into GUP (unless the mm is not yet globally visible), add an assertion to make sure it stays that way going forward. [1] https://lore.kernel.org/lkml/CAG48ez3tZAb9JVhw4T5e-i=h2_DUZxfNRTDsagSRCVazNXx5qA@mail.gmail.com/ Signed-off-by: Jann Horn <jannh@google.com> --- mm/gup.c | 2 ++ 1 file changed, 2 insertions(+)