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 8438DC433EF for ; Mon, 23 May 2022 19:00:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 010FB6B0005; Mon, 23 May 2022 15:00:02 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id EDBC86B0006; Mon, 23 May 2022 15:00:01 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D7CF36B0007; Mon, 23 May 2022 15:00:01 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id C354D6B0005 for ; Mon, 23 May 2022 15:00:01 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay12.hostedemail.com (Postfix) with ESMTP id 940E31210A4 for ; Mon, 23 May 2022 19:00:01 +0000 (UTC) X-FDA: 79497922602.05.68F4FEB Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by imf07.hostedemail.com (Postfix) with ESMTP id 800D04003B for ; Mon, 23 May 2022 18:59:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1653332399; x=1684868399; h=message-id:subject:from:to:cc:date:in-reply-to: references:mime-version:content-transfer-encoding; bh=NFWQ8Vo8BSVG/1PeY8TJCl1o40zJKtLANlhbW40DMqw=; b=j2XRB2r2esDM94H2dph0KnwiPmAtHlrriT/EZz1eC5T6vEcWRPKSv/cC 9HzvHQLwZ6eWl+FiaqSpYDpB4tdLlyKVi1qhVPj01deaZwLp5ogGmg1hF PBFQF7kiIsG76rEfqKjYmQK4m5NbRDwz/PYj4ThTMrsKV+zz/K4/QAFAF jDvjeGppF98iLY/gahKhEQ3nWzzW0UUR+U0PR3/XQutZ3Yoi4TV+FTINr cbINWyGXfTbfla7GNXR9xhCIrGND0rbR/VnNfc/WZC8coQMUKaKlwRalJ alN+tYHhNZBWPiYCdI/1FNIUEDzTJgcbnHka+64c3V87DvxXhJG2n5Jaf Q==; X-IronPort-AV: E=McAfee;i="6400,9594,10356"; a="260930714" X-IronPort-AV: E=Sophos;i="5.91,246,1647327600"; d="scan'208";a="260930714" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 May 2022 11:59:57 -0700 X-IronPort-AV: E=Sophos;i="5.91,246,1647327600"; d="scan'208";a="548122603" Received: from schen9-mobl.amr.corp.intel.com ([10.251.3.38]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 May 2022 11:59:57 -0700 Message-ID: <3d6d559e9a23dc2dc79cf453fd64488a4a45cc1b.camel@linux.intel.com> Subject: Re: [PATCH] Revert "mm/vmscan: never demote for memcg reclaim" From: Tim Chen To: "ying.huang@intel.com" , Johannes Weiner , Dave Hansen , Yang Shi , Andrew Morton Cc: linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-team@fb.com, Zi Yan , Michal Hocko , Shakeel Butt , Roman Gushchin Date: Mon, 23 May 2022 11:59:57 -0700 In-Reply-To: References: <20220518190911.82400-1-hannes@cmpxchg.org> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 800D04003B X-Rspam-User: Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=j2XRB2r2; spf=none (imf07.hostedemail.com: domain of tim.c.chen@linux.intel.com has no SPF policy when checking 134.134.136.20) smtp.mailfrom=tim.c.chen@linux.intel.com; dmarc=pass (policy=none) header.from=intel.com X-Stat-Signature: 3mbwbm8jg1cygtaes5otg4hae489dfqp X-HE-Tag: 1653332389-782549 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: On Thu, 2022-05-19 at 15:42 +0800, ying.huang@intel.com wrote: > On Wed, 2022-05-18 at 15:09 -0400, Johannes Weiner wrote: > > This reverts commit 3a235693d3930e1276c8d9cc0ca5807ef292cf0a. > > > > Its premise was that cgroup reclaim cares about freeing memory inside > > the cgroup, and demotion just moves them around within the cgroup > > limit. Hence, pages from toptier nodes should be reclaimed directly. > > > > However, with NUMA balancing now doing tier promotions, demotion is > > part of the page aging process. Global reclaim demotes the coldest > > toptier pages to secondary memory, where their life continues and from > > which they have a chance to get promoted back. Essentially, tiered > > memory systems have an LRU order that spans multiple nodes. > > > > When cgroup reclaims pages coming off the toptier directly, there can > > be colder pages on lower tier nodes that were demoted by global > > reclaim. This is an aging inversion, not unlike if cgroups were to > > reclaim directly from the active lists while there are inactive pages. > > > > Proactive reclaim is another factor. The goal of that it is to offload > > colder pages from expensive RAM to cheaper storage. When lower tier > > memory is available as an intermediate layer, we want offloading to > > take advantage of it instead of bypassing to storage. > > > > Revert the patch so that cgroups respect the LRU order spanning the > > memory hierarchy. > > > > Of note is a specific undercommit scenario, where all cgroup limits in > > the system add up to <= available toptier memory. In that case, > > shuffling pages out to lower tiers first to reclaim them from there is > > inefficient. This is something could be optimized/short-circuited > > later on (although care must be taken not to accidentally recreate the > > aging inversion). Let's ensure correctness first. > > > > Signed-off-by: Johannes Weiner > > Cc: Dave Hansen > > Cc: "Huang, Ying" > > Cc: Yang Shi > > Cc: Zi Yan > > Cc: Michal Hocko > > Cc: Shakeel Butt > > Cc: Roman Gushchin > > Reviewed-by: "Huang, Ying" > > This is also required by Tim's DRAM partition among cgroups in tiered > sytstem. Yes, while testing cgroup demotion, I also have to revert the commit in question. Acked-by: Tim Chen > > Best Regards, > Huang, Ying > > > --- > > mm/vmscan.c | 9 ++------- > > 1 file changed, 2 insertions(+), 7 deletions(-) > > > > diff --git a/mm/vmscan.c b/mm/vmscan.c > > index c6918fff06e1..7a4090712177 100644 > > --- a/mm/vmscan.c > > +++ b/mm/vmscan.c > > @@ -528,13 +528,8 @@ static bool can_demote(int nid, struct scan_control *sc) > > { > > if (!numa_demotion_enabled) > > return false; > > - if (sc) { > > - if (sc->no_demotion) > > - return false; > > - /* It is pointless to do demotion in memcg reclaim */ > > - if (cgroup_reclaim(sc)) > > - return false; > > - } > > + if (sc && sc->no_demotion) > > + return false; > > if (next_demotion_node(nid) == NUMA_NO_NODE) > > return false; > > > > > > > > > >