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 X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 14F85C5ACC1 for ; Tue, 17 Mar 2020 05:42:25 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id C642120719 for ; Tue, 17 Mar 2020 05:42:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="HUFtryvr" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C642120719 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 7B1FA6B000C; Tue, 17 Mar 2020 01:42:24 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 761A76B000D; Tue, 17 Mar 2020 01:42:24 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 678376B000E; Tue, 17 Mar 2020 01:42:24 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0253.hostedemail.com [216.40.44.253]) by kanga.kvack.org (Postfix) with ESMTP id 503AC6B000C for ; Tue, 17 Mar 2020 01:42:24 -0400 (EDT) Received: from smtpin22.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 3556E8248076 for ; Tue, 17 Mar 2020 05:42:24 +0000 (UTC) X-FDA: 76603759008.22.glove98_6c3ff8fbcda21 X-HE-Tag: glove98_6c3ff8fbcda21 X-Filterd-Recvd-Size: 5013 Received: from mail-pl1-f194.google.com (mail-pl1-f194.google.com [209.85.214.194]) by imf22.hostedemail.com (Postfix) with ESMTP for ; Tue, 17 Mar 2020 05:42:23 +0000 (UTC) Received: by mail-pl1-f194.google.com with SMTP id r3so1074043pls.13 for ; Mon, 16 Mar 2020 22:42:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=9wb/P3LPDlbugq1CDyVA5kSUSv6vOvUhfJXNTDD+0wY=; b=HUFtryvrggxqsdgMcaIJkI5PlHQpf/d426P43SkE68+uCxMmRxAgW29VbfpzX8Gv1c yuYoqfiyoGFGojyZCxXmM7OdvkkEXwLDG1qua3NoH6gYyqfvI/Vymne5r4jy+y1dTkVo nySuJtnMji4kGXAhfr83BtEllAi49vmjFpF4vBMshA9AF8T1dzHb/w83+KamUimjlTYC HTSHtUEYdZMdjf5oRc3H4Fz3Bcr2Mp2nEBTzMczPYr3oXCfXmuDp+u6mrelq1LwZH3fy RQrw6sd6fy44g54QZkG9MuAF87HzmemOJ1CEOBNoJcf9fHC0kPtqNoy6Uq5VzwwuVPLg +aXw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=9wb/P3LPDlbugq1CDyVA5kSUSv6vOvUhfJXNTDD+0wY=; b=s9RlOviSC4sfiFT/pdUkCnZIoCFWNiRhqQlRURB1nuzJmNlDnUAIMCgjxP9pi9Doea 7iDfuzdbQQuYMffGSC7i2STSjYvH+UqF6+k2pNMxiJr2tFCYNcD2pcskMy6fFlVSj0zK 8nEfQ1pJ8VVokck9sannbtDRUX1q+Q+ynWem8Q4NqL0n9diywjsK8hW/cG2Vhi/fU9Dy 43njNer4KGeYDV5lgZHuBuDepVzkN809DshUK4MBX5ivYHPXMMzPk5Q/GeHJI0ZgkYsO dtjds4hAaCZrCwiigrAZR6iXjfVguwGqbvQ1DsECZiAcJY5F9y1LgkuaBFa9hh6D9QRh eh/Q== X-Gm-Message-State: ANhLgQ1MuqerSta/1bkPoNrPX4Cif7bK2Sg5iLBA8HtLVNPbgqDfiehN DEUM5/AYA6rKDPlpp2I7qUE= X-Google-Smtp-Source: ADFU+vvuskdqe81LJ9kaYHhwm+SdnxkieUhqPLtJJEHaTvIKEEhTmeUaX7U2gRjxYIgNNDdxeo6XbQ== X-Received: by 2002:a17:90a:d80b:: with SMTP id a11mr3492749pjv.21.1584423742470; Mon, 16 Mar 2020 22:42:22 -0700 (PDT) Received: from localhost.localdomain ([114.206.198.176]) by smtp.gmail.com with ESMTPSA id i21sm1141757pgn.5.2020.03.16.22.42.19 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 16 Mar 2020 22:42:21 -0700 (PDT) From: js1304@gmail.com X-Google-Original-From: iamjoonsoo.kim@lge.com To: Andrew Morton Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Johannes Weiner , Michal Hocko , Hugh Dickins , Minchan Kim , Vlastimil Babka , Mel Gorman , kernel-team@lge.com, Joonsoo Kim Subject: [PATCH v3 5/9] mm/workingset: use the node counter if memcg is the root memcg Date: Tue, 17 Mar 2020 14:41:53 +0900 Message-Id: <1584423717-3440-6-git-send-email-iamjoonsoo.kim@lge.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1584423717-3440-1-git-send-email-iamjoonsoo.kim@lge.com> References: <1584423717-3440-1-git-send-email-iamjoonsoo.kim@lge.com> 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: From: Joonsoo Kim In the following patch, workingset detection is implemented for the swap cache. Swap cache's node is usually allocated by kswapd and it isn't charged by kmemcg since it is from the kernel thread. So the swap cache's shadow node is managed by the node list of the list_lru rather than the memcg specific one. If counting the shadow node on the root memcg happens to reclaim the slab object, the shadow node count returns the number of the shadow node on the node list of the list_lru since root memcg has the kmem_cache_id, -1. However, the size of pages on the LRU is calculated by using the specific memcg, so mismatch happens. This causes the number of shadow node not to be increased to the enough size and, therefore, workingset detection cannot work correctly. This patch fixes this bug by checking if the memcg is the root memcg or not. If it is the root memcg, instead of using the memcg-specific LRU, the system-wide LRU is used to calculate proper size of the shadow node so that the number of the shadow node can grow as expected. Signed-off-by: Joonsoo Kim --- mm/workingset.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/mm/workingset.c b/mm/workingset.c index 5fb8f85..a9f474a 100644 --- a/mm/workingset.c +++ b/mm/workingset.c @@ -468,7 +468,13 @@ static unsigned long count_shadow_nodes(struct shrinker *shrinker, * PAGE_SIZE / xa_nodes / node_entries * 8 / PAGE_SIZE */ #ifdef CONFIG_MEMCG - if (sc->memcg) { + /* + * Kernel allocation on root memcg isn't regarded as allocation of + * specific memcg. So, if sc->memcg is the root memcg, we need to + * use the count for the node rather than one for the specific + * memcg. + */ + if (sc->memcg && !mem_cgroup_is_root(sc->memcg)) { struct lruvec *lruvec; int i; -- 2.7.4