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 <[email protected]>
> ---
[...]> #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
[...]
