On 03/16/2018 09:31 PM, Aaron Lindsay wrote: > The instruction event is only enabled when icount is used, cycles are > always supported. Always defining get_cycle_count (but altering its > behavior depending on CONFIG_USER_ONLY) allows us to remove some > CONFIG_USER_ONLY #defines throughout the rest of the code. > > Signed-off-by: Aaron Lindsay <alind...@codeaurora.org> > --- [...]> #define SUPPORTED_EVENT_SENTINEL UINT16_MAX > static const pm_event pm_events[] = { > +#ifndef CONFIG_USER_ONLY > + { .number = 0x008, /* INST_RETIRED */
"Instruction architecturally executed" seems more explicit to me. > + .supported = instructions_supported, > + .get_count = instructions_get_count > + }, > + { .number = 0x011, /* CPU_CYCLES */ > + .supported = event_always_supported, > + .get_count = cycles_get_count > + }, > +#endif [...]