diff options
author | Srikar Dronamraju <srikar@linux.vnet.ibm.com> | 2015-06-24 07:10:04 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-06-25 11:28:35 -0400 |
commit | 2b42b09b88c831ba4da2d669581dde371c38c2af (patch) | |
tree | b08359e8f41722b226bd0fe7a681af30c30cf951 /tools | |
parent | 0f02adaa49af23ab9fa05375f4a237da6014401b (diff) |
perf bench numa: Fix to show proper convergence stats
With commit: e1e455f4f4d3 (perf tools: Work around lack of sched_getcpu
in glibc < 2.6), perf_bench numa mem with -c or -m option is not able to
correctly calculate convergence.
With the above commit, sched_getcpu always seems to return -1. The
intention of commit e1e455f was to add a sched_getcpu in glibc < 2.6.
Hence keep the sched_getcpu definition under an ifdef.
This regression happened occurred between v4.0 and v4.1
Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Acked-by: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Vinson Lee <vlee@twitter.com>
Fixes: e1e455f4f4d3 ("perf tools: Work around lack of sched_getcpu in glibc < 2.6")
Link: http://lkml.kernel.org/r/20150624111004.GA5220@linux.vnet.ibm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/perf/util/cloexec.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/perf/util/cloexec.c b/tools/perf/util/cloexec.c index 85b523885f9d..2babddaa2481 100644 --- a/tools/perf/util/cloexec.c +++ b/tools/perf/util/cloexec.c | |||
@@ -7,11 +7,15 @@ | |||
7 | 7 | ||
8 | static unsigned long flag = PERF_FLAG_FD_CLOEXEC; | 8 | static unsigned long flag = PERF_FLAG_FD_CLOEXEC; |
9 | 9 | ||
10 | #ifdef __GLIBC_PREREQ | ||
11 | #if !__GLIBC_PREREQ(2, 6) | ||
10 | int __weak sched_getcpu(void) | 12 | int __weak sched_getcpu(void) |
11 | { | 13 | { |
12 | errno = ENOSYS; | 14 | errno = ENOSYS; |
13 | return -1; | 15 | return -1; |
14 | } | 16 | } |
17 | #endif | ||
18 | #endif | ||
15 | 19 | ||
16 | static int perf_flag_probe(void) | 20 | static int perf_flag_probe(void) |
17 | { | 21 | { |