Re: [PATCH -tip/urgent 1/2] [BUGFIX]kprobes: set kprobes_all_disarmed earlier to enable re-optimization.

From: Andrew Morton
Date: Thu Feb 05 2015 - 17:49:59 EST


On Fri, 23 Jan 2015 15:42:24 +0900 Masami Hiramatsu <masami.hiramatsu.pt@xxxxxxxxxxx> wrote:

> From: Wang Nan <wangnan0@xxxxxxxxxx>
>
> In original code, the probed instruction doesn't get optimized after
>
> echo 0 > /sys/kernel/debug/kprobes/enabled
> echo 1 > /sys/kernel/debug/kprobes/enabled
>
> This is because original code checks kprobes_all_disarmed in
> optimize_kprobe(), but this flag is turned off after calling that
> function. Therefore, optimize_kprobe() will see
> kprobes_all_disarmed == true and doesn't do the optimization.
>
> This patch simply turns off kprobes_all_disarmed earlier to enable
> optimization.
>
> Signed-off-by: Wang Nan <wangnan0@xxxxxxxxxx>
> Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@xxxxxxxxxxx>

You sent the patches, so this should have been "Signed-off-by:".
Documentation/SubmittingPatches section 12 has the details.

I have made that change to my copy of these two patches.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/