Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCH 03/15 V3] perf c2c: Shared data analyser | Date | Wed, 09 Apr 2014 09:58:04 +0900 |
| |
On Tue, 8 Apr 2014 10:22:26 -0400, Don Zickus wrote: > On Tue, Apr 08, 2014 at 03:59:15PM +0900, Namhyung Kim wrote: >> Hi Don, >> >> On Mon, 24 Mar 2014 15:36:54 -0400, Don Zickus wrote: >> > From: Arnaldo Carvalho de Melo <acme@redhat.com> >> > >> > This is the start of a new perf tool that will collect information about >> > memory accesses and analyse it to find things like hot cachelines, etc. >> >> So why not integrating this into existing 'perf mem' command if it's all >> about analyzing memory accesses? > > Our expectations were different. We expeted to do system-wide analysis > with loads and stores. With 'perf mem' you didn't have the ability to > anlayze both load and stores at the same time.
But it's very simple to change perf mem to work with the both IMHO.
> > In all my private conversations with Stephane, Arnalado and Jiri, it was > never brought up. We had just assumed that is made more sense to keep it > separate.
Well, I'm not sure ;-) Yes, the c2c is a complex tool which might deserve an own command, but the functionality is very similar and I guess there's something to share between them.
Thanks, Namhyung
| |