Re: [PATCH v4 -tip 0/3] x86/kprobes: bugfixes and split optprobes

From: Ingo Molnar
Date: Tue Mar 06 2012 - 04:11:12 EST



* Masami Hiramatsu <masami.hiramatsu.pt@xxxxxxxxxxx> wrote:

> Hi,
>
> Here are the patches for fixing bugs in kprobes on x86.
>
> - Both bugs are related to the optprobe.
> - First bugfix is optprobe to recover instructions when
> thoes are modified with jump.
> - Second bugfix is kprobes to recover it too.
>
> This series also includes a patch which split optprobes
> parts out from kprobes x86 implementation.
>
> Ingo, I've added the splitting patch according your
> advice :)
>
> Thank you,
>
> ---
>
> Masami Hiramatsu (3):
> x86/kprobes: Split out optprobe related code to kprobes-opt.c
> [BUGFIX] x86/kprobes: Fix a bug which can modify kernel code
> [BUGFIX] x86/kprobes: Fix to recover instructions on optimized path
>
>
> arch/x86/kernel/Makefile | 1
> arch/x86/kernel/kprobes-common.h | 106 ++++++
> arch/x86/kernel/kprobes-opt.c | 516 +++++++++++++++++++++++++++++++
> arch/x86/kernel/kprobes.c | 633 ++++----------------------------------
> 4 files changed, 684 insertions(+), 572 deletions(-)
> create mode 100644 arch/x86/kernel/kprobes-common.h
> create mode 100644 arch/x86/kernel/kprobes-opt.c

Thanks, this looks very nice. I've applied them to tip:perf/core
and will push them out later today, assuming there are no test
failures.

Thanks,

Ingo
--
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/