diff options
| author |   Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-09-29 18:13:33 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2015-09-30 04:35:18 -0400 |
| commit | b5e87c0581319481399b6d8e8d6972b5523c18e6 (patch) | |
| tree | 4439fbdae48bb3dbd6b8023c25be9c618da1c11e /kernel/trace/trace.c | |
| parent | tracing: Always show all tracer options in the options directory (diff) | |
| download | linux-dev-b5e87c0581319481399b6d8e8d6972b5523c18e6.tar.xz linux-dev-b5e87c0581319481399b6d8e8d6972b5523c18e6.zip | |
tracing: Add build bug if we have more trace_flags than bits
Add a enum that denotes the last bit of the trace_flags and have a
BUILD_BUG_ON(last_bit > 32).
If we add more bits than we have in trace_flags, the kernel wont build.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace.c')
| -rw-r--r-- | kernel/trace/trace.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 7446d4238f87..991bab9b79d2 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -7046,6 +7046,12 @@ __init static int tracer_alloc_buffers(void) int ring_buf_size; int ret = -ENOMEM; + /* + * Make sure we don't accidently add more trace options + * than we have bits for. + */ + BUILD_BUG_ON(TRACE_ITER_LAST_BIT > 32); + if (!alloc_cpumask_var(&tracing_buffer_mask, GFP_KERNEL)) goto out; |