diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-05-29 22:25:30 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2009-06-01 19:18:33 -0400 |
commit | 43bd1236234cacbc18d1476a9b57e7a306efddf5 (patch) | |
tree | 087fd59a2f98e85fdf017267998a7364a47536c3 | |
parent | dbd3fbdfeecfad4e71139db05d72560c3583e2a9 (diff) |
tracing/stat: remove unappropriate safe walk on list
register_stat_tracer() uses list_for_each_entry_safe
to check whether a tracer is already present in the list.
But we don't delete anything from the list here, so
we don't need the safe version
[ Impact: cleanup list use is stat tracing ]
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
-rw-r--r-- | kernel/trace/trace_stat.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace_stat.c b/kernel/trace/trace_stat.c index 17f20ebdad2a..c00643733f4c 100644 --- a/kernel/trace/trace_stat.c +++ b/kernel/trace/trace_stat.c | |||
@@ -316,7 +316,7 @@ static int init_stat_file(struct stat_session *session) | |||
316 | 316 | ||
317 | int register_stat_tracer(struct tracer_stat *trace) | 317 | int register_stat_tracer(struct tracer_stat *trace) |
318 | { | 318 | { |
319 | struct stat_session *session, *node, *tmp; | 319 | struct stat_session *session, *node; |
320 | int ret; | 320 | int ret; |
321 | 321 | ||
322 | if (!trace) | 322 | if (!trace) |
@@ -327,7 +327,7 @@ int register_stat_tracer(struct tracer_stat *trace) | |||
327 | 327 | ||
328 | /* Already registered? */ | 328 | /* Already registered? */ |
329 | mutex_lock(&all_stat_sessions_mutex); | 329 | mutex_lock(&all_stat_sessions_mutex); |
330 | list_for_each_entry_safe(node, tmp, &all_stat_sessions, session_list) { | 330 | list_for_each_entry(node, &all_stat_sessions, session_list) { |
331 | if (node->ts == trace) { | 331 | if (node->ts == trace) { |
332 | mutex_unlock(&all_stat_sessions_mutex); | 332 | mutex_unlock(&all_stat_sessions_mutex); |
333 | return -EINVAL; | 333 | return -EINVAL; |