rcu-tasks: Correct firstreport usage in check_all_holdout_tasks_trace

In check_all_holdout_tasks_trace(), firstreport is a pointer argument;
so, check the dereferenced value, instead of checking the pointer.

Signed-off-by: Neeraj Upadhyay <neeraju@codeaurora.org>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>

diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
index 75e7888b3fc958f906c6b6b12eec360accbeeb93..e2ec548fc0c448282586503e2535a882234f0d4c 100644 (file)
--- a/kernel/rcu/tasks.h
+++ b/kernel/rcu/tasks.h
@@ -1129,7 +1129,7 @@ static void check_all_holdout_tasks_trace(struct list_head *hop,
        cpus_read_unlock();
 
        if (needreport) {
-               if (firstreport)
+               if (*firstreport)
                        pr_err("INFO: rcu_tasks_trace detected stalls? (Late IPI?)\n");
                show_stalled_ipi_trace();
        }