Commit 9c355917 authored by Balbir Singh's avatar Balbir Singh Committed by Michael Ellerman

powerpc/tracing: Allow tracing of mmap syscalls

Currently sys_mmap() and sys_mmap2() (32-bit only), are not visible to the
syscall tracing machinery. This means users are not able to see the execution of
mmap() syscalls using the syscall tracer.

Fix that by using SYSCALL_DEFINE6 for sys_mmap() and sys_mmap2() so that the
meta-data associated with these syscalls is visible to the syscall tracer.

A side-effect of this change is that the return type has changed from unsigned
long to long. However this should have no effect, the only code in the kernel
which uses the result of these syscalls is in the syscall return path, which is
written in asm and treats the result as unsigned regardless.

Example output:
  cat-3399  [001] ....   196.542410: sys_mmap(addr: 7fff922a0000, len: 20000, prot: 3, flags: 812, fd: 3, offset: 1b0000)
  cat-3399  [001] ....   196.542443: sys_mmap -> 0x7fff922a0000
  cat-3399  [001] ....   196.542668: sys_munmap(addr: 7fff922c0000, len: 6d2c)
  cat-3399  [001] ....   196.542677: sys_munmap -> 0x0
Signed-off-by: default avatarBalbir Singh <bsingharora@gmail.com>
[mpe: Massage change log, add detail on return type change]
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 03dfee6d
...@@ -8,10 +8,10 @@ ...@@ -8,10 +8,10 @@
struct rtas_args; struct rtas_args;
asmlinkage unsigned long sys_mmap(unsigned long addr, size_t len, asmlinkage long sys_mmap(unsigned long addr, size_t len,
unsigned long prot, unsigned long flags, unsigned long prot, unsigned long flags,
unsigned long fd, off_t offset); unsigned long fd, off_t offset);
asmlinkage unsigned long sys_mmap2(unsigned long addr, size_t len, asmlinkage long sys_mmap2(unsigned long addr, size_t len,
unsigned long prot, unsigned long flags, unsigned long prot, unsigned long flags,
unsigned long fd, unsigned long pgoff); unsigned long fd, unsigned long pgoff);
asmlinkage long ppc64_personality(unsigned long personality); asmlinkage long ppc64_personality(unsigned long personality);
......
...@@ -42,11 +42,11 @@ ...@@ -42,11 +42,11 @@
#include <asm/unistd.h> #include <asm/unistd.h>
#include <asm/asm-prototypes.h> #include <asm/asm-prototypes.h>
static inline unsigned long do_mmap2(unsigned long addr, size_t len, static inline long do_mmap2(unsigned long addr, size_t len,
unsigned long prot, unsigned long flags, unsigned long prot, unsigned long flags,
unsigned long fd, unsigned long off, int shift) unsigned long fd, unsigned long off, int shift)
{ {
unsigned long ret = -EINVAL; long ret = -EINVAL;
if (!arch_validate_prot(prot)) if (!arch_validate_prot(prot))
goto out; goto out;
...@@ -62,16 +62,16 @@ static inline unsigned long do_mmap2(unsigned long addr, size_t len, ...@@ -62,16 +62,16 @@ static inline unsigned long do_mmap2(unsigned long addr, size_t len,
return ret; return ret;
} }
unsigned long sys_mmap2(unsigned long addr, size_t len, SYSCALL_DEFINE6(mmap2, unsigned long, addr, size_t, len,
unsigned long prot, unsigned long flags, unsigned long, prot, unsigned long, flags,
unsigned long fd, unsigned long pgoff) unsigned long, fd, unsigned long, pgoff)
{ {
return do_mmap2(addr, len, prot, flags, fd, pgoff, PAGE_SHIFT-12); return do_mmap2(addr, len, prot, flags, fd, pgoff, PAGE_SHIFT-12);
} }
unsigned long sys_mmap(unsigned long addr, size_t len, SYSCALL_DEFINE6(mmap, unsigned long, addr, size_t, len,
unsigned long prot, unsigned long flags, unsigned long, prot, unsigned long, flags,
unsigned long fd, off_t offset) unsigned long, fd, off_t, offset)
{ {
return do_mmap2(addr, len, prot, flags, fd, offset, PAGE_SHIFT); return do_mmap2(addr, len, prot, flags, fd, offset, PAGE_SHIFT);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment