Re: [PATCH 05/12] kprobes: declare __{start,stop}_kprobe_blacklist as external array

From: Masami Hiramatsu
Date: Mon Oct 17 2016 - 02:44:13 EST


Could you please add the patch description for the patch?
And send me the whole series of the patches.

Thanks,

On Sun, 16 Oct 2016 17:16:09 +0200
Vegard Nossum <vegard.nossum@xxxxxxxxxx> wrote:

> Cc: Ananth N Mavinakayanahalli <ananth@xxxxxxxxxxxxxxxxxx>
> Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@xxxxxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> Signed-off-by: Vegard Nossum <vegard.nossum@xxxxxxxxxx>
> ---
> kernel/kprobes.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/kernel/kprobes.c b/kernel/kprobes.c
> index d630954..f163f74 100644
> --- a/kernel/kprobes.c
> +++ b/kernel/kprobes.c
> @@ -31,6 +31,7 @@
> * <jkenisto@xxxxxxxxxx> and Prasanna S Panchamukhi
> * <prasanna@xxxxxxxxxx> added function-return probes.
> */
> +#include <linux/extarray.h>
> #include <linux/kprobes.h>
> #include <linux/hash.h>
> #include <linux/init.h>
> @@ -2126,8 +2127,7 @@ static struct notifier_block kprobe_module_nb = {
> };
>
> /* Markers of _kprobe_blacklist section */
> -extern unsigned long __start_kprobe_blacklist[];
> -extern unsigned long __stop_kprobe_blacklist[];
> +DECLARE_EXTARRAY(unsigned long, kprobe_blacklist);
>
> static int __init init_kprobes(void)
> {
> @@ -2141,8 +2141,8 @@ static int __init init_kprobes(void)
> raw_spin_lock_init(&(kretprobe_table_locks[i].lock));
> }
>
> - err = populate_kprobe_blacklist(__start_kprobe_blacklist,
> - __stop_kprobe_blacklist);
> + err = populate_kprobe_blacklist(ext_start(kprobe_blacklist),
> + ext_end(kprobe_blacklist));
> if (err) {
> pr_err("kprobes: failed to populate blacklist: %d\n", err);
> pr_err("Please take care of using kprobes.\n");
> --
> 2.10.0.479.g221bd91
>


--
Masami Hiramatsu