On Wed, Jan 16, 2019 at 08:00:26AM -0800, Davidlohr Bueso wrote:
> On Tue, 15 Jan 2019, Jason Gunthorpe wrote:
>
> > On Tue, Jan 15, 2019 at 01:12:07PM -0800, Matthew Wilcox wrote:
> > > On Tue, Jan 15, 2019 at 08:53:16PM +0000, Jason Gunthorpe wrote:
> > > > > - new_pinned = atomic_long_read(&mm->pinned_vm) + npages;
> > > > > + new_pinned = atomic_long_add_return(npages, &mm->pinned_vm);
> > > > > if (new_pinned > lock_limit && !capable(CAP_IPC_LOCK)) {
> > > >
> > > > I thought a patch had been made for this to use check_overflow...
> > >
> > > That got removed again by patch 1 ...
> >
> > Well, that sure needs a lot more explanation. :(
>
> What if we just make the counter atomic64?
That could work.
Jason
