Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1454858

Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to NOKPROBE_SYMBOL

Show all headers | View raw

On Tue,  2 Aug 2016 12:14:06 +0530
Pratyush Anand <panand@redhat.com> wrote:

> uprobe_pre_sstep_notifier and uprobe_post_sstep_notifier are called from
> debug exception handler, so blacklist them for kprobing.

Actually, these exception notifers are kicked only if the debug exception
is not related to kprobes (at least on x86). In that case, we don't have
to take care about that. Or, would you hit any problem on it?

IOW, where do we have to prohibit kprobes are, the code path from where 
right after the breakpoint (debug) exception is occurred, to where right
before the kprobe is handled. After that, it should be safe.

Thank you,


> 
> Signed-off-by: Pratyush Anand <panand@redhat.com>
> ---
>  kernel/events/uprobes.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c
> index b7a525ab2083..206e594cb65e 100644
> --- a/kernel/events/uprobes.c
> +++ b/kernel/events/uprobes.c
> @@ -37,6 +37,7 @@
>  #include <linux/percpu-rwsem.h>
>  #include <linux/task_work.h>
>  #include <linux/shmem_fs.h>
> +#include <linux/kprobes.h>
>  
>  #include <linux/uprobes.h>
>  
> @@ -1997,6 +1998,7 @@ int uprobe_pre_sstep_notifier(struct pt_regs *regs)
>  	set_thread_flag(TIF_UPROBE);
>  	return 1;
>  }
> +NOKPROBE_SYMBOL(uprobe_pre_sstep_notifier);
>  
>  /*
>   * uprobe_post_sstep_notifier gets called in interrupt context as part of notifier
> @@ -2014,6 +2016,7 @@ int uprobe_post_sstep_notifier(struct pt_regs *regs)
>  	set_thread_flag(TIF_UPROBE);
>  	return 1;
>  }
> +NOKPROBE_SYMBOL(uprobe_post_sstep_notifier);
>  
>  static struct notifier_block uprobe_exception_nb = {
>  	.notifier_call		= arch_uprobe_exception_notify,
> -- 
> 2.5.5
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to NOKPROBE_SYMBOL Pratyush Anand <panand@redhat.com> - 2016-08-02 09:00 +0200
  Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to  NOKPROBE_SYMBOL Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-02 18:00 +0200
    Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to  NOKPROBE_SYMBOL Pratyush Anand <panand@redhat.com> - 2016-08-03 06:30 +0200
      Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to NOKPROBE_SYMBOL Pratyush Anand <panand@redhat.com> - 2016-08-03 12:50 +0200
        Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to  NOKPROBE_SYMBOL Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-03 16:50 +0200
  Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to  NOKPROBE_SYMBOL Oleg Nesterov <oleg@redhat.com> - 2016-08-02 22:50 +0200
    Re: [PATCH] uprobe: Add uprobe_pre/post_sstep_notifier to  NOKPROBE_SYMBOL Pratyush Anand <panand@redhat.com> - 2016-08-03 06:20 +0200

csiph-web