From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [PATCH 06/20] early_res: seperate common memmap func from e820.c to fw_memmap.cy Date: Tue, 23 Mar 2010 10:48:34 +0100 Message-ID: <20100323094834.GB4576@elte.hu> References: <4BA7EE72.7000507@kernel.org> <4BA80019.5000900@kernel.org> <4BA813A4.1010704@kernel.org> <4BA8594F.8090205@kernel.org> <20100323080252.GA4848@elte.hu> <4BA883C0.2070102@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mx3.mail.elte.hu ([157.181.1.138]:56973 "EHLO mx3.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751346Ab0CWJsw (ORCPT ); Tue, 23 Mar 2010 05:48:52 -0400 Content-Disposition: inline In-Reply-To: <4BA883C0.2070102@kernel.org> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Yinghai Lu Cc: Thomas Gleixner , David Miller , Andrew Morton , Linus Torvalds , benh@kernel.crashing.org, hpa@zytor.com, jbarnes@virtuousgeek.org, ebiederm@xmission.com, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org * Yinghai Lu wrote: > > 2) > > > > I think we also need to concentrate the changes back into LMB: > > yes. put them in kernel/early_res.c and move them to lmb.c if lmb gugs are > happy with the change. Yes, they seemed OK with changing it to accomodate x86, as long as current behavior stays compatible and as long as the changes are squeaky-clean. Both of which are highly reasonable expectations ;-) > > early_res.h will go away as well and all the new APIs will be in lmb.h. > > current have three levels > a. old lmb users > b. x86 with bootmem > c. x86 with no-bootmem > > some functions later could be moved to new bootmem.c I think we want to work towards the end result where we dont have bootmem.c anymore. I.e. a modern LMB architecture should generally not make use of bootmem at all. We could do that switch on x86 straight away, and make CONFIG_NO_BOOTMEM a default-y option, hm? We could also hide the interactivity behind CONFIG_DEBUG_VM or so - and eliminate it altogether later on. We should also switch around the flag and turn it into CONFIG_BOOTMEM. Hm? Ingo