| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <4A7B68C3.7040706@inria.fr>
Date: Fri, 07 Aug 2009 01:35:31 +0200
From: Brice Goglin <Brice.Goglin@...ia.fr>
To: Ingo Molnar <mingo@...e.hu>
CC: Peter Zijlstra <a.p.zijlstra@...llo.nl>, paulus@...ba.org,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] perf tools: Fix reading of perf.data file header
Ingo Molnar wrote:
> It would be nice to add this as some "perf report -s/--stats" flag,
> to not have to go via -D (which is a 'print debug output' kind of
> ad-hoc thing and subject to format changes in the future).
>
> Would you be interested in sending a patch that adds that flag to
> 'perf report', to print out these statistics entries (if any), in a
> tabular form suitable for your purposes? Below is a past patch to
> builtin-report.c that shows how to add new options.
>
Here's a quick'n'dirty first try. Read events are copied in the
show_stat_event array during process_read_event. And __cmd_report sorts
the array by tid before displaying it.
perf report -S now shows the following after the existing output: (-s is
already used for something else).
It shows things like
# Per-thread statistics:
# PID TID Event Count
16709 16709 cache-misses 82727
16709 16709 cache-references 41238768
16709 16710 cache-misses 6462
16709 16710 cache-references 76119375
or
# Per-thread statistics:
# PID TID Event Count
6268 6268 raw 0x1000001e0 494628
6268 6268 raw 0x1000002e0 209113
6268 6268 raw 0x1000004e0 307215
6268 6268 raw 0x1000008e0 9203221
6268 6269 raw 0x1000001e0 9210788
6268 6269 raw 0x1000002e0 302344
6268 6269 raw 0x1000004e0 198705
6268 6269 raw 0x1000008e0 473471
Obviously, there's some a lot of nice pretty printing to do, but you'll
be able to tell whether the general idea is ok or not.
Brice
Signed-off-by: Brice Goglin <Brice.Goglin@...ia.fr>
Index: linux-2.6.31/tools/perf/builtin-report.c
===================================================================
--- linux-2.6.31.orig/tools/perf/builtin-report.c 2009-08-07 00:42:55.000000000 +0200
+++ linux-2.6.31/tools/perf/builtin-report.c 2009-08-07 01:26:33.000000000 +0200
@@ -24,6 +24,8 @@
#include "util/parse-options.h"
#include "util/parse-events.h"
+#include <stdlib.h>
+
#define SHOW_KERNEL 1
#define SHOW_USER 2
#define SHOW_HV 4
@@ -52,6 +54,10 @@
static int full_paths;
static int show_nr_samples;
+static int show_stat;
+static int show_stat_events;
+static int show_stat_event_max;
+static struct read_event *show_stat_event;
static unsigned long page_size;
static unsigned long mmap_window = 32;
@@ -126,6 +132,8 @@
struct read_event read;
} event_t;
+static struct perf_counter_attr *perf_header__find_attr(u64 id);
+
static int repsep_fprintf(FILE *fp, const char *fmt, ...)
{
int n;
@@ -1350,6 +1358,13 @@
}
}
+static int compar_read_event_by_tid(const void *e1, const void *e2)
+{
+ const struct read_event *event1 = e1;
+ const struct read_event *event2 = e2;
+ return event1->tid - event2->tid;
+}
+
static size_t output__fprintf(FILE *fp, u64 total_samples)
{
struct hist_entry *pos;
@@ -1430,6 +1445,21 @@
}
fprintf(fp, "\n");
+ if (show_stat && show_stat_events) {
+ int i;
+ qsort(&show_stat_event[0], show_stat_events, sizeof(struct read_event), compar_read_event_by_tid);
+ fprintf(fp, "# Per-thread statistics:\n");
+ fprintf(fp, "# PID TID Event Count\n");
+ for(i=0; i<show_stat_events; i++) {
+ struct read_event *event = &show_stat_event[i];
+ struct perf_counter_attr *attr = perf_header__find_attr(event->id);
+ printf(" %d %d %s %Lu\n",
+ event->pid, event->tid,
+ attr ? __event_name(attr->type, attr->config) : "unknown",
+ event->value);
+ }
+ }
+
return ret;
}
@@ -1703,6 +1733,24 @@
{
struct perf_counter_attr *attr = perf_header__find_attr(event->read.id);
+ if (show_stat) {
+ if (!show_stat_event) {
+ show_stat_events = 0;
+ show_stat_event_max = 16;
+ show_stat_event = malloc(show_stat_event_max * sizeof(*show_stat_event));
+ if (!show_stat_event)
+ die("cannot allocate show_stat_event array");
+ }
+ if (show_stat_events == show_stat_event_max) {
+ show_stat_event_max *= 2;
+ show_stat_event = realloc(show_stat_event, show_stat_event_max * sizeof(*show_stat_event));
+ if (!show_stat_event)
+ die("cannot enlarge show_stat_event array");
+ }
+ memcpy(&show_stat_event[show_stat_events], &event->read, sizeof(struct read_event));
+ show_stat_events++;
+ }
+
dprintf("%p [%p]: PERF_EVENT_READ: %d %d %s %Lu\n",
(void *)(offset + head),
(void *)(long)(event->header.size),
@@ -1998,6 +2046,8 @@
"Show a column with the number of samples"),
OPT_STRING('s', "sort", &sort_order, "key[,key2...]",
"sort by key(s): pid, comm, dso, symbol, parent"),
+ OPT_BOOLEAN('S', "stat", &show_stat,
+ "show per-thread event counters"),
OPT_BOOLEAN('P', "full-paths", &full_paths,
"Don't shorten the pathnames taking into account the cwd"),
OPT_STRING('p', "parent", &parent_pattern, "regex",
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists