[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <37D7C6CF3E00A74B8858931C1DB2F0770167815D@SHSMSX103.ccr.corp.intel.com>
Date: Tue, 2 Dec 2014 15:51:13 +0000
From: "Liang, Kan" <kan.liang@...el.com>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
CC: "jolsa@...hat.com" <jolsa@...hat.com>,
"namhyung@...nel.org" <namhyung@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"ak@...ux.intel.com" <ak@...ux.intel.com>
Subject: RE: [PATCH V5 1/1] perf tool:perf diff support for different
binaries
>
> Em Fri, Nov 21, 2014 at 10:55:48AM -0500, kan.liang@...el.com escreveu:
> > From: Kan Liang <kan.liang@...el.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.
> >
> > Here is a simple examples (Only change f2 and f3 functions location.)
> > There is no change in behavior, but the old perf diff display the
> > wrong differential profile.
>
> [acme@zoo linux]$ am /wb/1.patch
> Applying: perf tool:perf diff support for different binaries
> /home/acme/git/linux/.git/rebase-apply/patch:23: trailing whitespace.
> If no parameters are passed the samples will be sorted by dso and symbol.
> /home/acme/git/linux/.git/rebase-apply/patch:26: trailing whitespace.
> symbols name.
> warning: 2 lines add whitespace errors.
>
Sorry for the inconvenience, I will repost a clean version then.
Thanks,
Kan
> [1]+ Stopped am /wb/1.patch
> [acme@zoo linux]$
>
> Fixing this up this time.
>
> Please consider:
>
> chmod +x .git/hooks/*
>
> - Arnaldo
--
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