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=-0.7 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 57EA3C432C0 for ; Fri, 22 Nov 2019 17:36:36 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 27DCD2071C for ; Fri, 22 Nov 2019 17:36:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 27DCD2071C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.page Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id B43526B051E; Fri, 22 Nov 2019 12:36:35 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id AF3F16B051F; Fri, 22 Nov 2019 12:36:35 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A08D76B0520; Fri, 22 Nov 2019 12:36:35 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0003.hostedemail.com [216.40.44.3]) by kanga.kvack.org (Postfix) with ESMTP id 8C0A46B051E for ; Fri, 22 Nov 2019 12:36:35 -0500 (EST) Received: from smtpin08.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with SMTP id 475B48249980 for ; Fri, 22 Nov 2019 17:36:35 +0000 (UTC) X-FDA: 76184617950.08.mom61_1c5aa83cc9927 X-HE-Tag: mom61_1c5aa83cc9927 X-Filterd-Recvd-Size: 2608 Received: from relay8-d.mail.gandi.net (relay8-d.mail.gandi.net [217.70.183.201]) by imf29.hostedemail.com (Postfix) with ESMTP for ; Fri, 22 Nov 2019 17:36:34 +0000 (UTC) X-Originating-IP: 153.3.140.100 Received: from localhost (unknown [153.3.140.100]) (Authenticated sender: fly@kernel.page) by relay8-d.mail.gandi.net (Postfix) with ESMTPSA id 93CF01BF207; Fri, 22 Nov 2019 17:36:26 +0000 (UTC) Date: Sat, 23 Nov 2019 01:36:13 +0800 From: Pengfei Li To: Christopher Lameter Cc: David Hildenbrand , akpm@linux-foundation.org, mgorman@techsingularity.net, mhocko@kernel.org, vbabka@suse.cz, iamjoonsoo.kim@lge.com, guro@fb.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, fly@kernel.page Subject: Re: [RFC v1 00/19] Modify zonelist to nodelist v1 Message-ID: <20191123013613.566bb40a.fly@kernel.page> In-Reply-To: References: <20191121151811.49742-1-fly@kernel.page> <1bb37491-72a7-feaa-722d-a5825813a409@redhat.com> <20191122234907.4da3bc81.fly@kernel.page> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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 Fri, 22 Nov 2019 15:53:57 +0000 (UTC) Christopher Lameter wrote: > On Fri, 22 Nov 2019, Pengfei Li wrote: > > > I am sorry that I did not make it clear. I want to express this > > series of patches will benefit NUMA systems with multiple nodes. > > Ok but that benefit needs to be quantified somehow. > Thanks for your comments. Yes, I will add detailed performance test data in v2. > > The main benefit is that it will be more efficient when traversing > > all nodes (for example when performing page reclamation). > > And you loose the prioritization of allocations through these > different zones. Sorry, I forgot to mention that the information about the zones that are available to the node is still there. The old for_each_zone_zonelist has been replaced with for_each_zone_nodelist. I will add some key implementation details in v2. > We create zonelists with a certain sequence of the > zones in order to prefer allocations from certain zones. This is in > particular important for the smaller DMA zones which may be easily > exhausted. > I'm not sure if I understand what you mean, but since commit c9bff3eebc09 ("mm, page_alloc: rip out ZONELIST_ORDER_ZONE"), the zonelist is always in "Node" order, so building the nodelist is fine. -- Pengfei