On 09/08, Jiri Olsa wrote:
>
> If user decided to take execution elsewhere, it makes little sense
> to execute the original instruction, so let's skip it.

...

> @@ -2768,6 +2768,13 @@ static void handle_swbp(struct pt_regs *regs)
>       /* Try to optimize after first hit. */
>       arch_uprobe_optimize(&uprobe->arch, bp_vaddr);
>
> +     /*
> +      * If user decided to take execution elsewhere, it makes little sense
> +      * to execute the original instruction, so let's skip it.
> +      */
> +     if (instruction_pointer(regs) != bp_vaddr)
> +             goto out;
> +

Acked-by: Oleg Nesterov <o...@redhat.com>


Reply via email to