aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Rostedt <srostedt@redhat.com>2009-03-26 11:03:29 -0400
committerSteven Rostedt <rostedt@goodmis.org>2009-04-23 23:03:19 -0400
commit89ec0dee9eba6275d47be0b878cf5f6d5c2fb6eb (patch)
tree30db465c8194f2502af497d7276710279de97a7f
parent9be24414aad047dcf9d8d2a9a929321536c7ebec (diff)
tracing: increase size of number of possible events
With the new event tracing registration, we must increase the number of events that can be registered. Currently the type field is only one byte, which leaves us only 256 possible events. Since we do not save the CPU number in the tracer anymore (it is determined by the per cpu ring buffer that is used) we have an extra byte to use. This patch increases the size of type from 1 byte (256 events) to 2 bytes (65,536 events). It also adds a WARN_ON_ONCE if we exceed that limit. [ Impact: allow more than 255 events ] Signed-off-by: Steven Rostedt <srostedt@redhat.com>
-rw-r--r--include/linux/ftrace_event.h5
-rw-r--r--kernel/trace/trace_events.c2
-rw-r--r--kernel/trace/trace_output.c2
3 files changed, 7 insertions, 2 deletions
diff --git a/include/linux/ftrace_event.h b/include/linux/ftrace_event.h
index 2a4a40749911..07e0a6d64a24 100644
--- a/include/linux/ftrace_event.h
+++ b/include/linux/ftrace_event.h
@@ -16,13 +16,16 @@ struct dentry;
16 * bash-15816 [01] 235.197585: idle_cpu <- irq_enter 16 * bash-15816 [01] 235.197585: idle_cpu <- irq_enter
17 */ 17 */
18struct trace_entry { 18struct trace_entry {
19 int type; 19 unsigned short type;
20 unsigned char flags; 20 unsigned char flags;
21 unsigned char preempt_count; 21 unsigned char preempt_count;
22 int pid; 22 int pid;
23 int tgid; 23 int tgid;
24}; 24};
25 25
26#define FTRACE_MAX_EVENT \
27 ((1 << (sizeof(((struct trace_entry *)0)->type) * 8)) - 1)
28
26/* 29/*
27 * Trace iterator - used by printout routines who present trace 30 * Trace iterator - used by printout routines who present trace
28 * results to users and which routines might sleep, etc: 31 * results to users and which routines might sleep, etc:
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c
index 5d6e879cf875..9887131afa03 100644
--- a/kernel/trace/trace_events.c
+++ b/kernel/trace/trace_events.c
@@ -398,7 +398,7 @@ static int trace_write_header(struct trace_seq *s)
398 "\tfield:%s %s;\toffset:%zu;\tsize:%zu;\n" 398 "\tfield:%s %s;\toffset:%zu;\tsize:%zu;\n"
399 "\tfield:%s %s;\toffset:%zu;\tsize:%zu;\n" 399 "\tfield:%s %s;\toffset:%zu;\tsize:%zu;\n"
400 "\n", 400 "\n",
401 FIELD(int, type), 401 FIELD(unsigned short, type),
402 FIELD(unsigned char, flags), 402 FIELD(unsigned char, flags),
403 FIELD(unsigned char, preempt_count), 403 FIELD(unsigned char, preempt_count),
404 FIELD(int, pid), 404 FIELD(int, pid),
diff --git a/kernel/trace/trace_output.c b/kernel/trace/trace_output.c
index 83a8abb9640f..06997e75114b 100644
--- a/kernel/trace/trace_output.c
+++ b/kernel/trace/trace_output.c
@@ -537,6 +537,8 @@ int register_ftrace_event(struct trace_event *event)
537 out: 537 out:
538 mutex_unlock(&trace_event_mutex); 538 mutex_unlock(&trace_event_mutex);
539 539
540 WARN_ON_ONCE(next_event_type > FTRACE_MAX_EVENT);
541
540 return ret; 542 return ret;
541} 543}
542EXPORT_SYMBOL_GPL(register_ftrace_event); 544EXPORT_SYMBOL_GPL(register_ftrace_event);