Messages in this thread |  | | | Subject | Re: [PATCH] trace: power: add trace_clock_set_parent | | From | Steven Rostedt <> | | Date | Wed, 09 May 2012 19:50:54 -0400 |
| |
On Wed, 2012-05-09 at 16:21 -0700, Colin Cross wrote: > Adds a new trace event to be called from clk_set_parent. Some > cpufreq drivers, including Tegra, reparent the cpu clock to a > slower clock while the main pll is relocking, tracing > clk_set_parent allows traces to show how for long the cpu is > running slower. > > Uses a separate TRACE_EVENT instead of the clock event class to > allow the event to contain string names for the child and the > parent. > > Signed-off-by: Colin Cross <ccross@android.com> > --- > include/trace/events/power.h | 19 +++++++++++++++++++ > 1 files changed, 19 insertions(+), 0 deletions(-) > > diff --git a/include/trace/events/power.h b/include/trace/events/power.h > index cae9a94..2b81df6 100644 > --- a/include/trace/events/power.h > +++ b/include/trace/events/power.h > @@ -204,6 +204,25 @@ enum { > TP_ARGS(name, state, cpu_id) > ); > > +TRACE_EVENT(clock_set_parent, > + > + TP_PROTO(const char *name, const char *parent_name), > + > + TP_ARGS(name, parent_name), > + > + TP_STRUCT__entry( > + __string( name, name ) > + __string( parent_name, parent_name ) > + ), > + > + TP_fast_assign( > + __assign_str(name, name); > + __assign_str(name, parent_name);
Don't you mean: __assign_str(parent_name, parent_name); ?
-- Steve
> + ), > + > + TP_printk("%s parent=%s", __get_str(name), __get_str(parent_name)) > +); > + > /* > * The power domain events are used for power domains transitions > */
|  |