From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.179]) by ozlabs.org (Postfix) with ESMTP id B4AB7DDE08 for ; Thu, 15 Mar 2007 03:30:43 +1100 (EST) From: Arnd Bergmann To: linuxppc-dev@ozlabs.org Subject: Re: [patch] PS3: Fix hardware watchpoints Date: Wed, 14 Mar 2007 17:30:14 +0100 References: <45F74D69.5070600@am.sony.com> <17911.24243.64287.434769@cargo.ozlabs.ibm.com> In-Reply-To: <17911.24243.64287.434769@cargo.ozlabs.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200703141730.15344.arnd@arndb.de> Cc: Paul Mackerras List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wednesday 14 March 2007, Paul Mackerras wrote: > Geoff Levand writes: > > > Add PS3 support for the PowerPC processor's Data Address Breakpoint > > Register (DABR). > > > > Signed-off-by: Arnd Bergmann > > Signed-off-by: Geoff Levand > > > > --- > > Paul, please consider for 2.6.21. > > It's nice and small, but it isn't really a bugfix as far as I can > see... What's the downside of it not going into 2.6.21, apart from > some inconvenience in debugging things? It makes it possible for a regular user to hang the partition, because the kernel falls back to access the actual SPR, which is forbidden by the hypervisor. My original changelog text was: The lv1 hypervisor silently kills the partition if the kernel attempts to access the dabr register directly. Instead, it needs to be set using an hcall. Since both pseries and celleb have the same function, the guess is that the implementation on ps3 is also the same. How about making the actual changelog text something like this: ---- The lv1 hypervisor silently kills the partition if the kernel attempts to access the dabr register directly, which can happen when gdb adds a hardware breakpoint. Instead, the kernel needs to be set using an hcall, the same way as it does for pseries and celleb. ---- The real question however should be if there is a point in merging any ps3 patches into 2.6.21 at this point. Geoff, my understanding is that the current ps3 platform code in 2.6.21-rc still relies on additional patches from you in order to even boot, right? Maybe it's better to just queue them for 2.6.22 now, independent of the severity of bugs fixed by a given patch. Arnd <><