Commit ec06f9b9 authored by Jiri Olsa's avatar Jiri Olsa Committed by Arnaldo Carvalho de Melo

perf c2c report: Add cacheline hists processing

Store cacheline related entries in nested hist object for each cacheline
data. Nested entries are sorted by 'offset' within related cacheline.

We will allow specific sort keys to be configured for nested cacheline
data entries in following patches.
Signed-off-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Don Zickus <dzickus@redhat.com>
Cc: Joe Mario <jmario@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1474558645-19956-18-git-send-email-jolsa@kernel.org
[ he__get_hists() should return NULL when c2c_hists__init() fails ]
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 78b27543
...@@ -61,6 +61,34 @@ static struct hist_entry_ops c2c_entry_ops = { ...@@ -61,6 +61,34 @@ static struct hist_entry_ops c2c_entry_ops = {
.free = c2c_he_free, .free = c2c_he_free,
}; };
static int c2c_hists__init(struct c2c_hists *hists,
const char *sort);
static struct hists*
he__get_hists(struct hist_entry *he,
const char *sort)
{
struct c2c_hist_entry *c2c_he;
struct c2c_hists *hists;
int ret;
c2c_he = container_of(he, struct c2c_hist_entry, he);
if (c2c_he->hists)
return &c2c_he->hists->hists;
hists = c2c_he->hists = zalloc(sizeof(*hists));
if (!hists)
return NULL;
ret = c2c_hists__init(hists, sort);
if (ret) {
free(hists);
return NULL;
}
return &hists->hists;
}
static int process_sample_event(struct perf_tool *tool __maybe_unused, static int process_sample_event(struct perf_tool *tool __maybe_unused,
union perf_event *event, union perf_event *event,
struct perf_sample *sample, struct perf_sample *sample,
...@@ -70,7 +98,7 @@ static int process_sample_event(struct perf_tool *tool __maybe_unused, ...@@ -70,7 +98,7 @@ static int process_sample_event(struct perf_tool *tool __maybe_unused,
struct hists *hists = &c2c.hists.hists; struct hists *hists = &c2c.hists.hists;
struct hist_entry *he; struct hist_entry *he;
struct addr_location al; struct addr_location al;
struct mem_info *mi; struct mem_info *mi, *mi_dup;
int ret; int ret;
if (machine__resolve(machine, &al, sample) < 0) { if (machine__resolve(machine, &al, sample) < 0) {
...@@ -83,19 +111,50 @@ static int process_sample_event(struct perf_tool *tool __maybe_unused, ...@@ -83,19 +111,50 @@ static int process_sample_event(struct perf_tool *tool __maybe_unused,
if (mi == NULL) if (mi == NULL)
return -ENOMEM; return -ENOMEM;
mi_dup = memdup(mi, sizeof(*mi));
if (!mi_dup)
goto free_mi;
he = hists__add_entry_ops(hists, &c2c_entry_ops, he = hists__add_entry_ops(hists, &c2c_entry_ops,
&al, NULL, NULL, mi, &al, NULL, NULL, mi,
sample, true); sample, true);
if (he == NULL) { if (he == NULL)
free(mi); goto free_mi_dup;
return -ENOMEM;
} hists__inc_nr_samples(hists, he->filtered);
ret = hist_entry__append_callchain(he, sample);
if (!ret) {
mi = mi_dup;
mi_dup = memdup(mi, sizeof(*mi));
if (!mi_dup)
goto free_mi;
hists = he__get_hists(he, "offset");
if (!hists)
goto free_mi_dup;
he = hists__add_entry_ops(hists, &c2c_entry_ops,
&al, NULL, NULL, mi,
sample, true);
if (he == NULL)
goto free_mi_dup;
hists__inc_nr_samples(hists, he->filtered); hists__inc_nr_samples(hists, he->filtered);
ret = hist_entry__append_callchain(he, sample); ret = hist_entry__append_callchain(he, sample);
}
out:
addr_location__put(&al); addr_location__put(&al);
return ret; return ret;
free_mi_dup:
free(mi_dup);
free_mi:
free(mi);
ret = -ENOMEM;
goto out;
} }
static struct perf_c2c c2c = { static struct perf_c2c c2c = {
...@@ -400,6 +459,27 @@ static int c2c_hists__reinit(struct c2c_hists *c2c_hists, ...@@ -400,6 +459,27 @@ static int c2c_hists__reinit(struct c2c_hists *c2c_hists,
return hpp_list__parse(&c2c_hists->list, output, sort); return hpp_list__parse(&c2c_hists->list, output, sort);
} }
static int filter_cb(struct hist_entry *he __maybe_unused)
{
return 0;
}
static int resort_cl_cb(struct hist_entry *he)
{
struct c2c_hist_entry *c2c_he;
struct c2c_hists *c2c_hists;
c2c_he = container_of(he, struct c2c_hist_entry, he);
c2c_hists = c2c_he->hists;
if (c2c_hists) {
hists__collapse_resort(&c2c_hists->hists, NULL);
hists__output_resort_cb(&c2c_hists->hists, NULL, filter_cb);
}
return 0;
}
static int perf_c2c__report(int argc, const char **argv) static int perf_c2c__report(int argc, const char **argv)
{ {
struct perf_session *session; struct perf_session *session;
...@@ -458,7 +538,7 @@ static int perf_c2c__report(int argc, const char **argv) ...@@ -458,7 +538,7 @@ static int perf_c2c__report(int argc, const char **argv)
ui_progress__init(&prog, c2c.hists.hists.nr_entries, "Sorting..."); ui_progress__init(&prog, c2c.hists.hists.nr_entries, "Sorting...");
hists__collapse_resort(&c2c.hists.hists, NULL); hists__collapse_resort(&c2c.hists.hists, NULL);
hists__output_resort(&c2c.hists.hists, &prog); hists__output_resort_cb(&c2c.hists.hists, &prog, resort_cl_cb);
ui_progress__finish(); ui_progress__finish();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment