Re: [RFC] btf: Some structs are doubled because of struct ring_buffer

From: Jiri Olsa
Date: Sat Dec 14 2019 - 06:35:33 EST


On Fri, Dec 13, 2019 at 02:05:31PM -0500, Steven Rostedt wrote:

SNIP

> struct trace_array *tr = filp->private_data;
> - struct ring_buffer *buffer = tr->trace_buffer.buffer;
> + struct trace_buffer *buffer = tr->trace_buffer.buffer;
> unsigned long val;
> int ret;
>
> diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h
> index 63bf60f79398..308fcd673102 100644
> --- a/kernel/trace/trace.h
> +++ b/kernel/trace/trace.h
> @@ -178,7 +178,7 @@ struct trace_option_dentry;
>
> struct trace_buffer {
> struct trace_array *tr;
> - struct ring_buffer *buffer;
> + struct trace_buffer *buffer;

perf change is fine, but 'trace_buffer' won't work because
we already have 'struct trace_buffer' defined in here

maybe we could change this name to trace_buffer_array?

thanks,
jirka