Re: [PATCH 0/2] perf: add sort by inclusive time functionality (v2)

From: Arun Sharma
Date: Mon Mar 12 2012 - 15:58:22 EST


On 3/12/12 11:21 AM, Arun Sharma wrote:


BTW, I don't like the name 'inclusive' as a sort key. If it cares about
time, IMHO, the name should contain 'time' - something like 'itime' or
'inctime'?

The existing sort orders: pid, comm, dso, symbol, parent -- all care
about time, but none of them have time in their name?

I'll take that back. What they sort on depends on the event.

perf record -ge cache-misses
perf report -s inclusive

will sort by the number of cache-misses and not time.

-Arun
--
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/