perf trace beauty fcntl: Beautify F_GETLEASE and F_SETLEASE arg/return

One more looking prettier.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-ytr7idkese8sjtvn5g60130e@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 0ae79636
...@@ -20,6 +20,19 @@ static size_t syscall_arg__scnprintf_fcntl_getfd(char *bf, size_t size, struct s ...@@ -20,6 +20,19 @@ static size_t syscall_arg__scnprintf_fcntl_getfd(char *bf, size_t size, struct s
return fcntl__scnprintf_getfd(arg->val, bf, size); return fcntl__scnprintf_getfd(arg->val, bf, size);
} }
static size_t fcntl__scnprintf_getlease(unsigned long val, char *bf, size_t size)
{
static const char *fcntl_setlease[] = { "RDLCK", "WRLCK", "UNLCK", };
static DEFINE_STRARRAY(fcntl_setlease);
return strarray__scnprintf(&strarray__fcntl_setlease, bf, size, "%x", val);
}
static size_t syscall_arg__scnprintf_fcntl_getlease(char *bf, size_t size, struct syscall_arg *arg)
{
return fcntl__scnprintf_getlease(arg->val, bf, size);
}
size_t syscall_arg__scnprintf_fcntl_cmd(char *bf, size_t size, struct syscall_arg *arg) size_t syscall_arg__scnprintf_fcntl_cmd(char *bf, size_t size, struct syscall_arg *arg)
{ {
if (arg->val == F_GETFL) { if (arg->val == F_GETFL) {
...@@ -38,11 +51,15 @@ size_t syscall_arg__scnprintf_fcntl_cmd(char *bf, size_t size, struct syscall_ar ...@@ -38,11 +51,15 @@ size_t syscall_arg__scnprintf_fcntl_cmd(char *bf, size_t size, struct syscall_ar
syscall_arg__set_ret_scnprintf(arg, syscall_arg__scnprintf_pid); syscall_arg__set_ret_scnprintf(arg, syscall_arg__scnprintf_pid);
goto mask_arg; goto mask_arg;
} }
if (arg->val == F_GETLEASE) {
syscall_arg__set_ret_scnprintf(arg, syscall_arg__scnprintf_fcntl_getlease);
goto mask_arg;
}
/* /*
* Some commands ignore the third fcntl argument, "arg", so mask it * Some commands ignore the third fcntl argument, "arg", so mask it
*/ */
if (arg->val == F_GET_SEALS || if (arg->val == F_GET_SEALS ||
arg->val == F_GETLEASE || arg->val == F_GETSIG) { arg->val == F_GETSIG) {
mask_arg: mask_arg:
arg->mask |= (1 << 2); arg->mask |= (1 << 2);
} }
...@@ -62,6 +79,9 @@ size_t syscall_arg__scnprintf_fcntl_arg(char *bf, size_t size, struct syscall_ar ...@@ -62,6 +79,9 @@ size_t syscall_arg__scnprintf_fcntl_arg(char *bf, size_t size, struct syscall_ar
if (cmd == F_SETOWN) if (cmd == F_SETOWN)
return syscall_arg__scnprintf_pid(bf, size, arg); return syscall_arg__scnprintf_pid(bf, size, arg);
if (cmd == F_SETLEASE)
return fcntl__scnprintf_getlease(arg->val, bf, size);
/* /*
* We still don't grab the contents of pointers on entry or exit, * We still don't grab the contents of pointers on entry or exit,
* so just print them as hex numbers * so just print them as hex numbers
......
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