Re: [PATCH v6 25/37] tracing: Add support for 'field variables'

[Namhyung Kim]

On Fri, Nov 17, 2017 at 02:33:04PM -0600, Tom Zanussi wrote:
> @@ -1387,6 +1405,8 @@ static struct trace_event_file *find_var_file(struct trace_array *tr,
>  	list_for_each_entry(var_data, &tr->hist_vars, list) {
>  		var_hist_data = var_data->hist_data;
>  		file = var_hist_data->event_file;
> +		if (file == found)
> +			continue;

Shouldn't it be moved to the patch 22?

Thanks,
Namhyung


>  		call = file->event_call;
>  		name = trace_event_name(call);
>