Re: [PATCH v2 08/11] kprobes: switch to kernel_clone()

From: Masami Hiramatsu
Date: Sun Sep 06 2020 - 21:13:00 EST


Hi,

On Wed, 19 Aug 2020 12:46:52 +0200
Christian Brauner <christian.brauner@xxxxxxxxxx> wrote:

> The old _do_fork() helper is removed in favor of the new kernel_clone() helper.
> The latter adheres to naming conventions for kernel internal syscall helpers.

This looks good to me.

Acked-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>

Thank you!

>
> Cc: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
> Cc: Alexandre Chartre <alexandre.chartre@xxxxxxxxxx>
> Cc: Jonathan Corbet <corbet@xxxxxxx>
> Cc: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> Signed-off-by: Christian Brauner <christian.brauner@xxxxxxxxxx>
> ---
> /* v2 */
> unchanged
> ---
> samples/kprobes/kprobe_example.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/samples/kprobes/kprobe_example.c b/samples/kprobes/kprobe_example.c
> index 240f2435ce6f..a02f53836ee1 100644
> --- a/samples/kprobes/kprobe_example.c
> +++ b/samples/kprobes/kprobe_example.c
> @@ -2,13 +2,13 @@
> /*
> * NOTE: This example is works on x86 and powerpc.
> * Here's a sample kernel module showing the use of kprobes to dump a
> - * stack trace and selected registers when _do_fork() is called.
> + * stack trace and selected registers when kernel_clone() is called.
> *
> * For more information on theory of operation of kprobes, see
> * Documentation/staging/kprobes.rst
> *
> * You will see the trace data in /var/log/messages and on the console
> - * whenever _do_fork() is invoked to create a new process.
> + * whenever kernel_clone() is invoked to create a new process.
> */
>
> #include <linux/kernel.h>
> @@ -16,7 +16,7 @@
> #include <linux/kprobes.h>
>
> #define MAX_SYMBOL_LEN 64
> -static char symbol[MAX_SYMBOL_LEN] = "_do_fork";
> +static char symbol[MAX_SYMBOL_LEN] = "kernel_clone";
> module_param_string(symbol, symbol, sizeof(symbol), 0644);
>
> /* For each probe you need to allocate a kprobe structure */
> --
> 2.28.0
>


--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>