From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751447Ab0KSHPW (ORCPT ); Fri, 19 Nov 2010 02:15:22 -0500 Received: from mga09.intel.com ([134.134.136.24]:61156 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751198Ab0KSHPV (ORCPT ); Fri, 19 Nov 2010 02:15:21 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.59,221,1288594800"; d="scan'208";a="575422853" Date: Fri, 19 Nov 2010 13:54:14 +0800 From: Shaohui Zheng To: Paul Mundt Cc: akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, haicheng.li@linux.intel.com, ak@linux.intel.com, shaohui.zheng@linux.intel.com Subject: Re: [0/8,v3] NUMA Hotplug Emulator - Introduction & Feedbacks Message-ID: <20101119055414.GC3327@shaohui> References: <20101117020759.016741414@intel.com> <20101117052213.GA10671@linux-sh.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101117052213.GA10671@linux-sh.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 17, 2010 at 02:22:13PM +0900, Paul Mundt wrote: > On Wed, Nov 17, 2010 at 10:07:59AM +0800, shaohui.zheng@intel.com wrote: > > * PATCHSET INTRODUCTION > > > > patch 1: Add function to hide memory region via e820 table. Then emulator will > > use these memory regions to fake offlined numa nodes. > > patch 2: Infrastructure of NUMA hotplug emulation, introduce "hide node". > > patch 3: Provide an userland interface to hotplug-add fake offlined nodes. > > patch 4: Abstract cpu register functions, make these interface friend for cpu > > hotplug emulation > > patch 5: Support cpu probe/release in x86, it provide a software method to hot > > add/remove cpu with sysfs interface. > > patch 6: Fake CPU socket with logical CPU on x86, to prevent the scheduling > > domain to build the incorrect hierarchy. > > patch 7: extend memory probe interface to support NUMA, we can add the memory to > > a specified node with the interface. > > patch 8: Documentations > > > > * FEEDBACKS & RESPONSES > > > I had some comments on the other patches in the series that possibly got > missed because of the mail-followup-to confusion: > > http://lkml.org/lkml/2010/11/15/11 About memblock API, it is a good APIs list to manage memory region. If all the e820 wrapper function use memblock API, the code should be very clean. currently, no body use memblock in e820 wrapper, so we should still keep this status, unless we decide rewrite these e820 wrapper. Anyway, we already select other way to hide memory, we will not add wrapper on e820 table anymore. > http://lkml.org/lkml/2010/11/15/14 I understand, the MACROs are not functions, it will not comsume memory after compile it. the IFDEF should be removed > http://lkml.org/lkml/2010/11/15/15 I think that you want to say ARCH_ENABLE_NUMA_HOTPLUG_EMU here, not ARCH_ENABLE_NUMA_EMU. the option NUMA_HOTPLUG_EMU is a dummy item, it does not control any codes, it just try to maintain the node/memory/cpu hotplug emulation option together, it provides convenience when use want to enable them. > > The other one you've already dealt with. -- Thanks & Regards, Shaohui