diff options
author | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2017-03-29 21:40:49 -0400 |
---|---|---|
committer | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2017-03-31 18:00:37 -0400 |
commit | 2d71d98900b8a4bd58c3ca92e404d5e3701de874 (patch) | |
tree | 0cc6b7911663623916107f64601bacfbcc8aeaac | |
parent | c20489dad156dd9919ebd854bbace46dbd2576a3 (diff) | |
download | linux-stable-2d71d98900b8a4bd58c3ca92e404d5e3701de874.tar.gz linux-stable-2d71d98900b8a4bd58c3ca92e404d5e3701de874.tar.bz2 linux-stable-2d71d98900b8a4bd58c3ca92e404d5e3701de874.zip |
ftrace: Return NULL at end of t_start() instead of calling t_hash_start()
The loop in t_start() of calling t_next() will call t_hash_start() if the
pos is beyond the functions and enters the hash items. There's no reason to
check if p is NULL and call t_hash_start(), as that would be redundant.
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
-rw-r--r-- | kernel/trace/ftrace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 3165b7f840e6..421530831ddd 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c @@ -3255,7 +3255,7 @@ static void *t_start(struct seq_file *m, loff_t *pos) } if (!p) - return t_hash_start(m, pos); + return NULL; return iter; } |