On Wed 20-06-18 15:38:36, Johannes Weiner wrote:
> On Wed, Jun 20, 2018 at 05:31:48PM +0200, Michal Hocko wrote:
> > * Please note that mem_cgroup_oom_synchronize might fail to find a
> > * victim and then we have rely on mem_cgroup_oom_synchronize otherwise
> > * we would fall back to the global oom killer in
> > pagefault_out_of_memory
>
> I can't quite figure out what this paragraph is trying to
> say. "oom_synchronize might fail [...] and we have to rely on
> oom_synchronize". Hm?
heh, vim autocompletion + a stale comment from the previous
implementation which ENOMEM on the fail path. I went with
* Please note that mem_cgroup_out_of_memory might fail to find a
* victim and then we have to bail out from the charge path.
--
Michal Hocko
SUSE Labs