diff options
author | Ingo Molnar <mingo@kernel.org> | 2015-07-02 10:48:16 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2015-07-02 10:48:16 +0200 |
commit | b9df84fd7c05cc300d6d14f022b8a00773ebcf8c (patch) | |
tree | b7a3346416c6d1229ff79ade4fe8646c95125dd8 /tools/perf/builtin-inject.c | |
parent | 93472aff802fd7b61f2209335207e9bd793012f7 (diff) | |
parent | 5ef7bbb09f7b91ef06524c72e1ab1fc48e0d6682 (diff) | |
download | lwn-b9df84fd7c05cc300d6d14f022b8a00773ebcf8c.tar.gz lwn-b9df84fd7c05cc300d6d14f022b8a00773ebcf8c.zip |
Merge tag 'perf-core-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/urgent
Pull perf/core fixes from Arnaldo Carvalho de Melo:
Build fixes:
- Create config.detected into OUTPUT directory, fixing parallel
builds sharing the same source directory (Aaro Kiskinen)
- Allow to specify custom linker command, fixing some MIPS64
builds. (Aaro Kiskinen)
Infrastructure fixes:
- Add missing break for PERF_RECORD_ITRACE_START, which caused those events
samples to be parsed as well as PERF_RECORD_LOST_SAMPLES. ITRACE_START only
appears when Intel PT or BTS are present, so (Jiri Olsa)
- Call the perf_session destructor when bailing out in the inject, kmem, report,
kvm and mem tools (Taeung Song)
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'tools/perf/builtin-inject.c')
-rw-r--r-- | tools/perf/builtin-inject.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/perf/builtin-inject.c b/tools/perf/builtin-inject.c index 52ec66b23607..01b06492bd6a 100644 --- a/tools/perf/builtin-inject.c +++ b/tools/perf/builtin-inject.c @@ -630,12 +630,13 @@ int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused) if (inject.session == NULL) return -1; - if (symbol__init(&inject.session->header.env) < 0) - return -1; + ret = symbol__init(&inject.session->header.env); + if (ret < 0) + goto out_delete; ret = __cmd_inject(&inject); +out_delete: perf_session__delete(inject.session); - return ret; } |