Re: [PATCH v4] arm64: Improve kprobes test for atomic sequence

2016-09-14 Thread Masami Hiramatsu
On Mon, 12 Sep 2016 21:07:40 -0400 David Long wrote: > > After the patch the function reads as follows: > > > enum kprobe_insn __kprobes > > arm_kprobe_decode_insn(kprobe_opcode_t *addr, struct arch_specific_insn > > *asi) > > { > > enum kprobe_insn decoded; > > kprobe_opcode_t insn = l

Re: [PATCH v4] arm64: Improve kprobes test for atomic sequence

2016-09-12 Thread David Long
On 09/12/2016 08:46 PM, Masami Hiramatsu wrote: On Mon, 12 Sep 2016 14:21:27 -0400 David Long wrote: From: "David A. Long" Kprobes searches backwards a finite number of instructions to determine if there is an attempt to probe a load/store exclusive sequence. It stops when it hits the maximu

Re: [PATCH v4] arm64: Improve kprobes test for atomic sequence

2016-09-12 Thread Masami Hiramatsu
On Mon, 12 Sep 2016 14:21:27 -0400 David Long wrote: > From: "David A. Long" > > Kprobes searches backwards a finite number of instructions to determine if > there is an attempt to probe a load/store exclusive sequence. It stops when > it hits the maximum number of instructions or a load or sto

[PATCH v4] arm64: Improve kprobes test for atomic sequence

2016-09-12 Thread David Long
From: "David A. Long" Kprobes searches backwards a finite number of instructions to determine if there is an attempt to probe a load/store exclusive sequence. It stops when it hits the maximum number of instructions or a load or store exclusive. However this means it can run up past the beginning