Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp1449722yba; Thu, 9 May 2019 17:18:12 -0700 (PDT) X-Google-Smtp-Source: APXvYqxbXHays+m6zGznDFBpyO/1xj5gQETrIqb6Imp0Z65bC7e8zgbLeqyqvtZq4oCGD80SDJwL X-Received: by 2002:a63:3d0b:: with SMTP id k11mr9337737pga.349.1557447492004; Thu, 09 May 2019 17:18:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1557447491; cv=none; d=google.com; s=arc-20160816; b=04Ys2/1OnMEcxMJEsh5ULj5xZ3CsSpd3MkAPPLS41ll4skxclBTNJ1OL8Kp3HE9Zaa kAaq06PNXCaQQ3x6xyKwoUZRfSir55R/JDDQQNcn3AgrerRdm6gS1h6pLxmDK4Dz3kfT wZByDOYMQQrASD0i6U+RBZEIM5FujghgqhcMnLi6WkhqUGlCeHe83EH7x4VA27mKrAfP HAW1ODJngWt17mBlXq+sOelMOeZ+/GbB813JMEo+GGSpYI41ZVCsNvjCBCTk5xZPUyV3 s25kIlSF7opGNnL6iftUD84QHIKWVythkEuimzXQkFtcTBB+ygMIFnfA7cCjYmX/72m4 H3SA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from; bh=kQcAB26OKDmQZCpxfRv57kduaPGhS8fCw6twnIJnydg=; b=q4528bF+e4Lr1sVDVQiupGHXAGLvPEaPbXzMxnze9lCdZID0XigZjEIOcVnWz5TrFM wk+uEB+sSl1yoKZQzymAdLvba6y5haAnmVOFeFS8zc35BxuhAG/MjKGe6dB59WHn5BgR Ojk1hZYxUNJsteXzacw3rAAoS0773TOf67gcPay3ix+FSJpo+cMC8IssznDzvkkikzCe WTq72hFie0rHMFFFt7unoYC0TPJbsBKxY6EPGYT1jA465qis3/Aq0o+up9226axszvmv bGAKzBkH3Yf2MSj5/pJkcbxcWFob10B60WlwG5GvlKmTf92mrYThvnI2qzkuBn0KHhhe Le8Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=alibaba.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q142si5039150pfq.175.2019.05.09.17.17.55; Thu, 09 May 2019 17:18:11 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=alibaba.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726851AbfEJAQn (ORCPT + 99 others); Thu, 9 May 2019 20:16:43 -0400 Received: from out30-54.freemail.mail.aliyun.com ([115.124.30.54]:43262 "EHLO out30-54.freemail.mail.aliyun.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726694AbfEJAQm (ORCPT ); Thu, 9 May 2019 20:16:42 -0400 X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R211e4;CH=green;DM=||false|;FP=0|-1|-1|-1|0|-1|-1|-1;HT=e01e04423;MF=yang.shi@linux.alibaba.com;NM=1;PH=DS;RN=10;SR=0;TI=SMTPD_---0TRHy6CV_1557447393; Received: from e19h19392.et15sqa.tbsite.net(mailfrom:yang.shi@linux.alibaba.com fp:SMTPD_---0TRHy6CV_1557447393) by smtp.aliyun-inc.com(127.0.0.1); Fri, 10 May 2019 08:16:39 +0800 From: Yang Shi To: ying.huang@intel.com, hannes@cmpxchg.org, mhocko@suse.com, mgorman@techsingularity.net, kirill.shutemov@linux.intel.com, hughd@google.com, akpm@linux-foundation.org Cc: yang.shi@linux.alibaba.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: [PATCH] mm: vmscan: correct nr_reclaimed for THP Date: Fri, 10 May 2019 08:16:32 +0800 Message-Id: <1557447392-61607-1-git-send-email-yang.shi@linux.alibaba.com> X-Mailer: git-send-email 1.8.3.1 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Since commit bd4c82c22c36 ("mm, THP, swap: delay splitting THP after swapped out"), THP can be swapped out in a whole. But, nr_reclaimed still gets inc'ed by one even though a whole THP (512 pages) gets swapped out. This doesn't make too much sense to memory reclaim. For example, direct reclaim may just need reclaim SWAP_CLUSTER_MAX pages, reclaiming one THP could fulfill it. But, if nr_reclaimed is not increased correctly, direct reclaim may just waste time to reclaim more pages, SWAP_CLUSTER_MAX * 512 pages in worst case. This change may result in more reclaimed pages than scanned pages showed by /proc/vmstat since scanning one head page would reclaim 512 base pages. Cc: "Huang, Ying" Cc: Johannes Weiner Cc: Michal Hocko Cc: Mel Gorman Cc: "Kirill A . Shutemov" Cc: Hugh Dickins Signed-off-by: Yang Shi --- I'm not quite sure if it was the intended behavior or just omission. I tried to dig into the review history, but didn't find any clue. I may miss some discussion. mm/vmscan.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index fd9de50..7e026ec 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1446,7 +1446,11 @@ static unsigned long shrink_page_list(struct list_head *page_list, unlock_page(page); free_it: - nr_reclaimed++; + /* + * THP may get swapped out in a whole, need account + * all base pages. + */ + nr_reclaimed += (1 << compound_order(page)); /* * Is there need to periodically free_page_list? It would -- 1.8.3.1