Commit 4dbbe2d8 authored by Matthias Kaehlcke's avatar Matthias Kaehlcke Committed by Steven Rostedt (VMware)

tracing: Use cpumask_available() to check if cpumask variable may be used

This fixes the following clang warning:

kernel/trace/trace.c:3231:12: warning: address of array 'iter->started'
  will always evaluate to 'true' [-Wpointer-bool-conversion]
        if (iter->started)

Link: http://lkml.kernel.org/r/20170421234110.117075-1-mka@chromium.orgSigned-off-by: default avatarMatthias Kaehlcke <mka@chromium.org>
Signed-off-by: default avatarSteven Rostedt (VMware) <rostedt@goodmis.org>
parent 77c0edde
...@@ -3311,13 +3311,14 @@ static void test_cpu_buff_start(struct trace_iterator *iter) ...@@ -3311,13 +3311,14 @@ static void test_cpu_buff_start(struct trace_iterator *iter)
if (!(iter->iter_flags & TRACE_FILE_ANNOTATE)) if (!(iter->iter_flags & TRACE_FILE_ANNOTATE))
return; return;
if (iter->started && cpumask_test_cpu(iter->cpu, iter->started)) if (cpumask_available(iter->started) &&
cpumask_test_cpu(iter->cpu, iter->started))
return; return;
if (per_cpu_ptr(iter->trace_buffer->data, iter->cpu)->skipped_entries) if (per_cpu_ptr(iter->trace_buffer->data, iter->cpu)->skipped_entries)
return; return;
if (iter->started) if (cpumask_available(iter->started))
cpumask_set_cpu(iter->cpu, iter->started); cpumask_set_cpu(iter->cpu, iter->started);
/* Don't print started cpu buffer for the first entry of the trace */ /* Don't print started cpu buffer for the first entry of the trace */
......
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