[tip:perf/urgent] perf trace: Use the ptr->name beautifier as default for "filename" args

[tip-bot for Arnaldo Carvalho de Melo]

Commit-ID:  12f3ca4fc8e27aa602c9c3c717d755b1e8f7fd47
Gitweb: http://git.kernel.org/tip/12f3ca4fc8e27aa602c9c3c717d755b1e8f7fd47
Author: Arnaldo Carvalho de Melo 
AuthorDate: Mon, 23 May 2016 16:37:55 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 23 May 2016 16:41:00 -0300

perf trace: Use the ptr->name beautifier as default for "filename" args

Auto-attach the ptr->name beautifier to syscall args "filename", "path"
and "pathname" if they are of type "const char *".

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Milian Wolff 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-jxii4qmcgoppftv0zdvml...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/builtin-trace.c | 120 -
 1 file changed, 43 insertions(+), 77 deletions(-)

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index 181f69c..5c50fe7 100644
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -576,44 +576,36 @@ static struct syscall_fmt {
bool   hexret;
 } syscall_fmts[] = {
{ .name = "access", .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */
-[1] = SCA_ACCMODE,  /* mode */ }, },
+ .arg_scnprintf = { [1] = SCA_ACCMODE,  /* mode */ }, },
{ .name = "arch_prctl", .errmsg = true, .alias = "prctl", },
{ .name = "bpf",.errmsg = true, STRARRAY(0, cmd, bpf_cmd), 
},
{ .name = "brk",.hexret = true,
  .arg_scnprintf = { [0] = SCA_HEX, /* brk */ }, },
-   { .name = "chdir",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
-   { .name = "chmod",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
-   { .name = "chroot", .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
+   { .name = "chdir",  .errmsg = true, },
+   { .name = "chmod",  .errmsg = true, },
+   { .name = "chroot", .errmsg = true, },
{ .name = "clock_gettime",  .errmsg = true, STRARRAY(0, clk_id, 
clockid), },
{ .name = "clone",  .errpid = true, },
{ .name = "close",  .errmsg = true,
  .arg_scnprintf = { [0] = SCA_CLOSE_FD, /* fd */ }, },
{ .name = "connect",.errmsg = true, },
-   { .name = "creat",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* pathname */ }, },
+   { .name = "creat",  .errmsg = true, },
{ .name = "dup",.errmsg = true, },
{ .name = "dup2",   .errmsg = true, },
{ .name = "dup3",   .errmsg = true, },
{ .name = "epoll_ctl",  .errmsg = true, STRARRAY(1, op, 
epoll_ctl_ops), },
{ .name = "eventfd2",   .errmsg = true,
  .arg_scnprintf = { [1] = SCA_EFD_FLAGS, /* flags */ }, },
-   { .name = "faccessat",  .errmsg = true,
- .arg_scnprintf = { [1] = SCA_FILENAME, /* filename */ }, },
+   { .name = "faccessat",  .errmsg = true, },
{ .name = "fadvise64",  .errmsg = true, },
{ .name = "fallocate",  .errmsg = true, },
{ .name = "fchdir", .errmsg = true, },
{ .name = "fchmod", .errmsg = true, },
{ .name = "fchmodat",   .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FDAT, /* fd */
-[1] = SCA_FILENAME, /* filename */ }, },
+ .arg_scnprintf = { [0] = SCA_FDAT, /* fd */ }, },
{ .name = "fchown", .errmsg = true, },
{ .name = "fchownat",   .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FDAT, /* fd */
-[1] = SCA_FILENAME, /* filename */ }, },
+ .arg_scnprintf = { [0] = SCA_FDAT, /* fd */ }, },
{ .name = "fcntl",  .errmsg = true,
  .arg_scnprintf = { [1] = SCA_STRARRAY, /* cmd */ },
  .arg_parm  = { [1] = __fcntl_cmds, /* cmd */ }, },
@@ -622,16 +614,14 @@ static struct syscall_fmt {
  .arg_scnprintf = { [1] = SCA_FLOCK, /* cmd */ }, },
{ .name = "fsetxattr",  .errmsg = true, },
{ .name = "fstat",  .errmsg = true, .alias = "newfstat", },
-   { .name = "fstatat",.errmsg = true, .alias = "newfstatat",
- .arg_scnprintf = { [1] = SCA_FILENAME, /* filename */ }, },
+   { .name = "fstatat",.errmsg = true, .alias = "newfstatat", },
{ .name = "fstatfs",.errmsg = true, },
{ .name = "fsync",.errmsg = true, },
{ .name = "ftruncate", .errmsg = true, },
{ .name = "futex",  .errmsg = true,
  

[tip:perf/urgent] perf trace: Use the ptr->name beautifier as default for "filename" args

[tip-bot for Arnaldo Carvalho de Melo]

Commit-ID:  12f3ca4fc8e27aa602c9c3c717d755b1e8f7fd47
Gitweb: http://git.kernel.org/tip/12f3ca4fc8e27aa602c9c3c717d755b1e8f7fd47
Author: Arnaldo Carvalho de Melo 
AuthorDate: Mon, 23 May 2016 16:37:55 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 23 May 2016 16:41:00 -0300

perf trace: Use the ptr->name beautifier as default for "filename" args

Auto-attach the ptr->name beautifier to syscall args "filename", "path"
and "pathname" if they are of type "const char *".

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Milian Wolff 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-jxii4qmcgoppftv0zdvml...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/builtin-trace.c | 120 -
 1 file changed, 43 insertions(+), 77 deletions(-)

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index 181f69c..5c50fe7 100644
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -576,44 +576,36 @@ static struct syscall_fmt {
bool   hexret;
 } syscall_fmts[] = {
{ .name = "access", .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */
-[1] = SCA_ACCMODE,  /* mode */ }, },
+ .arg_scnprintf = { [1] = SCA_ACCMODE,  /* mode */ }, },
{ .name = "arch_prctl", .errmsg = true, .alias = "prctl", },
{ .name = "bpf",.errmsg = true, STRARRAY(0, cmd, bpf_cmd), 
},
{ .name = "brk",.hexret = true,
  .arg_scnprintf = { [0] = SCA_HEX, /* brk */ }, },
-   { .name = "chdir",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
-   { .name = "chmod",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
-   { .name = "chroot", .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* filename */ }, },
+   { .name = "chdir",  .errmsg = true, },
+   { .name = "chmod",  .errmsg = true, },
+   { .name = "chroot", .errmsg = true, },
{ .name = "clock_gettime",  .errmsg = true, STRARRAY(0, clk_id, 
clockid), },
{ .name = "clone",  .errpid = true, },
{ .name = "close",  .errmsg = true,
  .arg_scnprintf = { [0] = SCA_CLOSE_FD, /* fd */ }, },
{ .name = "connect",.errmsg = true, },
-   { .name = "creat",  .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FILENAME, /* pathname */ }, },
+   { .name = "creat",  .errmsg = true, },
{ .name = "dup",.errmsg = true, },
{ .name = "dup2",   .errmsg = true, },
{ .name = "dup3",   .errmsg = true, },
{ .name = "epoll_ctl",  .errmsg = true, STRARRAY(1, op, 
epoll_ctl_ops), },
{ .name = "eventfd2",   .errmsg = true,
  .arg_scnprintf = { [1] = SCA_EFD_FLAGS, /* flags */ }, },
-   { .name = "faccessat",  .errmsg = true,
- .arg_scnprintf = { [1] = SCA_FILENAME, /* filename */ }, },
+   { .name = "faccessat",  .errmsg = true, },
{ .name = "fadvise64",  .errmsg = true, },
{ .name = "fallocate",  .errmsg = true, },
{ .name = "fchdir", .errmsg = true, },
{ .name = "fchmod", .errmsg = true, },
{ .name = "fchmodat",   .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FDAT, /* fd */
-[1] = SCA_FILENAME, /* filename */ }, },
+ .arg_scnprintf = { [0] = SCA_FDAT, /* fd */ }, },
{ .name = "fchown", .errmsg = true, },
{ .name = "fchownat",   .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FDAT, /* fd */
-[1] = SCA_FILENAME, /* filename */ }, },
+ .arg_scnprintf = { [0] = SCA_FDAT, /* fd */ }, },
{ .name = "fcntl",  .errmsg = true,
  .arg_scnprintf = { [1] = SCA_STRARRAY, /* cmd */ },
  .arg_parm  = { [1] = __fcntl_cmds, /* cmd */ }, },
@@ -622,16 +614,14 @@ static struct syscall_fmt {
  .arg_scnprintf = { [1] = SCA_FLOCK, /* cmd */ }, },
{ .name = "fsetxattr",  .errmsg = true, },
{ .name = "fstat",  .errmsg = true, .alias = "newfstat", },
-   { .name = "fstatat",.errmsg = true, .alias = "newfstatat",
- .arg_scnprintf = { [1] = SCA_FILENAME, /* filename */ }, },
+   { .name = "fstatat",.errmsg = true, .alias = "newfstatat", },
{ .name = "fstatfs",.errmsg = true, },
{ .name = "fsync",.errmsg = true, },
{ .name = "ftruncate", .errmsg = true, },
{ .name = "futex",  .errmsg = true,
  .arg_scnprintf = { [1] = SCA_FUTEX_OP, /* op */ }, },
{ .name = "futimesat", .errmsg = true,
- .arg_scnprintf = { [0] = SCA_FDAT, /* fd */
-