[PATCH] perf stat: tell user about unsupported events in the list

From: David Ahern
Date: Fri Apr 29 2011 - 18:04:26 EST


Similar to perf-record, tell user about unsupported events
that will not be counted if invoked in verbose mode.

e.g.,
perf stat -e dTLB-prefetch-misses -v -- sleep 1
dTLB-prefetch-misses event is not supported by the kernel.
dTLB-prefetch-misses: 0 0 0

Performance counter stats for 'sleep 1':

<not counted> dTLB-prefetch-misses

1.001884783 seconds time elapsed

Signed-off-by: David Ahern <dsahern@xxxxxxxxx>
---
tools/perf/builtin-stat.c | 6 +++++-
1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c
index c8b535b..602c3c9 100644
--- a/tools/perf/builtin-stat.c
+++ b/tools/perf/builtin-stat.c
@@ -377,8 +377,12 @@ static int run_perf_stat(int argc __used, const char **argv)

list_for_each_entry(counter, &evsel_list->entries, node) {
if (create_perf_stat_counter(counter) < 0) {
- if (errno == EINVAL || errno == ENOSYS || errno == ENOENT)
+ if (errno == EINVAL || errno == ENOSYS || errno == ENOENT) {
+ if (verbose)
+ ui__warning("%s event is not supported by the kernel.\n",
+ event_name(counter));
continue;
+ }

if (errno == EPERM || errno == EACCES) {
error("You may not have permission to collect %sstats.\n"
--
1.7.4.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/