Hi Steve,

On Thu, 2026-01-22 at 20:45 -0500, Steven Rostedt wrote:
> From: Steven Rostedt <[email protected]>
> 
> When debugging the synthetic events, being able to function trace its
> functions is very useful (now that CONFIG_FUNCTION_SELF_TRACING is
> available). For some reason trace_event_raw_event_synth() was marked as
> "notrace", which was totally unnecessary as all of the tracing directory
> had function tracing disabled until the recent FUNCTION_SELF_TRACING was
> added.
> 
> Remove the notrace annotation from trace_event_raw_event_synth() as
> there's no reason to not trace it when tracing synthetic event functions.
> 
> Signed-off-by: Steven Rostedt (Google) <[email protected]>

Acked-by: Tom Zanussi <[email protected]>

Thanks,

Tom

> ---
>  kernel/trace/trace_events_synth.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/kernel/trace/trace_events_synth.c 
> b/kernel/trace/trace_events_synth.c
> index 4554c458b78c..79a506fc2856 100644
> --- a/kernel/trace/trace_events_synth.c
> +++ b/kernel/trace/trace_events_synth.c
> @@ -493,9 +493,9 @@ static unsigned int trace_stack(struct synth_trace_event 
> *entry,
>       return len;
>  }
>  
> -static notrace void trace_event_raw_event_synth(void *__data,
> -                                             u64 *var_ref_vals,
> -                                             unsigned int *var_ref_idx)
> +static void trace_event_raw_event_synth(void *__data,
> +                                     u64 *var_ref_vals,
> +                                     unsigned int *var_ref_idx)
>  {
>       unsigned int i, n_u64, val_idx, len, data_size = 0;
>       struct trace_event_file *trace_file = __data;


Reply via email to