From: Jeremy Fitzhardinge Subject: Re: [RFC PATCH 09/33] Add start-of-day setup hooks to subarch Date: Tue, 18 Jul 2006 13:49:26 -0700 Message-ID: <44BD4956.5020706@goop.org> References: <20060718091807.467468000@sous-sol.org> <20060718091950.750213000@sous-sol.org> <1153217033.3038.25.camel@laptopd505.fenrus.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1153217033.3038.25.camel@laptopd505.fenrus.org> Sender: linux-kernel-owner@vger.kernel.org To: Arjan van de Ven Cc: Chris Wright , linux-kernel@vger.kernel.org, virtualization@lists.osdl.org, xen-devel@lists.xensource.com, Andi Kleen , Andrew Morton , Rusty Russell , Zachary Amsden , Ian Pratt , Christian Limpach , Jeremy Fitzhardinge List-Id: xen-devel@lists.xenproject.org Arjan van de Ven wrote: > what is this for? Isn't this 1) undocumented and 2) unclear and 3) > ugly ? (I'm pretty sure the HAVE_ARCH_* stuff is highly deprecated for > new things nowadays) > It appears to be completely unnecessary. Xen doesn't use sanitize_e820_map(), but there's no obvious harm in leaving it there (and it's init code, so there's no long-term cost). J