diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2019-08-09 10:53:39 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2019-08-09 10:53:39 -0400 |
commit | 0e1c438c44dd9cde56effb44c5f1cfeda72e108d (patch) | |
tree | fa3492d4d7d8b7444e5d8ebe6c78210826333e4b /tools/perf/builtin-stat.c | |
parent | c096397c78f766db972f923433031f2dec01cae0 (diff) | |
parent | cdb2d3ee0436d74fa9092f2df46aaa6f9e03c969 (diff) |
Merge tag 'kvmarm-fixes-for-5.3' of git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm into HEAD
KVM/arm fixes for 5.3
- A bunch of switch/case fall-through annotation, fixing one actual bug
- Fix PMU reset bug
- Add missing exception class debug strings
Diffstat (limited to 'tools/perf/builtin-stat.c')
-rw-r--r-- | tools/perf/builtin-stat.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c index e5e19b461061..b55a534b4de0 100644 --- a/tools/perf/builtin-stat.c +++ b/tools/perf/builtin-stat.c | |||
@@ -43,7 +43,6 @@ | |||
43 | #include "perf.h" | 43 | #include "perf.h" |
44 | #include "builtin.h" | 44 | #include "builtin.h" |
45 | #include "util/cgroup.h" | 45 | #include "util/cgroup.h" |
46 | #include "util/util.h" | ||
47 | #include <subcmd/parse-options.h> | 46 | #include <subcmd/parse-options.h> |
48 | #include "util/parse-events.h" | 47 | #include "util/parse-events.h" |
49 | #include "util/pmu.h" | 48 | #include "util/pmu.h" |
@@ -67,6 +66,7 @@ | |||
67 | #include "asm/bug.h" | 66 | #include "asm/bug.h" |
68 | 67 | ||
69 | #include <linux/time64.h> | 68 | #include <linux/time64.h> |
69 | #include <linux/zalloc.h> | ||
70 | #include <api/fs/fs.h> | 70 | #include <api/fs/fs.h> |
71 | #include <errno.h> | 71 | #include <errno.h> |
72 | #include <signal.h> | 72 | #include <signal.h> |
@@ -1349,8 +1349,8 @@ static int add_default_attributes(void) | |||
1349 | fprintf(stderr, | 1349 | fprintf(stderr, |
1350 | "Cannot set up top down events %s: %d\n", | 1350 | "Cannot set up top down events %s: %d\n", |
1351 | str, err); | 1351 | str, err); |
1352 | free(str); | ||
1353 | parse_events_print_error(&errinfo, str); | 1352 | parse_events_print_error(&errinfo, str); |
1353 | free(str); | ||
1354 | return -1; | 1354 | return -1; |
1355 | } | 1355 | } |
1356 | } else { | 1356 | } else { |
@@ -1586,7 +1586,7 @@ static void runtime_stat_delete(struct perf_stat_config *config) | |||
1586 | for (i = 0; i < config->stats_num; i++) | 1586 | for (i = 0; i < config->stats_num; i++) |
1587 | runtime_stat__exit(&config->stats[i]); | 1587 | runtime_stat__exit(&config->stats[i]); |
1588 | 1588 | ||
1589 | free(config->stats); | 1589 | zfree(&config->stats); |
1590 | } | 1590 | } |
1591 | 1591 | ||
1592 | static const char * const stat_report_usage[] = { | 1592 | static const char * const stat_report_usage[] = { |
@@ -2003,7 +2003,7 @@ int cmd_stat(int argc, const char **argv) | |||
2003 | perf_stat__exit_aggr_mode(); | 2003 | perf_stat__exit_aggr_mode(); |
2004 | perf_evlist__free_stats(evsel_list); | 2004 | perf_evlist__free_stats(evsel_list); |
2005 | out: | 2005 | out: |
2006 | free(stat_config.walltime_run); | 2006 | zfree(&stat_config.walltime_run); |
2007 | 2007 | ||
2008 | if (smi_cost && smi_reset) | 2008 | if (smi_cost && smi_reset) |
2009 | sysfs__write_int(FREEZE_ON_SMI_PATH, 0); | 2009 | sysfs__write_int(FREEZE_ON_SMI_PATH, 0); |