On Thu, Jun 5, 2025 at 10:19 PM Steven Rostedt <rost...@goodmis.org> wrote:
>
> From: Steven Rostedt <rost...@goodmis.org>
>
> The events clock_enable, clock_disable, and clock_set_rate were added back
> in 2010. In 2011 they were used by the arm architecture but removed in
> 2013. These events add around 7K of memory which was wasted for the last 12
> years.
>
> Remove them.
>
> Link: https://lore.kernel.org/all/20250529130138.544ff...@gandalf.local.home/
>
> Fixes: 74704ac6ea402 ("tracing, perf: Add more power related events")
> Signed-off-by: Steven Rostedt (Google) <rost...@goodmis.org>
Acked-by: Rafael J. Wysocki <raf...@kernel.org>
> ---
> include/trace/events/power.h | 47 ------------------------------------
> 1 file changed, 47 deletions(-)
>
> diff --git a/include/trace/events/power.h b/include/trace/events/power.h
> index 9253e83b9bb4..6c631eec23e3 100644
> --- a/include/trace/events/power.h
> +++ b/include/trace/events/power.h
> @@ -337,53 +337,6 @@ DEFINE_EVENT(wakeup_source, wakeup_source_deactivate,
> TP_ARGS(name, state)
> );
>
> -/*
> - * The clock events are used for clock enable/disable and for
> - * clock rate change
> - */
> -DECLARE_EVENT_CLASS(clock,
> -
> - TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
> -
> - TP_ARGS(name, state, cpu_id),
> -
> - TP_STRUCT__entry(
> - __string( name, name )
> - __field( u64, state )
> - __field( u64, cpu_id )
> - ),
> -
> - TP_fast_assign(
> - __assign_str(name);
> - __entry->state = state;
> - __entry->cpu_id = cpu_id;
> - ),
> -
> - TP_printk("%s state=%lu cpu_id=%lu", __get_str(name),
> - (unsigned long)__entry->state, (unsigned long)__entry->cpu_id)
> -);
> -
> -DEFINE_EVENT(clock, clock_enable,
> -
> - TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
> -
> - TP_ARGS(name, state, cpu_id)
> -);
> -
> -DEFINE_EVENT(clock, clock_disable,
> -
> - TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
> -
> - TP_ARGS(name, state, cpu_id)
> -);
> -
> -DEFINE_EVENT(clock, clock_set_rate,
> -
> - TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
> -
> - TP_ARGS(name, state, cpu_id)
> -);
> -
> /*
> * The power domain events are used for power domains transitions
> */
> --
> 2.47.2
>
>
