From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f200.google.com (mail-wr0-f200.google.com [209.85.128.200]) by kanga.kvack.org (Postfix) with ESMTP id C2BD56B0279 for ; Mon, 17 Jul 2017 04:19:46 -0400 (EDT) Received: by mail-wr0-f200.google.com with SMTP id u89so5707936wrc.1 for ; Mon, 17 Jul 2017 01:19:46 -0700 (PDT) Received: from mx1.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id h71si9319388wmd.111.2017.07.17.01.19.45 for (version=TLS1 cipher=AES128-SHA bits=128/128); Mon, 17 Jul 2017 01:19:45 -0700 (PDT) Date: Mon, 17 Jul 2017 10:19:42 +0200 From: Michal Hocko Subject: Re: [PATCH 6/9] mm, page_alloc: simplify zonelist initialization Message-ID: <20170717081942.GA12888@dhcp22.suse.cz> References: <20170714080006.7250-1-mhocko@kernel.org> <20170714080006.7250-7-mhocko@kernel.org> <20170714124645.i3duhuie6cczlybr@suse.de> <20170714130242.GQ2618@dhcp22.suse.cz> <20170714141823.2j7t37t6zdzdf3sv@suse.de> <20170717060639.GA7397@dhcp22.suse.cz> <20170717080723.wctyyukherj7bkqt@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170717080723.wctyyukherj7bkqt@suse.de> Sender: owner-linux-mm@kvack.org List-ID: To: Mel Gorman Cc: linux-mm@kvack.org, Andrew Morton , Johannes Weiner , Vlastimil Babka , LKML On Mon 17-07-17 09:07:23, Mel Gorman wrote: > On Mon, Jul 17, 2017 at 08:06:40AM +0200, Michal Hocko wrote: > > On Fri 14-07-17 15:18:23, Mel Gorman wrote: > > > Fairly sure that's not what you meant. > > > > > > > > > > pg_data_t *node = NODE_DATA(node_order[i]); > > > > > > > > - zoneref_idx = build_zonelists_node(node, zonelist, zoneref_idx); > > > > + nr_zones = build_zonelists_node(node, zonelist, nr_zones); > > > > > > I meant converting build_zonelists_node and passing in &nr_zones and > > > returning false when an empty node is encountered. In this context, > > > it's also not about zones, it really is nr_zonerefs. Rename nr_zones in > > > build_zonelists_node as well. > > > > hmm, why don't we rather make it zonerefs based then. Something > > like the following? > > Works for me. Should I fold it to the patch or make it a patch on its own? -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org