On Tue,  7 Jul 2015 18:10:41 +0800
Chunyan Zhang <[email protected]> wrote:

> output_stm is a link which is used to connect trace event
> logging with STMs, will be used in the upcoming patches.
> 
> Signed-off-by: Chunyan Zhang <[email protected]>
> ---
>  include/linux/trace_events.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/include/linux/trace_events.h b/include/linux/trace_events.h
> index 1063c85..28dcdff 100644
> --- a/include/linux/trace_events.h
> +++ b/include/linux/trace_events.h
> @@ -14,6 +14,7 @@ struct trace_buffer;
>  struct tracer;
>  struct dentry;
>  struct bpf_prog;
> +struct trace_buffer_stm;
>  
>  struct trace_print_flags {
>       unsigned long           mask;
> @@ -293,6 +294,9 @@ struct trace_event_call {
>        */
>       int                     flags; /* static flags of different events */
>  
> +     void (*output_stm)(struct trace_seq *tmp_seq, void *entry,
> +                        struct trace_buffer_stm *tb);
> +

Shouldn't the above also have a #ifdef CONFIG_STM_TRACE_EVENT
encompassing it? Otherwise we are adding 8 bytes to every trace event.

-- Steve

>  #ifdef CONFIG_PERF_EVENTS
>       int                             perf_refcount;
>       struct hlist_head __percpu      *perf_events;
> @@ -412,6 +416,10 @@ enum event_trigger_type {
>       ETT_EVENT_ENABLE        = (1 << 3),
>  };
>  
> +#ifdef CONFIG_STM_TRACE_EVENT
> +extern void stm_trace_event_write(const char *buf, unsigned len);
> +#endif
> +
>  extern int filter_match_preds(struct event_filter *filter, void *rec);
>  
>  extern int filter_check_discard(struct trace_event_file *file, void *rec,

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to