diff options
| author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-10-26 08:45:22 +0100 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2015-10-26 08:51:32 +0100 |
| commit | fb662288284e8f2ec26f13d50a6b0d5781771648 (patch) | |
| tree | c048e8da54d35d7abafe5f661a37a2a59e9f0a3c /kernel/trace/trace_events.c | |
| parent | tracing: Check all tasks on each CPU when filtering pids (diff) | |
| download | linux-fb662288284e8f2ec26f13d50a6b0d5781771648.tar.xz linux-fb662288284e8f2ec26f13d50a6b0d5781771648.zip | |
tracing: Fix sparse RCU warning
p_start() and p_stop() are seq_file functions that match. Teach sparse to
know that rcu_read_lock_sched() that is taken by p_start() is released by
p_stop.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_events.c')
| -rw-r--r-- | kernel/trace/trace_events.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 2b7fccd499c6..fb0261e90acc 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -938,6 +938,7 @@ static void t_stop(struct seq_file *m, void *p) } static void *p_start(struct seq_file *m, loff_t *pos) + __acquires(RCU) { struct trace_pid_list *pid_list; struct trace_array *tr = m->private; @@ -960,6 +961,7 @@ static void *p_start(struct seq_file *m, loff_t *pos) } static void p_stop(struct seq_file *m, void *p) + __releases(RCU) { rcu_read_unlock_sched(); mutex_unlock(&event_mutex); |