diff options
| author | K.Prasad <prasad@linux.vnet.ibm.com> | 2010-06-15 11:35:41 +0530 |
|---|---|---|
| committer | Paul Mackerras <paulus@samba.org> | 2010-06-22 19:40:50 +1000 |
| commit | 06532a6743d83fac4b79389fc8c86c88cb4e3302 (patch) | |
| tree | 2ca694f5cb71a2296c6b4e0b36f3ddd6e2033084 /arch/powerpc/include/asm/hw_breakpoint.h | |
| parent | 2538c2d08f46141550a1e68819efa8fe31c6e3dc (diff) | |
| download | linux-06532a6743d83fac4b79389fc8c86c88cb4e3302.tar.gz linux-06532a6743d83fac4b79389fc8c86c88cb4e3302.tar.xz | |
powerpc, hw_breakpoint: Enable hw-breakpoints while handling intervening signals
A signal delivered between a hw_breakpoint_handler() and the
single_step_dabr_instruction() will not have the breakpoint active
while the signal handler is running -- the signal delivery will
set up a new MSR value which will not have MSR_SE set, so we
won't get the signal step interrupt until and unless the signal
handler returns (which it may never do).
To fix this, we restore the breakpoint when delivering a signal --
we clear the MSR_SE bit and set the DABR again. If the signal
handler returns, the DABR interrupt will occur again when the
instruction that we were originally trying to single-step gets
re-executed.
[Paul Mackerras <paulus@samba.org> pointed out the need to do this.]
Signed-off-by: K.Prasad <prasad@linux.vnet.ibm.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/include/asm/hw_breakpoint.h')
| -rw-r--r-- | arch/powerpc/include/asm/hw_breakpoint.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/arch/powerpc/include/asm/hw_breakpoint.h b/arch/powerpc/include/asm/hw_breakpoint.h index b111713b593e..6576bad1069c 100644 --- a/arch/powerpc/include/asm/hw_breakpoint.h +++ b/arch/powerpc/include/asm/hw_breakpoint.h @@ -65,9 +65,12 @@ static inline void hw_breakpoint_disable(void) { set_dabr(0); } +extern void thread_change_pc(struct task_struct *tsk, struct pt_regs *regs); #else /* CONFIG_HAVE_HW_BREAKPOINT */ static inline void hw_breakpoint_disable(void) { } +static inline void thread_change_pc(struct task_struct *tsk, + struct pt_regs *regs) { } #endif /* CONFIG_HAVE_HW_BREAKPOINT */ #endif /* __KERNEL__ */ #endif /* _PPC_BOOK3S_64_HW_BREAKPOINT_H */ |