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