diff options
author | Steven Rostedt <srostedt@redhat.com> | 2010-09-09 08:43:22 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2010-09-14 11:42:29 -0400 |
commit | 2bccfffd1538f3523847583213567e2f7ce00926 (patch) | |
tree | 7062e0bfc46f66efa2ec3231a6c718a901a0a80c /kernel/trace | |
parent | bfa88ea7ee9e6b4fd673e45a8cc0a8e0b7ef4761 (diff) |
tracing: Do not reset *pos in set_ftrace_filter
After the filtered functions are read, the probed functions are read
from the hash in set_ftrace_filter. When the hashed probed functions
are read, the *pos passed in is reset. Instead of modifying the pos
given to the read function, just record the pos where the filtered
functions ended and subtract from that.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace')
-rw-r--r-- | kernel/trace/ftrace.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index fa7ece649fe1..585ea27025b1 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -1368,6 +1368,7 @@ enum { | |||
1368 | #define FTRACE_BUFF_MAX (KSYM_SYMBOL_LEN+4) /* room for wildcards */ | 1368 | #define FTRACE_BUFF_MAX (KSYM_SYMBOL_LEN+4) /* room for wildcards */ |
1369 | 1369 | ||
1370 | struct ftrace_iterator { | 1370 | struct ftrace_iterator { |
1371 | loff_t func_pos; | ||
1371 | struct ftrace_page *pg; | 1372 | struct ftrace_page *pg; |
1372 | int hidx; | 1373 | int hidx; |
1373 | int idx; | 1374 | int idx; |
@@ -1418,12 +1419,15 @@ static void *t_hash_start(struct seq_file *m, loff_t *pos) | |||
1418 | loff_t l; | 1419 | loff_t l; |
1419 | 1420 | ||
1420 | if (!(iter->flags & FTRACE_ITER_HASH)) | 1421 | if (!(iter->flags & FTRACE_ITER_HASH)) |
1421 | *pos = 0; | 1422 | iter->func_pos = *pos; |
1423 | |||
1424 | if (iter->func_pos > *pos) | ||
1425 | return NULL; | ||
1422 | 1426 | ||
1423 | iter->flags |= FTRACE_ITER_HASH; | 1427 | iter->flags |= FTRACE_ITER_HASH; |
1424 | 1428 | ||
1425 | iter->hidx = 0; | 1429 | iter->hidx = 0; |
1426 | for (l = 0; l <= *pos; ) { | 1430 | for (l = 0; l <= (*pos - iter->func_pos); ) { |
1427 | p = t_hash_next(m, p, &l); | 1431 | p = t_hash_next(m, p, &l); |
1428 | if (!p) | 1432 | if (!p) |
1429 | break; | 1433 | break; |