Re: [PATCH v13 04/10] arm64: Blacklist non-kprobe-able symbol

From: Masami Hiramatsu
Date: Fri Jun 03 2016 - 23:40:52 EST


Hi David,

On Thu, 2 Jun 2016 23:26:18 -0400
David Long <dave.long@xxxxxxxxxx> wrote:

> From: Pratyush Anand <panand@xxxxxxxxxx>
>
> Add all function symbols which are called from do_debug_exception under
> NOKPROBE_SYMBOL, as they can not kprobed.

I see, but this patch should be applied after kprobes
are implemented on arm64.

And also, I have a comment below.

> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c
> index 5954881..4359ca8 100644
> --- a/arch/arm64/mm/fault.c
> +++ b/arch/arm64/mm/fault.c
> @@ -563,6 +563,7 @@ asmlinkage void __exception do_sp_pc_abort(unsigned long addr,
> info.si_addr = (void __user *)addr;
> arm64_notify_die("Oops - SP/PC alignment exception", regs, &info, esr);
> }
> +NOKPROBE_SYMBOL(do_debug_exception)

This seems at wrong place. Please correct it in this patch.

Thank you,

--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>