diff options
| author | Arnaldo Carvalho de Melo <[email protected]> | 2017-06-27 14:22:31 +0000 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2017-06-27 14:22:31 +0000 |
| commit | 62d94b00f80b0ecb7fa9eea0539c59e9f82b0fcd (patch) | |
| tree | c950f6bc7b253c0e6005cf282fc0370a85c6171a /tools/perf/builtin-help.c | |
| parent | perf tools: Remove warning() (diff) | |
| download | kernel-62d94b00f80b0ecb7fa9eea0539c59e9f82b0fcd.tar.gz kernel-62d94b00f80b0ecb7fa9eea0539c59e9f82b0fcd.zip | |
perf tools: Replace error() with pr_err()
To consolidate the error reporting facility.
Cc: Adrian Hunter <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Wang Nan <[email protected]>
Link: http://lkml.kernel.org/n/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/builtin-help.c')
| -rw-r--r-- | tools/perf/builtin-help.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/tools/perf/builtin-help.c b/tools/perf/builtin-help.c index 64a10101fc71..530a7f2fa0f3 100644 --- a/tools/perf/builtin-help.c +++ b/tools/perf/builtin-help.c @@ -245,8 +245,10 @@ static int add_man_viewer_info(const char *var, const char *value) const char *name = var + 4; const char *subkey = strrchr(name, '.'); - if (!subkey) - return error("Config with no key for man viewer: %s", name); + if (!subkey) { + pr_err("Config with no key for man viewer: %s", name); + return -1; + } if (!strcmp(subkey, ".path")) { if (!value) @@ -330,7 +332,7 @@ static void setup_man_path(void) setenv("MANPATH", new_path, 1); free(new_path); } else { - error("Unable to setup man path"); + pr_err("Unable to setup man path"); } } |