aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Engraf <david.engraf@sysgo.com>2011-03-23 11:35:42 +0000
committerGreg Kroah-Hartman <gregkh@suse.de>2011-03-27 12:00:24 -0700
commit0030f64fbb5c6a6eb943a2ec1fdc43e17b0b124c (patch)
treecde32c200547224d65ea17b71999c1f7265471b8
parentsh: Fix ptrace fpu state initialisation (diff)
downloadlinux-stable-0030f64fbb5c6a6eb943a2ec1fdc43e17b0b124c.tar.xz
linux-stable-0030f64fbb5c6a6eb943a2ec1fdc43e17b0b124c.zip
sh: Fix ptrace hw_breakpoint handling
commit fb7f045ace0624f1e59a7db8497e460bd54b1cbc upstream. Since commit 34d0b5af50a063cded842716633501b38ff815fb it is no longer possible to debug an application using singlestep. The old commit converted singlestep handling via ptrace to hw_breakpoints. The hw_breakpoint is disabled when an event is triggered and not re-enabled again. This patch re-enables the existing hw_breakpoint before the existing breakpoint is reused. Signed-off-by: David Engraf <david.engraf@sysgo.com> Signed-off-by: Paul Mundt <lethal@linux-sh.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--arch/sh/kernel/ptrace_32.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/sh/kernel/ptrace_32.c b/arch/sh/kernel/ptrace_32.c
index b53664ef53e8..2130ca674e9b 100644
--- a/arch/sh/kernel/ptrace_32.c
+++ b/arch/sh/kernel/ptrace_32.c
@@ -101,6 +101,8 @@ static int set_single_step(struct task_struct *tsk, unsigned long addr)
attr = bp->attr;
attr.bp_addr = addr;
+ /* reenable breakpoint */
+ attr.disabled = false;
err = modify_user_hw_breakpoint(bp, &attr);
if (unlikely(err))
return err;