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=-15.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 7609DC4707F for ; Wed, 26 May 2021 00:43:31 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 094BC61417 for ; Wed, 26 May 2021 00:43:30 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 094BC61417 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=sangfor.com.cn Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 5F6E46B0036; Tue, 25 May 2021 20:43:30 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 5CCF06B006E; Tue, 25 May 2021 20:43:30 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 46E206B0070; Tue, 25 May 2021 20:43:30 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0115.hostedemail.com [216.40.44.115]) by kanga.kvack.org (Postfix) with ESMTP id 11AEB6B0036 for ; Tue, 25 May 2021 20:43:30 -0400 (EDT) Received: from smtpin26.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id A21F68249980 for ; Wed, 26 May 2021 00:43:29 +0000 (UTC) X-FDA: 78181533738.26.46B53ED Received: from mail-m121142.qiye.163.com (mail-m121142.qiye.163.com [115.236.121.142]) by imf25.hostedemail.com (Postfix) with SMTP id 3E30E600025B for ; Wed, 26 May 2021 00:43:20 +0000 (UTC) Received: from [0.0.0.0] (unknown [14.154.30.253]) by mail-m121142.qiye.163.com (Hmail) with ESMTPA id E88AF80134; Wed, 26 May 2021 08:43:22 +0800 (CST) Subject: Re: [PATCH v2] mm/page_alloc: fix counting of free pages after take off from buddy To: =?UTF-8?B?SE9SSUdVQ0hJIE5BT1lBKOWggOWPoyDnm7TkuZ8p?= Cc: "david@redhat.com" , "osalvador@suse.de" , "akpm@linux-foundation.org" , "linux-kernel@vger.kernel.org" , "linux-mm@kvack.org" References: <8cd355d2-1adc-4189-7b65-cfea13129db5@sangfor.com.cn> <20210508035533.23222-1-dinghui@sangfor.com.cn> <20210525083228.GA3327407@hori.linux.bs1.fc.nec.co.jp> From: Ding Hui Message-ID: <1eabe244-4568-e1e1-7f9e-235175cc8c1d@sangfor.com.cn> Date: Wed, 26 May 2021 08:43:28 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.10.0 MIME-Version: 1.0 In-Reply-To: <20210525083228.GA3327407@hori.linux.bs1.fc.nec.co.jp> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US X-HM-Spam-Status: e1kfGhgUHx5ZQUtXWQgYFAkeWUFZS1VLWVdZKFlBSE83V1ktWUFJV1kPCR oVCBIfWUFZQx1KHlZMTkxKQxhKH09KH05VEwETFhoSFyQUDg9ZV1kWGg8SFR0UWUFZT0tIVUpKS0 9ISVVLWQY+ X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6MD46Kio6ED8TFRpPISFKATIy FgkaChlVSlVKTUlKQkNCQ0tITktMVTMWGhIXVR8SFRwTDhI7CBoVHB0UCVUYFBZVGBVFWVdZEgtZ QVlKT1VKTk9VSEtVSU5IWVdZCAFZQUhNS0s3Bg++ X-HM-Tid: 0a79a62044d2b037kuuue88af80134 Content-Transfer-Encoding: quoted-printable Authentication-Results: imf25.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=sangfor.com.cn; spf=pass (imf25.hostedemail.com: domain of dinghui@sangfor.com.cn designates 115.236.121.142 as permitted sender) smtp.mailfrom=dinghui@sangfor.com.cn X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 3E30E600025B X-Stat-Signature: imzy9iqtb1s9pr6uju7jcoj4prr4j8bj X-HE-Tag: 1621989800-148036 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 2021/5/25 16:32, HORIGUCHI NAOYA(=E5=A0=80=E5=8F=A3 =E7=9B=B4=E4=B9=9F= ) wrote: > On Sat, May 08, 2021 at 11:55:33AM +0800, Ding Hui wrote: >> Recently we found there is a lot MemFree left in /proc/meminfo after >> do a lot of pages soft offline. >> >> I think it's incorrect since NR_FREE_PAGES should not contain HWPoison= pages. >> For offline free pages, after a successful call take_page_off_buddy(),= the >> page is no longer belong to buddy allocator, and will not be used any = more, >> but we missed accounting NR_FREE_PAGES in this situation. >> >> Do update like rmqueue() does. >> >> Signed-off-by: Ding Hui >> --- >> V2: >> use __mod_zone_freepage_state instead of __mod_zone_page_state >> >> mm/page_alloc.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/mm/page_alloc.c b/mm/page_alloc.c >> index cfc72873961d..e124a615303b 100644 >> --- a/mm/page_alloc.c >> +++ b/mm/page_alloc.c >> @@ -8947,6 +8947,7 @@ bool take_page_off_buddy(struct page *page) >> del_page_from_free_list(page_head, zone, page_order); >> break_down_buddy_pages(zone, page_head, page, 0, >> page_order, migratetype); >> + __mod_zone_freepage_state(zone, -1, migratetype); >=20 > Page offline code (see set_migratetype_isolate()) seems to handle > NR_FREE_PAGES counter in its own way, so I think that it's more correct= to > call __mod_zone_freepage_state() only when is_migrate_isolate(migratety= pe)) > is false. >=20 > Otherwise, the patch looks good to me. >=20 Thanks for reply and suggestion, I'll send v3 patch later. >> ret =3D true; >> break; >> } >> --=20 >> 2.17.1 --=20 Thanks, - Ding Hui