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 07424CD37B0 for ; Mon, 18 Sep 2023 12:57:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 8A1F18D0018; Mon, 18 Sep 2023 08:57:12 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 82AC48D0002; Mon, 18 Sep 2023 08:57:12 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6CAB98D0018; Mon, 18 Sep 2023 08:57:12 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 5A7148D0002 for ; Mon, 18 Sep 2023 08:57:12 -0400 (EDT) Received: from smtpin16.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 2FD85401A2 for ; Mon, 18 Sep 2023 12:57:12 +0000 (UTC) X-FDA: 81249718704.16.8B89EE7 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf02.hostedemail.com (Postfix) with ESMTP id 675A08001A for ; Mon, 18 Sep 2023 12:57:10 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=sI1T5llG; dmarc=none; spf=none (imf02.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1695041830; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=9XDsDeK220CTPqxHjZSibpxURfNBC/aG4oAlkDI8JTU=; b=ew9Vb1znU0cOG378b+bv8ICkwwTwCZXPoRCT2EjegOVnsX5UKiyjHlBeYgeaaF7pXWpr24 F/BAhC5YeIUtMR1X18eMHop1vDIQyRipXZ6NCOZeflaIJQZHCYierJNtV5cGQxamdIwANh k7skcg2J6jirJ8yoWvgIipbMJtmQIi8= ARC-Authentication-Results: i=1; imf02.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=sI1T5llG; dmarc=none; spf=none (imf02.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1695041830; a=rsa-sha256; cv=none; b=7eLMxghLIYslFFPYsMatiPHTN31gWxCv4eJW1Y0OeO9xIJNRSdGnbGCIA+dUDgkZDz9tKb RfmGEn/0/4eCy6j6Ly+o4y4JrzKmRArG2epzaKPh3uAcK6Key2yDNrHFCaKCxdUGSTx9y5 IRdtfXi/iHVxOZKa6WrOOjXThahdRFc= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=9XDsDeK220CTPqxHjZSibpxURfNBC/aG4oAlkDI8JTU=; b=sI1T5llGcQNIAw6gwKHoFy/lHm 8Rn3JWtMNlyuhJpaTt2HjwukPbrpQkBCILHYg/MYefRkJEO5BVSBYjaxKloU3JTQyeYFSRNz15fBx g7AgdV1uaDbIOJPfgiBplpsa5Pi++EoJAQ0zNp2d3HaCIF+h315VMbktFcWIUiR7XGzqxvio1+KMb h1dQRoUE2Q5UVpAj34Vy2avSSbQ1lksc7d2G0p7fPHsFNszG5TulMIS9kupTIuOvAyg3+elr3+GLh VVCA/s1Dvb0q9jdh7Ce4RiVBxMOElqqUpZREjyeoePSR9xjh0q0b6hPNGHilxEJvcCmGupuyl0HO+ +7+d0X0g==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1qiDoG-00B7La-2R; Mon, 18 Sep 2023 12:57:04 +0000 Date: Mon, 18 Sep 2023 13:57:04 +0100 From: Matthew Wilcox To: Kefeng Wang Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, ying.huang@intel.com, david@redhat.com, Zi Yan , Mike Kravetz , hughd@google.com Subject: Re: [PATCH 0/6] mm: convert numa balancing functions to use a folio Message-ID: References: <20230918103213.4166210-1-wangkefeng.wang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230918103213.4166210-1-wangkefeng.wang@huawei.com> X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 675A08001A X-Stat-Signature: edsc4m5ps9gann5eb1w4qy7kgcsfm6s6 X-HE-Tag: 1695041830-929651 X-HE-Meta: U2FsdGVkX186e9m1VX8Mc2O1821sbe7WMqpqovHS6kFjL5dg/zteDqVX040As75qrqwcUL01BJw+ikzrkKuz9rOp3RF0z8GlYnAHkWbnvFur0SMapy3ACiyDn5tHFzx5zUJznvg2aXMAvjLjUEvmx7br2A493/d+RBFmLZFzWPxtq/4DOfrmd8/BSOMWblXvw4OcVhoECuUUPxyeO5ZqV6p1i4ouEJtoWqnyhTspQyTOw153ttVLUVcEIvx9JgPPISbR92zqJVcdb21Uw1wi12gEx89SHQEiu4Uyngjd7han5qHzuzSX3iH0Ce3jU0FLKgWEuopIpnRd/2GhcTHQ2yhgrW2ryjRvw0DGJF5cY/oYoDgno+7C/kSoRR6Nb0o/JsgSMV32W0boKq0HdTTw9WyHm19J1hFU7rwh+a0pUlY3KJ5vfE4MidhSYTNURmw0xsqvPLKrzmKHjWWue8+DAQtEEpmA3J4MNwLOn382jkCI22gVsPD5rDk3h1pMKOUCgCTLHbkw2XBqgJDvvLfqaRQoRqciXTYyMKeq3EECOAXVxVZM4BROA+MMhgNtqL/L9WMnVC1X/zNNg4IKHImNolnG3Vy9SRWffIO/PvgBXQfAtcNWLUGm6KOcXqKVq/b3rCJbqJ2ihpkA+OaIPqRy+SEdPAaMuXgc5iucpzhCD3eW/M4v82t7kwOIizHfzcAYCkJP7+4ZhABWJqapA+oY5DXGUW5QGJ9kJGXMto4k1udKygWmJjz7ffKC99fDuEC+k8Z1vmgZrhvASRjstSXyc8oO1yjtzLGjy8CPahSkX6qMl/w3fzTIdGJJ/gN3gb9ATIvtVzQScU0ePaXp/bB1dkyPzEavH1jXpwWYyOLvgobtLXBpx6lWy5h1t2ibdok/3qt8f+bZv76XAGTxJ0rHEbRcjTFCBRHxm2h9bZrCnZoLN9PW+BNLaBSUYVTWwgm1d3iSKLAfDx+ta4qJpS9 eDO/QNPa csuQxKsMZafZ2pQUPHSUXjljXvn6gg4mF6TDsPEaHfaeMNiWV00+Ccl4Y7jNHM5Rp9vpDHq3nw5EHpHrY3wJ9Nj6TJEIozkVe9zC58uMRgHVB7jbzSs7kL2DsH57N+ciBMc4jg68Oif5B0Jg= 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 Mon, Sep 18, 2023 at 06:32:07PM +0800, Kefeng Wang wrote: > The do_numa_pages only handle non-compound page, and only PMD-mapped THP > is handled in do_huge_pmd_numa_page(), but large, PTE-mapped folio will > be supported, let's convert more numa balancing functions to use/take a > folio in preparation for that, no functional change intended for now. > > Kefeng Wang (6): > sched/numa, mm: make numa migrate functions to take a folio > mm: mempolicy: make mpol_misplaced() to take a folio > mm: memory: make numa_migrate_prep() to take a folio > mm: memory: use a folio in do_numa_page() > mm: memory: add vm_normal_pmd_folio() > mm: huge_memory: use a folio in do_huge_pmd_numa_page() This all seems OK. It's kind of hard to review though because you change the same line multiple times. I think it works out better to go top-down instead of bottom-up. That is, start with do_numa_page() and pass &folio->page to numa_migrate_prep. Then do vm_normal_pmd_folio() followed by do_huge_pmd_numa_page(). Fourth would have been numa_migrate_prep(), etc. I don't want to ask you to redo the entire series, but for future patch series. Also, it's nce to do things like remove the unnecessary 'extern' from function declarations when you change them from page to folio. And please try to stick to 80 columns; I know it's not always easy/possible.