aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/events
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@redhat.com>2012-10-28 11:55:36 -0400
committerOleg Nesterov <oleg@redhat.com>2012-11-03 12:15:12 -0400
commit65b2c8f0e53347583168423de0f32227d8baf01b (patch)
tree87b1e9fb5dcbc3e2a5a59192c27b959dc6cc0323 /kernel/events
parentf57d56dd29003435d1bfc0e675896c368180f8ec (diff)
uprobes/powerpc: Do not use arch_uprobe_*_step() helpers
No functional changes. powerpc is the only user of arch_uprobe_enable/disable_step() helpers, but they should die. They can not be used correctly, every arch needs its own implementation (like x86 does). And they do not really help even as initial-and-almost-working code, arch_uprobe_*_xol() hooks can easily use user_enable/disable_single_step() directly. Change arch_uprobe_*_step() to do nothing, and convert powerpc to use ptrace helpers. This is equally wrong, powerpc needs the arch-specific fixes. Signed-off-by: Oleg Nesterov <oleg@redhat.com> Acked-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com> Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/events')
-rw-r--r--kernel/events/uprobes.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c
index 5cc4e7e42e68..abbfd8440a6d 100644
--- a/kernel/events/uprobes.c
+++ b/kernel/events/uprobes.c
@@ -1432,12 +1432,10 @@ static struct uprobe *find_active_uprobe(unsigned long bp_vaddr, int *is_swbp)
1432 1432
1433void __weak arch_uprobe_enable_step(struct arch_uprobe *arch) 1433void __weak arch_uprobe_enable_step(struct arch_uprobe *arch)
1434{ 1434{
1435 user_enable_single_step(current);
1436} 1435}
1437 1436
1438void __weak arch_uprobe_disable_step(struct arch_uprobe *arch) 1437void __weak arch_uprobe_disable_step(struct arch_uprobe *arch)
1439{ 1438{
1440 user_disable_single_step(current);
1441} 1439}
1442 1440
1443/* 1441/*