diff options
author | Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> | 2014-08-13 22:22:47 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2014-08-15 12:07:47 -0400 |
commit | fb74fbda42dc5bcbd9bae5d75bfb6755948db21d (patch) | |
tree | 018b17d1e8501599af3b2ab966cccdb790fca084 | |
parent | ba3dfff8ad2d98df0c8116faaeb281c93e161636 (diff) |
perf sched: Use strerror_r instead of strerror
Use strerror_r instead of strerror in error message for thread-safety.
Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Naohiro Aota <naota@elisp.net>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20140814022247.3545.4564.stgit@kbuild-fedora.novalocal
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/builtin-sched.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c index f5874a27b346..9c9287fbf8e9 100644 --- a/tools/perf/builtin-sched.c +++ b/tools/perf/builtin-sched.c | |||
@@ -428,6 +428,7 @@ static u64 get_cpu_usage_nsec_parent(void) | |||
428 | static int self_open_counters(void) | 428 | static int self_open_counters(void) |
429 | { | 429 | { |
430 | struct perf_event_attr attr; | 430 | struct perf_event_attr attr; |
431 | char sbuf[STRERR_BUFSIZE]; | ||
431 | int fd; | 432 | int fd; |
432 | 433 | ||
433 | memset(&attr, 0, sizeof(attr)); | 434 | memset(&attr, 0, sizeof(attr)); |
@@ -440,7 +441,8 @@ static int self_open_counters(void) | |||
440 | 441 | ||
441 | if (fd < 0) | 442 | if (fd < 0) |
442 | pr_err("Error: sys_perf_event_open() syscall returned " | 443 | pr_err("Error: sys_perf_event_open() syscall returned " |
443 | "with %d (%s)\n", fd, strerror(errno)); | 444 | "with %d (%s)\n", fd, |
445 | strerror_r(errno, sbuf, sizeof(sbuf))); | ||
444 | return fd; | 446 | return fd; |
445 | } | 447 | } |
446 | 448 | ||