Re: [PATCH 2/3] PERF(kernel): Cleanup power events

Arjan van de Ven
2010-10-25T06:58:04Z

On 10/19/2010 4:36 AM, Thomas Renninger wrote:
> New power trace events:
> power:processor_idle
> power:processor_frequency
> power:machine_suspend
>
>
> C-state/idle accounting events:
> power:power_start
> power:power_end
> are replaced with:
> power:processor_idle
>
I think you need two trace points for this
one to enter idle
one to exit

because using magic encoding games to encode "exit"is a mistake; as can
be seen in this patch.
You're currently trying to use "0" to signal "end of idle", but "0" is
also a valid idle state (namely that of polling)