Re: [Patch][hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2012-05-14 Thread K.Prasad
On Mon, May 14, 2012 at 10:53:42AM +1000, Benjamin Herrenschmidt wrote: > On Fri, 2012-05-11 at 14:13 +0530, K.Prasad wrote: > > > +#ifdef CONFIG_HAVE_HW_BREAKPOINT > > + int ret, len = 0; > > + struct thread_struct *thread = &(child->thread); > > +

[Patch][hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2012-05-11 Thread K.Prasad
h for inclusion into the relevant PowerPC tree. Thanks, K.Prasad -- PPC_PTRACE_GETHWDBGINFO, PPC_PTRACE_SETHWDEBUG and PPC_PTRACE_DELHWDEBUG are PowerPC specific ptrace flags that use the watchpoint register. While they are targeted primarily towards BookE users, user-space applications such a

Re: [PATCH 2/2] [PowerPC Book3E] Introduce new ptrace debug feature flag

2011-12-22 Thread K.Prasad
On Wed, Dec 21, 2011 at 11:55:02AM +1100, David Gibson wrote: > On Thu, Dec 08, 2011 at 04:53:30PM +0530, K.Prasad wrote: > > While PPC_PTRACE_SETHWDEBUG ptrace flag in PowerPC accepts > > PPC_BREAKPOINT_MODE_EXACT mode of breakpoint, the same is not intimated to > > the >

[PATCH 2/2] [PowerPC Book3E] Introduce new ptrace debug feature flag

2011-12-08 Thread K.Prasad
tures" member of "struct ppc_debug_info" to advertise support for the same on Book3E PowerPC processors. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/ptrace.h |1 + arch/powerpc/kernel/ptrace.c |1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/

[PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-12-08 Thread K.Prasad
of the variable can be specified). Signed-off-by: K.Prasad --- Documentation/powerpc/ptrace.txt | 16 arch/powerpc/kernel/ptrace.c | 77 +++--- 2 files changed, 87 insertions(+), 6 deletions(-) diff --git a/Documentation/powerpc/ptrace.txt b

[PATCH 0/2] Changes to PowerPC ptrace flags using watchpoints - v2

2011-12-08 Thread K.Prasad
. Request an delete, followed by set breakpoint request instead. Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-12-08 Thread K.Prasad
On Wed, Dec 07, 2011 at 05:01:57PM -0200, Thiago Jung Bauermann wrote: > On Thu, 2011-12-01 at 15:50 +0530, K.Prasad wrote: > > On Mon, Nov 28, 2011 at 02:11:11PM +1100, David Gibson wrote: > > > [snip] > > > On Wed, Oct 12, 2011 at 11:09:48PM +0530, K.Prasad

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-12-01 Thread K.Prasad
On Mon, Nov 28, 2011 at 02:11:11PM +1100, David Gibson wrote: > [snip] > On Wed, Oct 12, 2011 at 11:09:48PM +0530, K.Prasad wrote: > > > > + if (bp) { > > > > + attr = bp->attr; > > > > + attr.bp_addr = (unsigned long)

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-10-12 Thread K.Prasad
On Wed, Oct 12, 2011 at 02:33:59PM +1100, David Gibson wrote: > On Fri, Sep 16, 2011 at 12:57:10PM +0530, K.Prasad wrote: > > On Fri, Aug 26, 2011 at 03:05:52PM +0530, K.Prasad wrote: > > > On Wed, Aug 24, 2011 at 01:59:39PM +1000, David Gibson wrote: > > > > On T

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-09-16 Thread K.Prasad
On Fri, Aug 26, 2011 at 03:05:52PM +0530, K.Prasad wrote: > On Wed, Aug 24, 2011 at 01:59:39PM +1000, David Gibson wrote: > > On Tue, Aug 23, 2011 at 02:55:13PM +0530, K.Prasad wrote: > > > On Tue, Aug 23, 2011 at 03:08:50PM +1000, David Gibson wrote: > > > > On F

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-08-26 Thread K.Prasad
On Wed, Aug 24, 2011 at 01:59:39PM +1000, David Gibson wrote: > On Tue, Aug 23, 2011 at 02:55:13PM +0530, K.Prasad wrote: > > On Tue, Aug 23, 2011 at 03:08:50PM +1000, David Gibson wrote: > > > On Fri, Aug 19, 2011 at 01:21:36PM +0530, K.Prasad wrote: > > >

Re: [PATCH 2/2] [PowerPC Book3E] Introduce new ptrace debug feature flag

2011-08-23 Thread K.Prasad
On Tue, Aug 23, 2011 at 03:09:31PM +1000, David Gibson wrote: > On Fri, Aug 19, 2011 at 01:23:38PM +0530, K.Prasad wrote: > > > > While PPC_PTRACE_SETHWDEBUG ptrace flag in PowerPC accepts > > PPC_BREAKPOINT_MODE_EXACT mode of breakpoint, the same is not intimated to &

Re: [PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-08-23 Thread K.Prasad
On Tue, Aug 23, 2011 at 03:08:50PM +1000, David Gibson wrote: > On Fri, Aug 19, 2011 at 01:21:36PM +0530, K.Prasad wrote: > > PPC_PTRACE_GETHWDBGINFO, PPC_PTRACE_SETHWDEBUG and PPC_PTRACE_DELHWDEBUG are > > PowerPC specific ptrace flags that use the watchpoint register. Whil

[PATCH 2/2] [PowerPC Book3E] Introduce new ptrace debug feature flag

2011-08-19 Thread K.Prasad
tures" member of "struct ppc_debug_info" to advertise support for the same on Book3E PowerPC processors. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/ptrace.h |1 + arch/powerpc/kernel/ptrace.c |1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff

[PATCH 1/2] [hw-breakpoint] Use generic hw-breakpoint interfaces for new PPC ptrace flags

2011-08-19 Thread K.Prasad
nior Barbosa Machado Signed-off-by: K.Prasad --- Documentation/powerpc/ptrace.txt | 16 ++ arch/powerpc/kernel/ptrace.c | 104 +++--- 2 files changed, 112 insertions(+), 8 deletions(-) diff --git a/Documentation/powerpc/ptrace.txt b/Documentation/powerpc/ptra

[PATCH 0/2] Changes to PowerPC ptrace flags using watchpoints

2011-08-19 Thread K.Prasad
began using the PPC_PTRACE_GETHWDBGINFO/PPC_PTRACE_SETHWDEBUG/PPC_PTRACE_DELHWDEBUG flags on BookS processors is presently unable to set watchpoints. The changes in Patch1, will fix that issue and help it use a common set of code across BookE and BookS. K.Prasad (2): [hw-breakpoint] Use generic hw

Re: [PATCH 2/6] hw_breakpoints: Migrate breakpoint conditional build under new config

2011-07-04 Thread K.Prasad
On Mon, Jul 04, 2011 at 03:29:14PM +0200, Frederic Weisbecker wrote: > On Mon, Jul 04, 2011 at 06:57:46PM +0530, K.Prasad wrote: > > On Tue, May 24, 2011 at 11:52:23PM +0200, Frederic Weisbecker wrote: > > > Migrate conditional hw_breakpoint code compilation under > > >

Re: [PATCH 4/6] hw_breakpoints: Breakpoints arch ability don't need perf events

2011-07-04 Thread K.Prasad
ptions isn't very clear to me (and is a bit confusing with very similar names). Why not make HAVE_HW_BREAKPOINT selectable by the user (which in turn would turn on PERF_EVENTS) for a given architecture? Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH 2/6] hw_breakpoints: Migrate breakpoint conditional build under new config

2011-07-04 Thread K.Prasad
kpoint patches modular has always been a goal. I've looked at the PowerPC parts of the code and they look harmless. Acked-by: K.Prasad Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH 3/5] powerpc, hw_breakpoints: Fix racy access to ptrace breakpoints

2011-04-24 Thread K.Prasad
> + return -ESRCH; > ret = ptrace_set_debugreg(child, addr, data); > + ptrace_put_breakpoints(child); > break; > > #ifdef CONFIG_PPC64 > -- > 1.7.3.2 > Hi Frederic, Looks fine to me. Acked-by: K.Prasad Thanks, K.Prasad __

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-10 Thread K.Prasad
On Mon, Feb 07, 2011 at 04:13:37PM +1100, Benjamin Herrenschmidt wrote: > On Mon, 2011-02-07 at 10:22 +0530, K.Prasad wrote: > > Okay! Another wrapper of "#ifndef CONFIG_HAVE_HW_BREAKPOINT" around > > the > > definition of 'set_debug_reg_defaults'. &g

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread K.Prasad
On Mon, Feb 07, 2011 at 02:10:39PM +1100, Benjamin Herrenschmidt wrote: > On Mon, 2011-02-07 at 08:26 +0530, K.Prasad wrote: > > On Mon, Feb 07, 2011 at 09:54:13AM +1100, Benjamin Herrenschmidt wrote: > > > A typo in the #ifdef statement makes us never call it >

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread K.Prasad
On Mon, Feb 07, 2011 at 09:54:13AM +1100, Benjamin Herrenschmidt wrote: > A typo in the #ifdef statement makes us never call it > in flush_thread() > I wish it never compiled for such typos :-) > Signed-off-by: Benjamin Herrenschmidt Acked-by: K.Prasad > --- > > diff

Re: ppc_set_hwdebug vs ptrace_set_debugreg

2011-01-02 Thread K.Prasad
On Thu, Dec 16, 2010 at 06:07:47PM +0100, Andreas Schwab wrote: > "K.Prasad" writes: > > > How about the revised patch below? It is only compile-tested; have you > > got a quick test case that I can run? > > It crashes the kernel when running the watch-vfork

Re: ppc_set_hwdebug vs ptrace_set_debugreg

2010-12-14 Thread K.Prasad
On Mon, Dec 13, 2010 at 08:05:36PM +0100, Andreas Schwab wrote: > "K.Prasad" writes: > > > +#ifdef CONFIG_HAVE_HW_BREAKPOINT > > + /* Create a new breakpoint request if one doesn't exist already */ > > + hw_breakpoint_init(&attr); > > + at

Re: ppc_set_hwdebug vs ptrace_set_debugreg

2010-12-13 Thread K.Prasad
On Wed, Dec 01, 2010 at 10:07:58AM +0530, K.Prasad wrote: > On Mon, Nov 29, 2010 at 11:15:51AM +0100, Andreas Schwab wrote: > > "K.Prasad" writes: > > > > > Although ppc_set_hwdebug() can set DABR through set_dabr() in > > > arch/powerpc/kernel/proces

Re: ppc_set_hwdebug vs ptrace_set_debugreg

2010-11-30 Thread K.Prasad
On Mon, Nov 29, 2010 at 11:15:51AM +0100, Andreas Schwab wrote: > "K.Prasad" writes: > > > Although ppc_set_hwdebug() can set DABR through set_dabr() in > > arch/powerpc/kernel/process.c, it is good to have it converted to use > > register_user_hw_breakpoint()

Re: ppc_set_hwdebug vs ptrace_set_debugreg

2010-11-28 Thread K.Prasad
eakpoint() to be used for these new ptrace flags? Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH 02/13] powerpc/book3e: Hack to get gdb moving along on Book3E 64-bit

2010-07-09 Thread K.Prasad
in processor.h however "u32 dbcr0" here must be fine (given that DBCR0 uses 32-bits in ppc32 and uses only 32:63 bits in BOOKIIIE_64). Should dbcr<0-n> be made u32, given that there will be no 64-bit long value to store (or am I missing something)? Thanks, K.Prasad >

[RFC Patch 1/1] Implement hardware breakpoint interfaces for PowerPC BookE processors

2010-06-29 Thread K.Prasad
Introduce support for generic hw-breakpoint interfaces for PowerPC BookIII E processors. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |2 arch/powerpc/include/asm/cputable.h|4 arch/powerpc/include/asm/hw_breakpoint_booke.h | 46 ++ arch

[RFC Patch 0/1] hw-breakpoint interfaces for BookE - ver II

2010-06-29 Thread K.Prasad
patch progress. Kindly let me know your comments. Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXII

2010-06-14 Thread K.Prasad
On Tue, Jun 15, 2010 at 11:54:59AM +1000, Paul Mackerras wrote: > On Fri, Jun 04, 2010 at 12:21:45PM +0530, K.Prasad wrote: > > > Meanwhile I tested the per-cpu breakpoints with the new emulate_step > > patch (refer linuxppc-dev message-id: > > 20100602112903.gb30...@b

Re: [Patch 5/5] PPC64-HWBKPT: Discard extraneous interrupt due to accesses outside symbol length

2010-06-14 Thread K.Prasad
On Thu, Jun 10, 2010 at 10:40:24PM +1000, Paul Mackerras wrote: > On Wed, Jun 09, 2010 at 03:55:59PM +0530, K.Prasad wrote: > > > + if (!((bp->attr.bp_addr <= dar) && > > +(dar <= (bp->attr.bp_addr + bp->attr.bp_len { > > + /*

[Patch 5/5] PPC64-HWBKPT: Discard extraneous interrupt due to accesses outside symbol length

2010-06-14 Thread K.Prasad
(we don't want to change ptrace behaviour for fear of breaking compatability). [Suggestions from Paul Mackerras to add a new flag in 'struct arch_hw_breakpoint' to identify extraneous interrupts] Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |1 + arch

[Patch 4/5] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-06-14 Thread K.Prasad
delivery. Restore hw-breakpoints if the user-context is altered in the signal handler. [With inputs from Paul Mackerras which helped identify a need to restore breakpoints before handling a signal delivered in certain cases] Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h

[Patch 3/5] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-06-14 Thread K.Prasad
single-step handler after alignment interrupt pointed out by Paul Mackerras ] Signed-off-by: K.Prasad --- arch/powerpc/kernel/traps.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) Index: linux-2.6.ppc64_test/arch/powerpc/kernel/traps.c

[Patch 2/5] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC BookIII S

2010-06-14 Thread K.Prasad
per-task breakpoints - perform arch-specific cleanup before unregistration through arch_unregister_hw_breakpoint() ] Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm/cputable.h |4 arch/powerpc/include/asm/hw_breakpoint.h | 73

[Patch 1/5] Allow arch-specific cleanup before breakpoint unregistration

2010-06-14 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad Acked-by

[Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXIV

2010-06-14 Thread K.Prasad
extraneous exceptions (through clearing of extraneous exception flag and preventing the invocation of callback). Kindly let me know if you have any further comments. Thanks, K.Prasad Changelog - ver XXIII (Version XXII: 20100528063924.ga8...@in.ibm.com) - Detection of

Re: [Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXII

2010-06-09 Thread K.Prasad
On Mon, Jun 07, 2010 at 09:25:59PM +1000, Paul Mackerras wrote: > On Mon, Jun 07, 2010 at 12:33:51PM +0530, K.Prasad wrote: > > > Given that 'ptrace_bps' is used only for ptrace originated breakpoints > > and that we return early i.e. before detecting

[Patch 5/5] PPC64-HWBKPT: Discard extraneous interrupt due to accesses outside symbol length

2010-06-09 Thread K.Prasad
(we don't want to change ptrace behaviour for fear of breaking compatability). Signed-off-by: K.Prasad Signed-off-by: Paul Mackerras --- arch/powerpc/include/asm/hw_breakpoint.h |1 + arch/powerpc/kernel/hw_breakpoint.c | 19 ++- 2 files changed, 19 insertions(

[Patch 4/5] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-06-09 Thread K.Prasad
delivery. Restore hw-breakpoints if the user-context is altered in the signal handler. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |3 +++ arch/powerpc/kernel/hw_breakpoint.c | 18 ++ arch/powerpc/kernel/signal.c |3 +++ 3 files

[Patch 3/5] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-06-09 Thread K.Prasad
An alignment interrupt may intervene between a DSI/hw-breakpoint exception and the single-step exception. Enable the alignment interrupt (through modifications to emulate_single_step()) to notify the single-step exception handler for proper restoration of hw-breakpoints. Signed-off-by: K.Prasad

[Patch 2/5] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC BookIII S

2010-06-09 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

[Patch 1/5] Allow arch-specific cleanup before breakpoint unregistration

2010-06-09 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad Acked-by

[Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXIII

2010-06-09 Thread K.Prasad
extraneous breakpoint exceptions is now done using a boolean flag in 'struct arch_hw_breakpoint'. - A dangling put_cpu() (remnant from previous patch versions) in arch_unregister_hw_breakpoint() is now removed. Kindly let me know your comments. Thanks, K.Prasad Changelog

Re: [Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXII

2010-06-07 Thread K.Prasad
On Fri, Jun 04, 2010 at 07:06:48PM +1000, Paul Mackerras wrote: > On Fri, Jun 04, 2010 at 12:21:45PM +0530, K.Prasad wrote: > > > Meanwhile I tested the per-cpu breakpoints with the new emulate_step > > patch (refer linuxppc-dev message-id: > > 20100602112903.gb30...@b

Re: [Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXII

2010-06-03 Thread K.Prasad
On Wed, Jun 02, 2010 at 09:33:16PM +1000, Paul Mackerras wrote: > On Fri, May 28, 2010 at 12:09:24PM +0530, K.Prasad wrote: > > > Please find a new set of patches that have the following changes. > > Thanks. There are a couple of minor things still remaining (dan

Re: [RFC PATCH] powerpc: Emulate most load and store instructions in emulate_step()

2010-06-01 Thread K.Prasad
t; FP/VMX/VSX registers, using assembly helpers in ldstfp.S. > Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [Patch 3/4] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-05-28 Thread K.Prasad
On Thu, May 27, 2010 at 04:20:44PM +1000, Paul Mackerras wrote: > On Tue, May 25, 2010 at 02:44:35PM +0530, K.Prasad wrote: > > > An alignment interrupt may intervene between a DSI/hw-breakpoint exception > > and the single-step exception. Enable the alignment interrupt (through

Re: [Patch 2/4] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC BookIII S

2010-05-28 Thread K.Prasad
On Thu, May 27, 2010 at 04:19:40PM +1000, Paul Mackerras wrote: > On Tue, May 25, 2010 at 02:44:20PM +0530, K.Prasad wrote: > > > Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S > > processors. These interfaces help arbitrate requests from

[Patch 5/5] PPC64-HWBKPT: Discard extraneous interrupt due to accesses outside symbol length

2010-05-27 Thread K.Prasad
(we don't want to change ptrace behaviour for fear of breaking compatability). Signed-off-by: K.Prasad --- arch/powerpc/kernel/hw_breakpoint.c | 39 1 file changed, 35 insertions(+), 4 deletions(-) Index: linux-2.6.ppc64_test/arch/powerpc/k

[Patch 4/5] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-05-27 Thread K.Prasad
delivery. Restore hw-breakpoints if the user-context is altered in the signal handler. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |3 +++ arch/powerpc/kernel/hw_breakpoint.c | 18 ++ arch/powerpc/kernel/signal.c |3 +++ 3 files

[Patch 3/5] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-05-27 Thread K.Prasad
An alignment interrupt may intervene between a DSI/hw-breakpoint exception and the single-step exception. Enable the alignment interrupt (through modifications to emulate_single_step()) to notify the single-step exception handler for proper restoration of hw-breakpoints. Signed-off-by: K.Prasad

[Patch 2/5] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC BookIII S

2010-05-27 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

[Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXII

2010-05-27 Thread K.Prasad
' and 'last_hit_ubp' members are now promptly cleaned-up. - Single-step exception is now conditionally emulated upon hitting alignment_exception. - Rebased to commit 31f46717997a83bdf6db0dd04810c0a329eb3148 of linux-2.6 tree. Kindly let me know your comments for the same. Thanks,

[Patch 1/5] Allow arch-specific cleanup before breakpoint unregistration

2010-05-27 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad Acked-by

Re: [Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-26 Thread K.Prasad
On Wed, May 26, 2010 at 07:23:15PM +0200, Frederic Weisbecker wrote: > On Wed, May 26, 2010 at 10:47:42PM +0530, K.Prasad wrote: > > On Wed, May 26, 2010 at 10:54:41AM +0100, David Howells wrote: > > > K.Prasad wrote: > > > > > > > > My understanding is

Re: [Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-26 Thread K.Prasad
On Wed, May 26, 2010 at 10:47:42PM +0530, K.Prasad wrote: > On Wed, May 26, 2010 at 10:54:41AM +0100, David Howells wrote: > > K.Prasad wrote: > > > > > > My understanding is weak function definitions must appear in a > > > > different C > > &

Re: [Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-26 Thread K.Prasad
On Wed, May 26, 2010 at 10:54:41AM +0100, David Howells wrote: > K.Prasad wrote: > > > > My understanding is weak function definitions must appear in a different C > > > file than their call sites to work on some toolchains. > > > > > > > Atleast

Re: [Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-25 Thread K.Prasad
On Tue, May 25, 2010 at 06:39:19AM -0500, Millton Miller wrote: > On Tue, 25 May 2010 at 14:43:56 +0530, K.Prasad wrote: > > Certain architectures (such as PowerPC Book III S) have a need to cleanup > > data-structures before the breakpoint is unregistered. This patch introdu

[Patch 4/4] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-05-25 Thread K.Prasad
delivery. Restore hw-breakpoints if the user-context is altered in the signal handler. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |2 ++ arch/powerpc/kernel/hw_breakpoint.c | 16 arch/powerpc/kernel/signal.c |3 +++ arch

[Patch 3/4] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-05-25 Thread K.Prasad
An alignment interrupt may intervene between a DSI/hw-breakpoint exception and the single-step exception. Enable the alignment interrupt (through modifications to emulate_single_step()) to notify the single-step exception handler for proper restoration of hw-breakpoints. Signed-off-by: K.Prasad

[Patch 2/4] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC BookIII S

2010-05-25 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

[Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-25 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad --- kernel

[Patch 0/4] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XXI

2010-05-25 Thread K.Prasad
x27;t have to be cleared during sigreturn. A 'double-hit' on hw_breakpoint_handler() is harmless for non-ptrace instructions. - Minor changes to enhance code brevity. Kindly let me know your comments. Thanks, K.Prasad Changelog - ver XX (Version XIX: l

Re: [Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-05-24 Thread K.Prasad
On Thu, May 20, 2010 at 11:10:03PM +1000, Paul Mackerras wrote: > On Thu, May 20, 2010 at 09:36:03AM +0530, K.Prasad wrote: > (Had this mail composed along with the patchset...but mail server issues caused delay in sending this...) Hi Paul, While we continue to discuss some of the

[Patch 1/4] Allow arch-specific cleanup before breakpoint unregistration

2010-05-24 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad --- kernel

[Patch 0/4] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XX

2010-05-24 Thread K.Prasad
oints will only be emulated. Breakpoint will be unregistered with a warning if emulation fails. Thanks, K.Prasad Changelog - ver XIX (Version XVIII: linuxppc-dev ref: 20100512033055.ga6...@in.ibm.com) - Increased coverage of breakpoints during concurrent alignment_exception

[Patch 4/4] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-05-24 Thread K.Prasad
red in the signal handler (causing loss of MSR_SE). Side-effect: 'Double-hit' of breakpoint if the instruction pointer is unaltered in the new context. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |3 +++ arch/powerpc/kernel/hw_brea

[Patch 3/4] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-05-24 Thread K.Prasad
An alignment interrupt may intervene between a DSI/hw-breakpoint exception and the single-step exception. Enable the alignment interrupt (through modifications to emulate_single_step()) to notify the single-step exception handler for proper restoration of hw-breakpoints. Signed-off-by: K.Prasad

[Patch 2/4] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC Book III S

2010-05-24 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

Re: [RFC Patch 2/5] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC Book III S

2010-05-23 Thread K.Prasad
On Mon, May 24, 2010 at 09:32:25AM +0530, K.Prasad wrote: > Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S > processors. These interfaces help arbitrate requests from various users and > schedules them as appropriate. > > +/* > + * Hand

[RFC Patch 5/5] PPC64-HWBKPT: Enable proper distinction of per-task and per-cpu breakpoints

2010-05-23 Thread K.Prasad
This helps seamless handling of kernel-space breakpoints within the context of a user-space process and breakpoints for kernel-threads. Signed-off-by: K.Prasad --- arch/powerpc/kernel/hw_breakpoint.c | 24 +--- include/linux/perf_event.h |1 + kernel/pe

[RFC Patch 4/5] PPC64-HWBKPT: Enable hw-breakpoints while handling intervening signals

2010-05-23 Thread K.Prasad
red in the signal handler (causing loss of MSR_SE). Side-effect: 'Double-hit' of breakpoint if the instruction pointer is unaltered in the new context. Signed-off-by: K.Prasad --- arch/powerpc/include/asm/hw_breakpoint.h |3 +++ arch/powerpc/kernel/hw_brea

[RFC Patch 3/5] PPC64-HWBKPT: Handle concurrent alignment interrupts

2010-05-23 Thread K.Prasad
An alignment interrupt may intervene between a DSI/hw-breakpoint exception and the single-step exception. Enable the alignment interrupt (through modifications to emulate_single_step()) to notify the single-step exception handler for proper restoration of hw-breakpoints. Signed-off-by: K.Prasad

[RFC Patch 2/5] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC Book III S

2010-05-23 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

[RFC Patch 1/5] Allow arch-specific cleanup before breakpoint unregistration

2010-05-23 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad --- kernel

[RFC Patch 0/5] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XIX

2010-05-23 Thread K.Prasad
based to commit f4b87dee923342505e1ddba8d34ce9de33e75050, thereby necessitating minor changes to arch_validate_hwbkpt_settings(). Thanks, K.Prasad Changelog - ver XVIII (Version XVII: linuxppc-dev ref: 20100414034340.ga6...@in.ibm.com) - hw-breakpoint restoration variables are cl

Re: [Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-05-19 Thread K.Prasad
On Mon, May 17, 2010 at 10:32:41PM +1000, Paul Mackerras wrote: > On Fri, May 14, 2010 at 12:25:31PM +0530, K.Prasad wrote: > > > Okay. I will re-use single_step_exception() after modifications; it > > appearsto have no in-kernel users for it. > > It's called from ex

Re: [Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-05-13 Thread K.Prasad
On Wed, May 12, 2010 at 04:32:47PM +1000, Paul Mackerras wrote: > On Wed, May 05, 2010 at 02:03:03AM +0530, K.Prasad wrote: > > > It is true that the breakpoint exceptions will go amiss following the > > alignment exception, and be restored when the thread single-steps due >

Re: [Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-05-11 Thread K.Prasad
On Wed, May 05, 2010 at 02:03:03AM +0530, K.Prasad wrote: > On Mon, May 03, 2010 at 04:23:30PM +1000, Paul Mackerras wrote: > > On Wed, Apr 14, 2010 at 09:18:27AM +0530, K.Prasad wrote: > > [snipped] > > It has been pointed out to me before (Roland's mail Ref

[Patch 3/3] PPC64-HWBKPT: Implement hw-breakpoints for PowerPC Book III S

2010-05-11 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PowerPC Book III S processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm

[Patch 2/3] Allow arch-specific cleanup before breakpoint unregistration

2010-05-11 Thread K.Prasad
Certain architectures (such as PowerPC Book III S) have a need to cleanup data-structures before the breakpoint is unregistered. This patch introduces an arch-specific hook in release_bp_slot() along with a weak definition in the form of a stub funciton. Signed-off-by: K.Prasad --- kernel

[Patch 1/3] PPC64-HWBKPT: Disable interrupts for data breakpoint exceptions

2010-05-11 Thread K.Prasad
Data address breakpoint exceptions are currently handled along with page-faults which require interrupts to remain in enabled state. Since exception handling for data breakpoints aren't pre-empt safe, we handle them separately. Signed-off-by: K.Prasad Acked-by: Paul Mackerras --- arch/po

[Patch 0/3] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XVIII

2010-05-11 Thread K.Prasad
handlers; causes single_step_dabr_instruction() to be invoked after alignment_exception. - SIGTRAP is no longer generated for non-ptrace user-space breakpoints. - Slight code restructuring for brevity and style corrections. Kindly accept them to be a part of -next tree. Thanks, K.Prasad Changelog

Re: [Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-05-04 Thread K.Prasad
On Mon, May 03, 2010 at 04:23:30PM +1000, Paul Mackerras wrote: > On Wed, Apr 14, 2010 at 09:18:27AM +0530, K.Prasad wrote: > > > Implement perf-events based hw-breakpoint interfaces for PPC64 processors. > > These interfaces help arbitrate requests from various users and sch

[RFC Patch 1/1] Implement hw-breakpoint interfaces for BookE processors

2010-04-27 Thread K.Prasad
Implement hardware breakpoint interfaces for PowerPC BookE processors Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |2 arch/powerpc/include/asm/cputable.h|4 arch/powerpc/include/asm/hw_breakpoint_booke.h | 42 +++ arch/powerpc/kernel

[RFC Patch 0/1] [hw-bkpt BookE] hw-breakpoint interfaces for BookE - ver I

2010-04-27 Thread K.Prasad
BookS code intergration (hw_breakpoint.c and hw_breakpoint_booke.c) - Code clean-up and reduction. Kindly let me know about comments/suggestions, if any. Thank You, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.

Re: [Patch 1/1] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-04-13 Thread K.Prasad
limited to scope using CONFIG_HAVE_HW_BREAKPOINT. At this stage, I didnot find a need for a seperate CONFIG_HW_BREAKPOINTS though. Let me know what you think. Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-04-13 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PPC64 processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm/cputable.h |4

[Patch 1/2] PPC64-HWBKPT: Disable interrupts for data breakpoint exceptions

2010-04-13 Thread K.Prasad
Data address breakpoint exceptions are currently handled along with page-faults which require interrupts to remain in enabled state. Since exception handling for data breakpoints aren't pre-empt safe, we handle them separately. Signed-off-by: K.Prasad --- arch/powerpc/kernel/exceptions-

[Patch 0/2] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XVII

2010-04-13 Thread K.Prasad
. Thanks, K.Prasad Changelog - ver XVI (Version XV: linuxppc-dev ref: 20100323140639.ga21...@in.ibm.com) - Used a new config option CONFIG_PPC_BOOK3S_64 (in lieu of CONFIG_PPC64/CPU_FTR_HAS_DABR) to limit the scope of the new code. - Disabled breakpoints before kexec of the

Re: [RFC Patch 1/2] PPC64-HWBKPT: Disable interrupts for data breakpoint exceptions

2010-03-30 Thread K.Prasad
On Tue, Mar 30, 2010 at 04:32:25PM +1100, Benjamin Herrenschmidt wrote: > On Tue, 2010-03-30 at 16:24 +1100, Paul Mackerras wrote: > > On Tue, Mar 23, 2010 at 07:37:02PM +0530, K.Prasad wrote: > > > > > Index: linux-2.6.ppc64_test/arch/powerpc

Re: [RFC Patch 1/2] PPC64-HWBKPT: Disable interrupts for data breakpoint exceptions

2010-03-30 Thread K.Prasad
On Tue, Mar 30, 2010 at 04:24:42PM +1100, Paul Mackerras wrote: > On Tue, Mar 23, 2010 at 07:37:02PM +0530, K.Prasad wrote: > > > Index: linux-2.6.ppc64_test/arch/powerpc/kernel/exceptions-64s.S > > ===

Re: [RFC Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-03-30 Thread K.Prasad
On Mon, Mar 29, 2010 at 02:53:36PM -0500, Dave Kleikamp wrote: > On Mon, 2010-03-29 at 17:01 +0530, K.Prasad wrote: > > On Fri, Mar 26, 2010 at 04:11:45PM -0500, Dave Kleikamp wrote: > > > On Tue, 2010-03-23 at 19:37 +0530, K.Prasad wrote: > > > > plain text docum

[Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-03-30 Thread K.Prasad
Implement perf-events based hw-breakpoint interfaces for PPC64 processors. These interfaces help arbitrate requests from various users and schedules them as appropriate. Signed-off-by: K.Prasad --- arch/powerpc/Kconfig |1 arch/powerpc/include/asm/cputable.h |4

[Patch 1/2] PPC64-HWBKPT: Disable interrupts for data breakpoint exceptions

2010-03-30 Thread K.Prasad
Data address breakpoint exceptions are currently handled along with page-faults which require interrupts to remain in enabled state. Since exception handling for data breakpoints aren't pre-empt safe, we handle them separately. Signed-off-by: K.Prasad --- arch/powerpc/kernel/exceptions-

[Patch 0/2] PPC64-HWBKPT: Hardware Breakpoint interfaces - ver XVI

2010-03-30 Thread K.Prasad
exceptions in DSISR finally (after check for other causes) + changes in code comments and representation of DSISR_DABRMATCH constant. - Rebased to commit ae6be51ed01d6c4aaf249a207b4434bc7785853b of linux-2.6. Let me know if you have any comments. Thanks, K.Prasad Changelog - ver XV

Re: [RFC Patch 2/2] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-03-29 Thread K.Prasad
On Fri, Mar 26, 2010 at 04:11:45PM -0500, Dave Kleikamp wrote: > On Tue, 2010-03-23 at 19:37 +0530, K.Prasad wrote: > > plain text document attachment (ppc64_hbkpt_02) > > Implement perf-events based hw-breakpoint interfaces for PPC64 processors. > > These interfaces help ar

  1   2   >