Messages in this thread | | | Date | Wed, 20 Jun 2018 17:34:38 +0200 | From | Michal Hocko <> | Subject | Re: [RFC PATCH] memcg, oom: move out_of_memory back to the charge path |
| |
On Wed 20-06-18 17:31:48, Michal Hocko wrote: > On Wed 20-06-18 11:18:12, Johannes Weiner wrote: [...] > > 1) Why warn for kernel allocations, but not userspace ones? This > > should have a comment at least. > > I am not sure I understand. We do warn for all allocations types of > mem_cgroup_out_of_memory fails as long as we are not in a legacy - > oom_disabled case.
OK, I can see it now. It wasn't in the quoted context and I just forgot that WARN(!current->memcg_may_oom, ...). Well, I do not remember why I've made it conditional and you are right it doesn't make any sense. Probably a different code flow back then.
Updated to warn regardless of memcg_may_oom. -- Michal Hocko SUSE Labs
| |