From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 65439E7717F for ; Fri, 13 Dec 2024 00:32:23 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id CB3106B0095; Thu, 12 Dec 2024 19:32:22 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id C62C96B0096; Thu, 12 Dec 2024 19:32:22 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B51626B0098; Thu, 12 Dec 2024 19:32:22 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 965646B0095 for ; Thu, 12 Dec 2024 19:32:22 -0500 (EST) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 25F76AEFAC for ; Fri, 13 Dec 2024 00:32:22 +0000 (UTC) X-FDA: 82888058358.07.4432FD1 Received: from out-175.mta0.migadu.com (out-175.mta0.migadu.com [91.218.175.175]) by imf25.hostedemail.com (Postfix) with ESMTP id 2C7BAA000C for ; Fri, 13 Dec 2024 00:32:01 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=lLU7cSnH; spf=pass (imf25.hostedemail.com: domain of roman.gushchin@linux.dev designates 91.218.175.175 as permitted sender) smtp.mailfrom=roman.gushchin@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1734049917; a=rsa-sha256; cv=none; b=7U3rRRcXYSHsvhwv9ppVwbcsehuS0ln64jywBEGWzrs4/TLfeJOAAR3LB+rmd2RQB67paa Jd74eU6y7Ewqlzosl8W6TMSdGUQkyDm/vlE+9EfLjXt1vqllU34Aq1OXlOl7H1Q7xP+8wq Ni6u+qP3UTII0NiYSm8SUUkgX63bWIk= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=lLU7cSnH; spf=pass (imf25.hostedemail.com: domain of roman.gushchin@linux.dev designates 91.218.175.175 as permitted sender) smtp.mailfrom=roman.gushchin@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1734049917; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=gDknTbX0Ls82zv0uHrNtgXqzSCQIM2ghtBvYdeKv+Ws=; b=fFPXZIk7TKgzrqUqA6dgV5kT0EhL0d+Vt+ib1a+A0SFZwNOllGrM+mymDd0yip5VTRnW8p ISvzI6A1e094oXPeNV8YnSeKVcnqcutw71dBTGkwjjDZJjS0qQ2NWUW6x2+GILu3Pf+PFc eNzpMidTN/GdL88lWB6g0qMoJwJ2rDE= Date: Fri, 13 Dec 2024 00:32:11 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1734049936; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=gDknTbX0Ls82zv0uHrNtgXqzSCQIM2ghtBvYdeKv+Ws=; b=lLU7cSnHl5hgYCbx0CoAIPaOKjm+s5hEdfw7h7d+6hVK6j9BxfsJjHyh4fT+jSKdtqz9Ky JxNXRsAum8+QyuhJSzw1mIKgOhgqq7N6IPh7bAsgyhXcBDT0H5Dsmajsk+OoauC7Vqnqb9 o3NePYZKtuvcu52Pcoa/rER4oFcj3n8= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Roman Gushchin To: Johannes Weiner Cc: Yosry Ahmed , Rik van Riel , Balbir Singh , Michal Hocko , hakeel Butt , Muchun Song , Andrew Morton , cgroups@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, kernel-team@meta.com, Nhat Pham Subject: Re: [PATCH v2] memcg: allow exiting tasks to write back data to swap Message-ID: References: <20241212115754.38f798b3@fangorn> <20241212183012.GB1026@cmpxchg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20241212183012.GB1026@cmpxchg.org> X-Migadu-Flow: FLOW_OUT X-Rspamd-Queue-Id: 2C7BAA000C X-Stat-Signature: aq8rzfoxf7bmij4qaxh8kmuopiabkkjy X-Rspam-User: X-Rspamd-Server: rspam09 X-HE-Tag: 1734049921-995349 X-HE-Meta: U2FsdGVkX1+p9vxh/DwSJVq7TsVf50olRRCVogDUDLvBuhjgjjTCcTmPs1mAQiG+oFIZGJekiX8A+Tn6Ifu27eN5lkZJTaKhwiKQeJlemyXG0lLqfrUw5mf3dy+m+fcyDjhmzvN+WdAlJjBbVj0LZnXJKjYXjmjbjdUnwaQwYvWgEMRmfbj72d+lMl0ko8sZrhVd/EXViN/0CxDk+P/TpfN3N6wfQzp/jzo7Sa1YuY3U5KBL9dKUP5GgkAmf/Qvjmut+CzCSZrnEktqzUq79NL6SBIy6JZjBMlRRwsgRXb4fW4nwMNogUnSd2BK7N1Drwgx3bqSGaZqeJ7x9Hu1vlJNRCUvuf9S/UB/Aqixfs+o+lXPdb9KPVpFDG/AihYEdv811s9Te/8Q0IdeEreSAOhorOcm6w/Yfmkdzp2+0IRVrp/uOyxGF14IItYYO5rRKW/Anh8lBJYOA6n14wHfOgFABREgE5iFQiSohsJDLIlLgaHHHliuGqkyvH/r+hv/5NlQUfl/HHgh3hdGeryStmOoJtp45+fIoX61+Jga2j2zCL5NhyiiyWMFXtsG3uA8FHzkAHP0N6DqO4qeJ6AArbjg6psdcIXLIi01WTmcPRrXyhrgYws2cvJD5kW8gF1v7iXwKffp1zT9YS1CPw6TxbWNOp8v4GNXJNO9V6ypyF8tuJZvTOPhR2yVG78+abj+ycIHa1h89p2j6TVvFrpospu0jiC4XfiokWSU5veGXjfDv1uFV7NhpRKbRr3OkL4cXXiUrsXZCaQ2nhy64eeNZlka//tbYA09AEVX3fJ4xV/2c9RddgSuzgS3J5A/Qm/pv5pyZWIXwL2ysJJlDg7IaBMcHLFiuo9YT4Y7KtaMHixVnYcqiv8xjOG/vEBMFI8fQ78dcgTohwpauoBtr2xTDvZREZYPSF6FSlNtoFC8Tq5aAIvND2DVFPsmztIRaatnBdtj91cNLF5O0B2LUFBU mlyorJrE Vs8F4VEevBYz+BB51XCxC615Js0Rd792PkI0TAdTVThuCprVkTKRo1IjmOASoGsnJasVxaOPp4zkduWq5wajC1NsRFL8ATh5HcGZC7DPiZd9NHrZeqoDmAyeoXvLA85mqirxyhOwa3cGf9ArLl90WrH00eSuT9SCXWME9vHE5jL6QJtbBfLUN2njBbkgw8xeCbRJT9rx7g0htXjCCStd6X79N1GGFBiAlcqWjNHyzHOQa1mjIa+jCV9lscxruBc0wnHzoyIPW7X5OMww= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Thu, Dec 12, 2024 at 01:30:12PM -0500, Johannes Weiner wrote: > On Thu, Dec 12, 2024 at 09:06:25AM -0800, Yosry Ahmed wrote: > > On Thu, Dec 12, 2024 at 8:58 AM Rik van Riel wrote: > > > > > > A task already in exit can get stuck trying to allocate pages, if its > > > cgroup is at the memory.max limit, the cgroup is using zswap, but > > > zswap writeback is enabled, and the remaining memory in the cgroup is > > > not compressible. > > > > > > This seems like an unlikely confluence of events, but it can happen > > > quite easily if a cgroup is OOM killed due to exceeding its memory.max > > > limit, and all the tasks in the cgroup are trying to exit simultaneously. > > > > > > When this happens, it can sometimes take hours for tasks to exit, > > > as they are all trying to squeeze things into zswap to bring the group's > > > memory consumption below memory.max. > > > > > > Allowing these exiting programs to push some memory from their own > > > cgroup into swap allows them to quickly bring the cgroup's memory > > > consumption below memory.max, and exit in seconds rather than hours. > > > > > > Signed-off-by: Rik van Riel > > > > Thanks for sending a v2. > > > > I still think maybe this needs to be fixed on the memcg side, at least > > by not making exiting tasks try really hard to reclaim memory to the > > point where this becomes a problem. IIUC there could be other reasons > > why reclaim may take too long, but maybe not as pathological as this > > case to be fair. I will let the memcg maintainers chime in for this. > > > > If there's a fundamental reason why this cannot be fixed on the memcg > > side, I don't object to this change. > > > > Nhat, any objections on your end? I think your fleet workloads were > > the first users of this interface. Does this break their expectations? > > Yes, I don't think we can do this, unfortunately :( There can be a > variety of reasons for why a user might want to prohibit disk swap for > a certain cgroup, and we can't assume it's okay to make exceptions. > > There might also not *be* any disk swap to overflow into after Nhat's > virtual swap patches. Presumably zram would still have the issue too. > > So I'm also inclined to think this needs a reclaim/memcg-side fix. We > have a somewhat tumultous history of policy in that space: > > commit 7775face207922ea62a4e96b9cd45abfdc7b9840 > Author: Tetsuo Handa > Date: Tue Mar 5 15:46:47 2019 -0800 > > memcg: killed threads should not invoke memcg OOM killer > > allowed dying tasks to simply force all charges and move on. This > turned out to be too aggressive; there were instances of exiting, > uncontained memcg tasks causing global OOMs. This lead to that: > > commit a4ebf1b6ca1e011289677239a2a361fde4a88076 > Author: Vasily Averin > Date: Fri Nov 5 13:38:09 2021 -0700 > > memcg: prohibit unconditional exceeding the limit of dying tasks > > which reverted the bypass rather thoroughly. Now NO dying tasks, *not > even OOM victims*, can force charges. I am not sure this is correct, > either: > > If we return -ENOMEM to an OOM victim in a fault, the fault handler > will re-trigger OOM, which will find the existing OOM victim and do > nothing, then restart the fault. This is a memory deadlock. The page > allocator gives OOM victims access to reserves for that reason. > > Actually, it looks even worse. For some reason we're not triggering > OOM from dying tasks: > > ret = task_is_dying() || out_of_memory(&oc); > > Even though dying tasks are in no way privileged or allowed to exit > expediently. Why shouldn't they trigger the OOM killer like anybody > else trying to allocate memory? > > As it stands, it seems we have dying tasks getting trapped in an > endless fault->reclaim cycle; with no access to the OOM killer and no > access to reserves. Presumably this is what's going on here? > > I think we want something like this: > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 53db98d2c4a1..be6b6e72bde5 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -1596,11 +1596,7 @@ static bool mem_cgroup_out_of_memory(struct mem_cgroup *memcg, gfp_t gfp_mask, > if (mem_cgroup_margin(memcg) >= (1 << order)) > goto unlock; > > - /* > - * A few threads which were not waiting at mutex_lock_killable() can > - * fail to bail out. Therefore, check again after holding oom_lock. > - */ > - ret = task_is_dying() || out_of_memory(&oc); > + ret = out_of_memory(&oc); I like the idea, but at first glance it might reintroduce the problem fixed by 7775face2079 ("memcg: killed threads should not invoke memcg OOM killer").