Re: [PATCH 04/31] perf, core: Add generic intx/intx_checkpointedcounter modifiers

From: Peter Zijlstra
Date: Fri Sep 28 2012 - 05:02:36 EST


On Thu, 2012-09-27 at 21:31 -0700, Andi Kleen wrote:
> + intx : 1, /* count inside transaction */
> + intx_checkpointed : 1, /* checkpointed in transaction */

I really hate those names.. what are they called in transactional memory
literature?

Also do we really need this? Using the event format stuff we could
equally well do:

{cpu/cycles/, cpu/cycles,intx/, cpu/cycles,intx_checkpointed/}

No need to push those bits through perf_event_attr::flags when you can
stuff then through perf_event_attr::config, esp. for very hardware
specific features.
--
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/