From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zachary Amsden Subject: Re: [PATCH 2/7] paravirtualization: Patch inline replacements for common paravirt operations. Date: Wed, 01 Nov 2006 16:54:00 -0800 Message-ID: <454941A8.6000601@vmware.com> References: <20061029024504.760769000@sous-sol.org> <20061029024607.401333000@sous-sol.org> <200610290831.21062.ak@suse.de> <1162178936.9802.34.camel@localhost.localdomain> <20061030231132.GA98768@muc.de> <1162376827.23462.5.camel@localhost.localdomain> <1162376894.23462.7.camel@localhost.localdomain> <20061101152715.f1f94d5c.akpm@osdl.org> <1162428424.6848.14.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1162428424.6848.14.camel@localhost.localdomain> Sender: linux-kernel-owner@vger.kernel.org To: Rusty Russell Cc: Andrew Morton , Andi Kleen , Andi Kleen , virtualization@lists.osdl.org, Chris Wright , linux-kernel@vger.kernel.org List-Id: virtualization@lists.linuxfoundation.org Rusty Russell wrote: > On Wed, 2006-11-01 at 15:27 -0800, Andrew Morton wrote: > >> On Wed, 01 Nov 2006 21:28:13 +1100 >> Rusty Russell wrote: >> >>> +#ifdef CONFIG_DEBUG_KERNEL >>> + /* Deliberately clobber regs using "not %reg" to find bugs. */ >>> >> That would be considered to be abusive of CONFIG_DEBUG_KERNEL. A >> CONFIG_DEBUG_PARAVIRT which depends on CONFIG_DEBUG_KERNEL would be more >> harmonious. >> > > I wasn't sure. Making a config option for what is a one-liner seemed > overkill. > I have further stuff in my vmi-debug patch that can use CONFIG_DEBUG_PARAVIRT as well :) Zach