Commit 8c501acc authored by He Chuyue's avatar He Chuyue Committed by guzitao
Browse files

sw64: remove single step setting in uprobe

Sunway inclusion
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I5XTMN



--------------------------------

There is no point in calling user_{enable,disable}_single_step()
since uprobe breakpoints are implemented by illegal instructions
on sw64.

The arch-specific implementation of these two functions have been
removed before, and it's time to remove the calling now.

Signed-off-by: default avatarHe Chuyue <hechuyue@wxiat.com>
Reviewed-by: default avatarHe Sheng <hesheng@wxiat.com>
Signed-off-by: default avatarGu Zitao <guzitao@wxiat.com>
parent e9df0c72
Loading
Loading
Loading
Loading
+0 −6
Original line number Diff line number Diff line
@@ -4,8 +4,6 @@
#include <linux/uprobes.h>
#include <linux/ptrace.h>

#define UPROBE_TRAP_NR	ULONG_MAX

/**
 * arch_uprobe_analyze_insn - instruction analysis including validity and fixups.
 * @mm: the probed address space.
@@ -54,8 +52,6 @@ int arch_uprobe_pre_xol(struct arch_uprobe *aup, struct pt_regs *regs)
	/* Instruction points to execute ol */
	instruction_pointer_set(regs, utask->xol_vaddr);

	user_enable_single_step(current);

	return 0;
}

@@ -66,8 +62,6 @@ int arch_uprobe_post_xol(struct arch_uprobe *aup, struct pt_regs *regs)
	/* Instruction points to execute next to breakpoint address */
	instruction_pointer_set(regs, utask->vaddr + 4);

	user_disable_single_step(current);

	return 0;
}