diff options
author | Michael Ellerman <mpe@ellerman.id.au> | 2014-09-24 16:08:33 +1000 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2014-09-26 11:11:12 +0200 |
commit | e8ac6ea8a42d6506345056c6de3e81c97d3f20af (patch) | |
tree | 82bd3e744fd2ecd0f95fbee3b40fcf061bb8fdef | |
parent | 2d69049ab7028ac656a8545a068cc10a663a9805 (diff) | |
download | lwn-e8ac6ea8a42d6506345056c6de3e81c97d3f20af.tar.gz lwn-e8ac6ea8a42d6506345056c6de3e81c97d3f20af.zip |
kprobes: update jprobe_example.c for do_fork() change
In commit e80d666 "flagday: kill pt_regs argument of do_fork()", the
arguments to do_fork() changed.
The example code in jprobe_example.c was not updated to match, so the
arguments inside the jprobe handler do not match reality.
Fix it by updating the arguments to match do_fork(). While we're at it
use pr_info() for brevity, and print stack_start as well for interest.
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
-rw-r--r-- | samples/kprobes/jprobe_example.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/samples/kprobes/jprobe_example.c b/samples/kprobes/jprobe_example.c index b7541355b92b..9119ac6a8270 100644 --- a/samples/kprobes/jprobe_example.c +++ b/samples/kprobes/jprobe_example.c @@ -24,12 +24,11 @@ /* Proxy routine having the same arguments as actual do_fork() routine */ static long jdo_fork(unsigned long clone_flags, unsigned long stack_start, - struct pt_regs *regs, unsigned long stack_size, - int __user *parent_tidptr, int __user *child_tidptr) + unsigned long stack_size, int __user *parent_tidptr, + int __user *child_tidptr) { - printk(KERN_INFO "jprobe: clone_flags = 0x%lx, stack_size = 0x%lx," - " regs = 0x%p\n", - clone_flags, stack_size, regs); + pr_info("jprobe: clone_flags = 0x%lx, stack_start = 0x%lx " + "stack_size = 0x%lx\n", clone_flags, stack_start, stack_size); /* Always end with a call to jprobe_return(). */ jprobe_return(); |