Re: [patch v2] mm, oom: fix concurrent munlock and oom reaperunmap

Michal Hocko
2018-04-24T20:31:52Z

On Tue 24-04-18 13:22:45, David Rientjes wrote:
[...]
> > > My patch has passed intensive testing on both x86 and powerpc, so I'll ask
> > > that it's pushed for 4.17-rc3. Many thanks to Tetsuo for the suggestion
> > > on calling __oom_reap_task_mm() from exit_mmap().
> >
> > Yeah, but your patch does have a problem with blockable mmu notifiers
> > IIUC.
>
> What on earth are you talking about? exit_mmap() does
> mmu_notifier_release(). There are no blockable mmu notifiers.

MMF_OOM_SKIP - remember? The thing that guarantees a forward progress.
So we cannot really depend on setting MMF_OOM_SKIP if a
mmu_notifier_release blocks for an excessive/unbounded amount of time.

Look I am not really interested in disussing this to death but it would
be really _nice_ if you could calm down a bit, stop fighting for the solution
you have proposed and ignore the feedback you are getting.

There are two things to care about here. Stop the race that can blow up
and do not regress MMF_OOM_SKIP guarantee. Can we please do that.
--
Michal Hocko
SUSE Labs