Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932271AbdDDQo7 (ORCPT ); Tue, 4 Apr 2017 12:44:59 -0400 Received: from mx2.suse.de ([195.135.220.15]:53857 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932072AbdDDQo6 (ORCPT ); Tue, 4 Apr 2017 12:44:58 -0400 Date: Tue, 4 Apr 2017 18:44:53 +0200 From: Michal Hocko To: Reza Arbab Cc: Mel Gorman , linux-mm@kvack.org, Andrew Morton , Vlastimil Babka , Andrea Arcangeli , Yasuaki Ishimatsu , Tang Chen , qiuxishi@huawei.com, Kani Toshimitsu , slaoub@gmail.com, Joonsoo Kim , Andi Kleen , Zhang Zhen , David Rientjes , Daniel Kiper , Igor Mammedov , Vitaly Kuznetsov , LKML , Chris Metcalf , Dan Williams , Heiko Carstens , Lai Jiangshan , Martin Schwidefsky Subject: Re: [PATCH 0/6] mm: make movable onlining suck less Message-ID: <20170404164452.GQ15132@dhcp22.suse.cz> References: <20170330115454.32154-1-mhocko@kernel.org> <20170403115545.GK24661@dhcp22.suse.cz> <20170403195830.64libncet5l6vuvb@arbab-laptop> <20170403202337.GA12482@dhcp22.suse.cz> <20170403204213.rs7k2cvsnconel2z@arbab-laptop> <20170404072329.GA15132@dhcp22.suse.cz> <20170404073412.GC15132@dhcp22.suse.cz> <20170404082302.GE15132@dhcp22.suse.cz> <20170404160239.ftvuxklioo6zvuxl@arbab-laptop> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170404160239.ftvuxklioo6zvuxl@arbab-laptop> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1140 Lines: 38 On Tue 04-04-17 11:02:39, Reza Arbab wrote: > On Tue, Apr 04, 2017 at 10:23:02AM +0200, Michal Hocko wrote: > >diff --git a/drivers/base/node.c b/drivers/base/node.c > >index 5548f9686016..ee080a35e869 100644 > >--- a/drivers/base/node.c > >+++ b/drivers/base/node.c > >@@ -368,8 +368,6 @@ int unregister_cpu_under_node(unsigned int cpu, unsigned int nid) > >} > > > >#ifdef CONFIG_MEMORY_HOTPLUG_SPARSE > >-#define page_initialized(page) (page->lru.next) > >- > >static int __ref get_nid_for_pfn(unsigned long pfn) > >{ > > struct page *page; > >@@ -380,9 +378,6 @@ static int __ref get_nid_for_pfn(unsigned long pfn) > > if (system_state == SYSTEM_BOOTING) > > return early_pfn_to_nid(pfn); > >#endif > >- page = pfn_to_page(pfn); > >- if (!page_initialized(page)) > >- return -1; > > return pfn_to_nid(pfn); > >} > > > > You can get rid of 'page' altogether. > > drivers/base/node.c: In function ‘get_nid_for_pfn’: > drivers/base/node.c:373:15: warning: unused variable ‘page’ [-Wunused-variable] Right, updated. Thanks for your testing! This is highly appreciated. Can I assume your Tested-by? -- Michal Hocko SUSE Labs