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 DFBD0C433EF for ; Thu, 23 Jun 2022 12:26:54 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 36EDA8E014C; Thu, 23 Jun 2022 08:26:54 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 31E1D8E0144; Thu, 23 Jun 2022 08:26:54 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 20C338E014C; Thu, 23 Jun 2022 08:26:54 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 13D7D8E0144 for ; Thu, 23 Jun 2022 08:26:54 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id DD933341A7 for ; Thu, 23 Jun 2022 12:26:53 +0000 (UTC) X-FDA: 79609424706.23.1CA4AB3 Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by imf23.hostedemail.com (Postfix) with ESMTP id CFC52140028 for ; Thu, 23 Jun 2022 12:26:52 +0000 (UTC) Received: from canpemm500002.china.huawei.com (unknown [172.30.72.56]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4LTKCw652rzShK9; Thu, 23 Jun 2022 20:23:24 +0800 (CST) Received: from [10.174.177.76] (10.174.177.76) by canpemm500002.china.huawei.com (7.192.104.244) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.24; Thu, 23 Jun 2022 20:26:49 +0800 Subject: Re: [PATCH 14/16] mm/huge_memory: fix comment of page_deferred_list To: Muchun Song CC: , , , , , References: <20220622170627.19786-1-linmiaohe@huawei.com> <20220622170627.19786-15-linmiaohe@huawei.com> From: Miaohe Lin Message-ID: <0b1a1633-2e07-87df-b707-3058353f5609@huawei.com> Date: Thu, 23 Jun 2022 20:26:49 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.177.76] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To canpemm500002.china.huawei.com (7.192.104.244) X-CFilter-Loop: Reflected ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1655987213; a=rsa-sha256; cv=none; b=2zhnRG8Vr405f5ia7NwkiWMfaOsr6PGNkhG3CRSF/NvN//nkl176VByZFqkJYdYxIj4TMb xYkZCZyQoIRCf0xjpUoVO3i8ujmkTuKveBCapyOxAjaZG7QoPBCBN7ALsc2r/qaatVwCII OFMFzv3UpmX2f/5H5Un02IixE0mVFaA= ARC-Authentication-Results: i=1; imf23.hostedemail.com; dkim=none; spf=pass (imf23.hostedemail.com: domain of linmiaohe@huawei.com designates 45.249.212.188 as permitted sender) smtp.mailfrom=linmiaohe@huawei.com; dmarc=pass (policy=quarantine) header.from=huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1655987213; 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; bh=cPWgd2wjvd6GDSpUEm3GcOhNeuxnmuiLE68JzDGg2JE=; b=TT68bgFB3ZZV6soo5HOpOOWBcr5/d31vZ8mPQQfF/hbyIt1GqD45sHkS4pDlKPx0jsjbDO OBxV32s+wfFOXajrqPR/bAxW9qlPx2UkP4BEWJwjzP8CoNOWnKYTdmYH8IP8F6Ut0uqIps yhYsOHNPAzBfJiuKdOlZNVBC3iPstF4= X-Stat-Signature: ayyc51xxyzqfuqwspjbbww4btzpfed91 X-Rspamd-Server: rspam06 Authentication-Results: imf23.hostedemail.com; dkim=none; spf=pass (imf23.hostedemail.com: domain of linmiaohe@huawei.com designates 45.249.212.188 as permitted sender) smtp.mailfrom=linmiaohe@huawei.com; dmarc=pass (policy=quarantine) header.from=huawei.com X-Rspam-User: X-Rspamd-Queue-Id: CFC52140028 X-HE-Tag: 1655987212-534698 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 2022/6/23 15:24, Muchun Song wrote: > On Thu, Jun 23, 2022 at 01:06:25AM +0800, Miaohe Lin wrote: >> The current comment is confusing because if global or memcg deferred list >> in the second tail page is occupied by compound_head, why we still use >> page[2].deferred_list here? I think it wants to say that Global or memcg >> deferred list in the first tail page is occupied by compound_mapcount and >> compound_pincount so we use the second tail page's deferred_list instead. >> >> Signed-off-by: Miaohe Lin >> --- >> include/linux/huge_mm.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h >> index 12b297f9951d..2e8062b3417a 100644 >> --- a/include/linux/huge_mm.h >> +++ b/include/linux/huge_mm.h >> @@ -294,8 +294,8 @@ static inline bool thp_migration_supported(void) >> static inline struct list_head *page_deferred_list(struct page *page) >> { >> /* >> - * Global or memcg deferred list in the second tail pages is >> - * occupied by compound_head. >> + * Global or memcg deferred list in the first tail page is >> + * occupied by compound_mapcount and compound_pincount. >> */ > > The structure of "struct page" seems to have told us the information that > we resue the 2nd tail page to be used as deferred_list. I am not sure the Yes, it does. > value of those comments. Maybe better to remove them? IMHO above comment tries to tell us why deferred list in the second tail page is used instead of first tail page. But it should be fine to remove the above comments as they don't seem to provide much info (thought I'm not really sure). Thanks. > > Thanks. > >> return &page[2].deferred_list; >> } >> -- >> 2.23.0 >> >> > . >