aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrederic Weisbecker <fweisbec@gmail.com>2009-05-29 22:25:30 -0400
committerFrederic Weisbecker <fweisbec@gmail.com>2009-06-01 19:18:33 -0400
commit43bd1236234cacbc18d1476a9b57e7a306efddf5 (patch)
tree087fd59a2f98e85fdf017267998a7364a47536c3
parentdbd3fbdfeecfad4e71139db05d72560c3583e2a9 (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.c4
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
317int register_stat_tracer(struct tracer_stat *trace) 317int 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;