On Wed, Sep 30, 2020 at 5:32 AM Jason Gunthorpe <jgg@xxxxxxxx> 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@xxxxxxxxxx>