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>
