Commit 43bd1236 authored by Frederic Weisbecker's avatar Frederic Weisbecker

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: default avatarFrederic Weisbecker <fweisbec@gmail.com>
parent dbd3fbdf
...@@ -316,7 +316,7 @@ static int init_stat_file(struct stat_session *session) ...@@ -316,7 +316,7 @@ static int init_stat_file(struct stat_session *session)
int register_stat_tracer(struct tracer_stat *trace) int register_stat_tracer(struct tracer_stat *trace)
{ {
struct stat_session *session, *node, *tmp; struct stat_session *session, *node;
int ret; int ret;
if (!trace) if (!trace)
...@@ -327,7 +327,7 @@ int register_stat_tracer(struct tracer_stat *trace) ...@@ -327,7 +327,7 @@ int register_stat_tracer(struct tracer_stat *trace)
/* Already registered? */ /* Already registered? */
mutex_lock(&all_stat_sessions_mutex); mutex_lock(&all_stat_sessions_mutex);
list_for_each_entry_safe(node, tmp, &all_stat_sessions, session_list) { list_for_each_entry(node, &all_stat_sessions, session_list) {
if (node->ts == trace) { if (node->ts == trace) {
mutex_unlock(&all_stat_sessions_mutex); mutex_unlock(&all_stat_sessions_mutex);
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment