diff options
author | Ingo Molnar <mingo@kernel.org> | 2012-09-26 02:57:22 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2012-09-26 02:57:22 -0400 |
commit | a91f408e4435d4f572392295ee47756b6f5011c7 (patch) | |
tree | 298177b72d51218a69ffd92a940029c13a3bdde6 /tools/perf | |
parent | f74eb728687afbf239abe2ea66921583db4eaa4b (diff) | |
parent | f1b2256d66fe8d613b9afcc1c16079362f9fc05c (diff) |
Merge tag 'perf-core-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core
Pull perf/core fixes from Arnaldo Carvalho de Melo:
* The new perf_evsel__tp_sched_test 'perf test' broke the build by setting the
'ret' variable but not using it, caught by newer gcc
-Werror=unused-but-set-variable, fix from Namhyung Kim.
* pevent_parse_event should return a proper PEVENT_ERRNO__ and call
pevent_free_format on its failure path, fixes from Namhyung Kim.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/builtin-test.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/builtin-test.c b/tools/perf/builtin-test.c index 32caf13cfe01..78b47a75a7c9 100644 --- a/tools/perf/builtin-test.c +++ b/tools/perf/builtin-test.c | |||
@@ -1233,7 +1233,7 @@ static int perf_evsel__test_field(struct perf_evsel *evsel, const char *name, | |||
1233 | ret = -1; | 1233 | ret = -1; |
1234 | } | 1234 | } |
1235 | 1235 | ||
1236 | return 0; | 1236 | return ret; |
1237 | } | 1237 | } |
1238 | 1238 | ||
1239 | static int perf_evsel__tp_sched_test(void) | 1239 | static int perf_evsel__tp_sched_test(void) |
@@ -1286,7 +1286,7 @@ static int perf_evsel__tp_sched_test(void) | |||
1286 | if (perf_evsel__test_field(evsel, "target_cpu", 4, true)) | 1286 | if (perf_evsel__test_field(evsel, "target_cpu", 4, true)) |
1287 | ret = -1; | 1287 | ret = -1; |
1288 | 1288 | ||
1289 | return 0; | 1289 | return ret; |
1290 | } | 1290 | } |
1291 | 1291 | ||
1292 | static struct test { | 1292 | static struct test { |