Re: [PATCH] mm: remove redundant check in handle_mm_fault

From: David Hildenbrand
Date: Mon Mar 06 2023 - 08:50:30 EST


On 06.03.23 03:49, Haifeng Xu wrote:
mem_cgroup_oom_synchronize() has checked whether current memcg_in_oom is
set or not, so remove the check in handle_mm_fault().

"mem_cgroup_oom_synchronize() will returned immediately if memcg_in_oom is not set, so remove the check from handle_mm_fault()".

However, that requires now always an indirect function call -- do we care about dropping that optimization?


--
Thanks,

David / dhildenb