Em Tue, Nov 18, 2014 at 11:38:19AM -0500, kan.li...@intel.com escreveu:
> From: Kan Liang <kan.li...@intel.com>
>
> Currently, the perf diff only works with same binaries. That's because
> it compares the symbol start address. It doesn't work if the perf.data
> comes from different binaries. This patch matches the function names.
>
> Signed-off-by: Kan Liang <kan.li...@intel.com>
> Acked-by: Namhyung Kim <namhy...@kernel.org>
Can you provide an example of this?
Also, can you add some note on the documentation? The example you use in
the changelog may also be appropriate to have in the man page.
Its been a looong while since I looked at the diff code, but yeah,
comparing by symbol address makes no sense, it had to do a diff from
file name!
The canonical example is:
vim workload.c
make workload
perf record ./workload
vim workload.c # do some changes
perf record ./workload
perf diff
And of course doing that would probably result in symbols moving around
as functions grow in size and new ones are added...
/me scratches head...
- Arnaldo
> ---
> tools/perf/util/sort.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c
> index 95efaaf..bea2e07 100644
> --- a/tools/perf/util/sort.c
> +++ b/tools/perf/util/sort.c
> @@ -1432,6 +1432,15 @@ int sort_dimension__add(const char *tok)
> sort__has_parent = 1;
> } else if (sd->entry == &sort_sym) {
> sort__has_sym = 1;
> + /*
> + * perf diff displays the performance difference amongst
> + * two or more perf.data files. Those files could come
> + * from different binaries. So we should not compare
> + * their ips, but the name of symble.
s/symble/symbol/g
> + */
> + if (sort__mode == SORT_MODE__DIFF)
> + sd->entry->se_collapse = sort__sym_sort;
> +
> } else if (sd->entry == &sort_dso) {
> sort__has_dso = 1;
> }
> --
> 1.8.3.2
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
