On Fri, 31 Jan 2020 15:55:31 -0600 Tom Zanussi <zanussi@xxxxxxxxxx> wrote: > +++ b/kernel/trace/trace_events_hist.c > @@ -2053,24 +2053,72 @@ int synth_event_trace_start(struct trace_event_file *file, > } > EXPORT_SYMBOL_GPL(synth_event_trace_start); > > -static int save_synth_val(struct synth_field *field, u64 val, > +int __synth_event_add_val(const char *field_name, u64 val, Hmm, shouldn't __synth_event_add_val() still be static? -- Steve > struct synth_event_trace_state *trace_state) > { > - struct synth_trace_event *entry = trace_state->entry; > + struct synth_field *field = NULL; > + struct synth_trace_event *entry; > + struct synth_event *event; > + int i, ret = 0; > +