Re: [PATCH 4/6] perf, sort: Add physid sorting based on mmap2 data

From: Don Zickus
Date: Mon Mar 24 2014 - 16:18:55 EST


On Mon, Mar 24, 2014 at 12:54:31PM -0700, Andi Kleen wrote:
> Don Zickus <dzickus@xxxxxxxxxx> writes:
>
> > In order for the c2c tool to work correctly, it needs to properly
> > sort all the records on uniquely identifiable data addresses. These
> > unique addresses are converted from virtual addresses provided by the
> > hardware into a kernel address using an mmap2 record as the decoder.
>
> No documentation for the new option?
>
> Probably the new mode should be also supported by --sort

I hid the new option further in the changelog, so it isn't obvious. Sorry
about that.

Sample output: (perf report --stdio --physid-mode)

So the option was '--physid-mode' and if you don't pass in a '--sort' then
it takes the default sort of

'daddr,iaddr,pid,tid,major,minor,inode,inode_gen'

Otherwise you could pass in a combination of the other fields.

The output is not the best way to use the mmap2 data as it just gives you
hottest data addresses. Our c2c tool really takes the data addresses and
combines them into a cacheline and then processes the cacheline for
interesting bottlenecks (HITMs in our case).

I don't know a good way to present the data and yet still have the sort
useful for our c2c tool. So I threw this interface together. I am open
to suggestions.

Cheers,
Don
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/