function tracing: fix wrong position computing of stack_trace
authorLiming Wang <liming.wang@windriver.com>
Fri, 21 Nov 2008 03:00:18 +0000 (11:00 +0800)
committerIngo Molnar <mingo@elte.hu>
Fri, 21 Nov 2008 07:49:52 +0000 (08:49 +0100)
commit522a110b42b306d696cf84e34c677ed0e7080194
treea77d238aed14185cb5662daae5f01f0da405a834
parented313489badef16d700f5a3be50e8fd8f8294bc8
function tracing: fix wrong position computing of stack_trace

Impact: make output of stack_trace complete if buffer overruns

When read buffer overruns, the output of stack_trace isn't complete.

When printing records with seq_printf in t_show, if the read buffer
has overruned by the current record, then this record won't be
printed to user space through read buffer, it will just be dropped in
this printing.

When next printing, t_start should return the "*pos"th record, which
is the one dropped by previous printing, but it just returns
(m->private + *pos)th record.

Here we use a more sane method to implement seq_operations which can
be found in kernel code. Thus we needn't initialize m->private.

About testing, it's not easy to overrun read buffer, but we can use
seq_printf to print more padding bytes in t_show, then it's easy to
check whether or not records are lost.

This commit has been tested on both condition of overrun and non
overrun.

Signed-off-by: Liming Wang <liming.wang@windriver.com>
Acked-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/trace/trace_stack.c