From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from co9outboundpool.messaging.microsoft.com (co9ehsobe004.messaging.microsoft.com [207.46.163.27]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (Client CN "mail.global.frontbridge.com", Issuer "MSIT Machine Auth CA 2" (not verified)) by ozlabs.org (Postfix) with ESMTPS id 778D22C008F for ; Tue, 25 Jun 2013 10:51:08 +1000 (EST) Date: Mon, 24 Jun 2013 19:51:00 -0500 From: Scott Wood To: Joakim Tjernlund Subject: Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL Message-ID: <20130625005100.GA22976@home.buserror.net> References: <13517672561.20120531113057@abatron.ch> <4FC7AEC9.5050203@freescale.com> <4FC7E606.1070205@freescale.com> <4FC7EDD5.8090302@freescale.com> <3E027F8168735B46AC006B1D0C7BB002079C3290@039-SN2MPN1-013.039d.mgd.msft.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: Cc: Wood Scott-B07421 , Zang Roy-R61911 , Bob Cochran , "linuxppc-dev@ozlabs.org" , Dan Malek , Support List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Jul 20, 2012 at 10:37:17AM +0200, Joakim Tjernlund wrote: > Zang Roy-R61911 wrote on 2012/07/20 10:27:52: > > > > > > > > > -----Original Message----- > > > From: linuxppc-dev-bounces+tie-fei.zang=freescale.com@lists.ozlabs.org > > > [mailto:linuxppc-dev-bounces+tie-fei.zang=freescale.com@lists.ozlabs.org] > > > On Behalf Of Joakim Tjernlund > > > Sent: Friday, June 01, 2012 6:36 AM > > > To: Wood Scott-B07421 > > > Cc: linuxppc-dev@ozlabs.org; Dan Malek; Bob Cochran; Support > > > Subject: Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL > > > > > > It just occurred to me that you guys have this already in your Linux SDK so > > > it can't be that bad. > > No. MSR_DE is ONLY added when using CW debug in SDK. > > Roy > > > > Yes, and I later found that user space debugging is busted if you turn on MSR_DE in > kernel. So, how should we handle the CONFIG_BDI_SWITCH patch? It seems like it should at least have a warning in the kconfig help text that it breaks userspace debugging (to the point of causing a kernel oops if it's tried). Or maybe it can deselect CONFIG_PPC_ADV_DEBUG_REGS? It'd also be nice to keep things like this, that are a consequence of how external debug works on e500, separate from the Abatron-specific stuff. -Scott