Re: [PATCH 1/1] ftrace: do not update max buffer with no users

From: Steven Rostedt
Date: Wed Nov 12 2008 - 21:43:21 EST



On Wed, 12 Nov 2008, Andrew Morton wrote:
> This sort of thing:
>
> +#ifdef CONFIG_TRACER_MAX_TRACE
> if (current_trace && current_trace->print_max)
> iter->tr = &max_tr;
> else
> +#endif
> iter->tr = inode->i_private;
>
> is particularly regrettable.

Yeah, I know it is particularly ugly :-(

I was hoping to spend some time to find a way to clean it up. The other
answer is to just scrap the config all together, and let the unused max
latency tracer buffer be allocated and take up space (with no users).

-- Steve

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