aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/ptrace.c
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@redhat.com>2013-07-08 19:00:54 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2013-07-09 13:33:26 -0400
commit7c8df28633bf0b7eb253f866029be0ac59ddb062 (patch)
treed8fba38a54a339d460026f79952cab16240e4e5a /kernel/ptrace.c
parente8c073c4ff51207f5c1c37fb054360bbc0f38251 (diff)
ptrace: revert "Prepare to fix racy accesses on task breakpoints"
This reverts commit bf26c018490c ("Prepare to fix racy accesses on task breakpoints"). The patch was fine but we can no longer race with SIGKILL after commit 9899d11f6544 ("ptrace: ensure arch_ptrace/ptrace_request can never race with SIGKILL"), the __TASK_TRACED tracee can't be woken up and ->ptrace_bps[] can't go away. Now that ptrace_get_breakpoints/ptrace_put_breakpoints have no callers, we can kill them and remove task->ptrace_bp_refcnt. Signed-off-by: Oleg Nesterov <oleg@redhat.com> Acked-by: Frederic Weisbecker <fweisbec@gmail.com> Acked-by: Michael Neuling <mikey@neuling.org> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Ingo Molnar <mingo@kernel.org> Cc: Jan Kratochvil <jan.kratochvil@redhat.com> Cc: Paul Mackerras <paulus@samba.org> Cc: Paul Mundt <lethal@linux-sh.org> Cc: Will Deacon <will.deacon@arm.com> Cc: Prasad <prasad@linux.vnet.ibm.com> Cc: Russell King <linux@arm.linux.org.uk> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/ptrace.c')
-rw-r--r--kernel/ptrace.c16
1 files changed, 0 insertions, 16 deletions
diff --git a/kernel/ptrace.c b/kernel/ptrace.c
index ba5e6cea181a..a146ee327f6a 100644
--- a/kernel/ptrace.c
+++ b/kernel/ptrace.c
@@ -1221,19 +1221,3 @@ asmlinkage long compat_sys_ptrace(compat_long_t request, compat_long_t pid,
1221 return ret; 1221 return ret;
1222} 1222}
1223#endif /* CONFIG_COMPAT */ 1223#endif /* CONFIG_COMPAT */
1224
1225#ifdef CONFIG_HAVE_HW_BREAKPOINT
1226int ptrace_get_breakpoints(struct task_struct *tsk)
1227{
1228 if (atomic_inc_not_zero(&tsk->ptrace_bp_refcnt))
1229 return 0;
1230
1231 return -1;
1232}
1233
1234void ptrace_put_breakpoints(struct task_struct *tsk)
1235{
1236 if (atomic_dec_and_test(&tsk->ptrace_bp_refcnt))
1237 flush_ptrace_hw_breakpoint(tsk);
1238}
1239#endif /* CONFIG_HAVE_HW_BREAKPOINT */