Eric Saxe wrote:
> Here is the webrev for the DTrace probes that powertop is consuming:
> 
>     - cpu_speed_change
>     - idle_state_transition   (formerly cstate_transition)
>     - cyclic_{start | end}
> 
> Please take a look, and send out any comments / feedback. I'd like to 
> get these integrated into Nevada so that Powertop can be otherwise self 
> contained.
> 
> I've incorporated Sasha's suggestion to avoid conditionals/dereferencing 
> in the probe implementation, have renamed the "cstate_transition" probe 
> to something more suitably platform independent, and have #defined the 
> idle CPU states.
> 
> http://cr.opensolaris.org/~esaxe/probes/

Your changes look good.

Mark

> Thanks,
> -Eric
> 
> 
> _______________________________________________
> tesla-dev mailing list
> tesla-dev at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/tesla-dev


Reply via email to