From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: [PATCH 04/25] xen: Add XEN config options Date: Mon, 23 Apr 2007 16:11:45 -0700 Message-ID: <462D3D31.1090702@goop.org> References: <20070423215638.563901986@goop.org> <20070423215709.990824532@goop.org> <200704240100.31011.ak@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <200704240100.31011.ak@suse.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Andi Kleen Cc: Ian Pratt , lkml , Chris Wright , virtualization@lists.osdl.org, Andrew Morton List-Id: virtualization@lists.linuxfoundation.org Andi Kleen wrote: > On Monday 23 April 2007 23:56:42 Jeremy Fitzhardinge wrote: > >> The XEN config option enables the Xen paravirt_ops interface, which is >> installed when the kernel finds itself running under Xen. >> >> Xen is no longer a sub-architecture, so the X86_XEN subarch config >> option has gone. >> >> Xen is currently incompatible with PREEMPT, but this is fixed up later >> in the series. >> > > Shouldn't this be after the change that adds arch/i386/xen/Kconfig? > > Otherwise you break bisects > It should be OK. The series should build and run at each patch (though I have to admit I haven't tested this). In general I've been adding config options for each feature as the feature itself is added. J