Re: [PATCH] memcg, thp: do not invoke oom killer on thp charges

From: David Rientjes
Date: Wed Mar 21 2018 - 17:22:22 EST


On Wed, 21 Mar 2018, Michal Hocko wrote:

> diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> index d1a917b5b7b7..08accbcd1a18 100644
> --- a/mm/memcontrol.c
> +++ b/mm/memcontrol.c
> @@ -1493,7 +1493,7 @@ static void memcg_oom_recover(struct mem_cgroup *memcg)
>
> static void mem_cgroup_oom(struct mem_cgroup *memcg, gfp_t mask, int order)
> {
> - if (!current->memcg_may_oom)
> + if (!current->memcg_may_oom || order > PAGE_ALLOC_COSTLY_ORDER)
> return;
> /*
> * We are in the middle of the charge context here, so we

What bug reports have you received about order-4 and higher order non thp
charges that this fixes?

The patch title and the changelog specifically single out thp, which I've
fixed, since it has sane fallback behavior and everything else uses
__GFP_NORETRY. I think this is misusing a page allocator heuristic that
hasn't been applied to the memcg charge path before to address a thp
regression but generalizing it for all charges.

PAGE_ALLOC_COSTLY_ORDER is a heuristic used by the page allocator because
it cannot free high-order contiguous memory. Memcg just needs to reclaim
a number of pages. Two order-3 charges can cause a memcg oom kill but now
an order-4 charge cannot. It's an unfair bias against high-order charges
that are not explicitly using __GFP_NORETRY.