From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH 0/8] 2.6.17 merge additions Date: Fri, 02 Mar 2007 11:17:05 +0000 Message-ID: References: <45E80393.76E4.0078.0@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <45E80393.76E4.0078.0@novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Jan Beulich , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On 2/3/07 09:59, "Jan Beulich" wrote: > Isn't that patch just doing this - paging_init() is almost identical to > native > after the patch (the sole difference is the setting of > init_mm.context.pinned). > The only real addition (to find_early_table_space()) is the space reservation > for the fixmap tables (so these can be set up earlier) and the stuff moved > out of paging_init() into init_memory_mapping(). And I don't think it is > reasonable to expect init_memory_mapping() to get very close to native. Okay I probably should have done a three-way comparison with the native code as well, then. My first impression was definitely that the code looked more complicated rather than less. :-) I'll take another look. -- Keir