Re: [PATCH][RFC] vsprintf: unify the format decoding layer for its3 users

[Steven Rostedt]

On Thu, 26 Feb 2009, Andrew Morton wrote:

> On Fri, 27 Feb 2009 07:19:37 +0100 Frederic Weisbecker <fweisbec@xxxxxxxxx> wrote:
> 
> > 
> > An new optimization is making its way to ftrace. Its purpose is to
> > make ftrace_printk() consuming less memory and be faster.
> > 
> > Written by Lai Jiangshan, the approach is to delay the formatting
> > job from tracing time to output time.
> > Currently, a call to ftrace_printk will format the whole string and
> > insert it into the ring buffer.
> 
> It does that?   eek.

Yes, I remember when Linus suggested having a pointer to the format and
the args saved. There was two issues I found.

1) You still needed to read the format string to find out what arguments 
   to save.

2) The format string could be part of module data, and if the module is
   unloaded between the time of recording and the time of reading the 
   trace, we now have a format pointer pointing to God knows what.

When posting ftrace_printk, I said this was for debugging purposes only,
and I was not about to work on optimizations there of. But Frederic and
Lai are doing a great job in that aspect ;-)

HAHAHAHHAAAA!!!! My evil scheme is working! I post some sub-optimal code, 
and have others do the nasty work for me!!!!

Oh, did I just say that out loud?

-- 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/