Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754412Ab3JJBVw (ORCPT ); Wed, 9 Oct 2013 21:21:52 -0400 Received: from cn.fujitsu.com ([222.73.24.84]:31090 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751598Ab3JJBVt (ORCPT ); Wed, 9 Oct 2013 21:21:49 -0400 X-IronPort-AV: E=Sophos;i="4.93,462,1378828800"; d="scan'208";a="8706979" Message-ID: <525600E0.8020001@cn.fujitsu.com> Date: Thu, 10 Oct 2013 09:20:32 +0800 From: Zhang Yanfei User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 MIME-Version: 1.0 To: "H. Peter Anvin" , Tejun Heo CC: Zhang Yanfei , Toshi Kani , Andrew Morton , "Rafael J . Wysocki" , lenb@kernel.org, Thomas Gleixner , mingo@elte.hu, Wanpeng Li , Thomas Renninger , Yinghai Lu , Jiang Liu , Wen Congyang , Lai Jiangshan , isimatu.yasuaki@jp.fujitsu.com, izumi.taku@jp.fujitsu.com, Mel Gorman , Minchan Kim , mina86@mina86.com, gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com, lwoodman@redhat.com, Rik van Riel , jweiner@redhat.com, prarit@redhat.com, "x86@kernel.org" , linux-doc@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux MM , linux-acpi@vger.kernel.org, imtangchen@gmail.com, Tang Chen Subject: Re: [PATCH part1 v6 4/6] x86/mem-hotplug: Support initialize page tables in bottom-up References: <524E2032.4020106@gmail.com> <524E2127.4090904@gmail.com> <5251F9AB.6000203@zytor.com> <525442A4.9060709@gmail.com> <20131009164449.GG22495@htj.dyndns.org> <52558EEF.4050009@gmail.com> <20131009192040.GA5592@mtj.dyndns.org> <1381352311.5429.115.camel@misato.fc.hp.com> <20131009211136.GH5592@mtj.dyndns.org> <5255C730.90602@zytor.com> <5255CE7D.8030007@gmail.com> <5255E253.3080905@zytor.com> <5255E60F.5010102@gmail.com> In-Reply-To: <5255E60F.5010102@gmail.com> X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/10/10 09:19:31, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/10/10 09:19:38, Serialize complete at 2013/10/10 09:19:38 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1679 Lines: 46 Hello guys, On 10/10/2013 07:26 AM, Zhang Yanfei wrote: > Hello Peter, > > On 10/10/2013 07:10 AM, H. Peter Anvin wrote: >> On 10/09/2013 02:45 PM, Zhang Yanfei wrote: >>>> >>>> I would also argue that in the VM scenario -- and arguable even in the >>>> hardware scenario -- the right thing is to not expose the flexible >>>> memory in the e820/EFI tables, and instead have it hotadded (possibly >>>> *immediately* so) on boot. This avoids both the boot time funnies as >>>> well as the scaling issues with metadata. >>>> >>> >>> So in this kind of scenario, hotpluggable memory will not be detected >>> at boot time, and admin should not use this movable_node boot option >>> and the kernel will act as before, using top-down allocation always. >>> >> >> Yes. The idea is that the kernel will boot up without the hotplug >> memory, but if desired, will immediately see a hotplug-add event for the >> movable memory. > > Yeah, this is good. > > But in the scenario that boot with hotplug memory, we need the movable_node > option. So as tejun has explained a lot about this patchset, do you still > have objection to it or could I ask andrew to merge it into -mm tree for > more tests? > Since tejun has explained a lot about this approach, could we come to an agreement on this one? Peter? If you have no objection, I'll post a new v7 version which will fix the __pa_symbol problem pointed by you. -- Thanks. Zhang Yanfei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/