]> bbs.cooldavid.org Git - net-next-2.6.git/commit
tracing: Combine event filter_active and enable into single flags field
authorSteven Rostedt <srostedt@redhat.com>
Fri, 23 Apr 2010 15:12:36 +0000 (11:12 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Fri, 14 May 2010 18:33:22 +0000 (14:33 -0400)
commit553552ce1796c32cf4e3d4f45cd5b537de91dd1d
treea65defc1055bcc3e9f34327d2cc59704e536948b
parent32c0edaeaad74a7883e736ae0f3798784cfc2a80
tracing: Combine event filter_active and enable into single flags field

The filter_active and enable both use an int (4 bytes each) to
set a single flag. We can save 4 bytes per event by combining the
two into a single integer.

   text    data     bss     dec     hex filename
4913961 1088356  861512 6863829  68bbd5 vmlinux.orig
4894944 1018052  861512 6774508  675eec vmlinux.id
4894871 1012292  861512 6768675  674823 vmlinux.flags

This gives us another 5K in savings.

The modification of both the enable and filter fields are done
under the event_mutex, so it is still safe to combine the two.

Note: Although Mathieu gave his Acked-by, he would like it documented
 that the reads of flags are not protected by the mutex. The way the
 code works, these reads will not break anything, but will have a
 residual effect. Since this behavior is the same even before this
 patch, describing this situation is left to another patch, as this
 patch does not change the behavior, but just brought it to Mathieu's
 attention.

v2: Updated the event trace self test to for this change.

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Acked-by: Masami Hiramatsu <mhiramat@redhat.com>
Acked-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Tom Zanussi <tzanussi@gmail.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
include/linux/ftrace_event.h
kernel/trace/trace.h
kernel/trace/trace_events.c
kernel/trace/trace_events_filter.c
kernel/trace/trace_kprobe.c