btrfs: Use trace condition for get_extent tracepoint
authorSteven Rostedt <rostedt@goodmis.org>
Fri, 15 Nov 2013 03:57:29 +0000 (22:57 -0500)
committerChris Mason <chris.mason@fusionio.com>
Thu, 21 Nov 2013 01:44:47 +0000 (20:44 -0500)
commit4cd8587ce8fb79e49d1d6d1fc065f056188fb86a
tree3139157ec70bf16dcc5a436eccd743d41397dc5e
parent52a157592140ac76d9f15637543aecc2100f95cb
btrfs: Use trace condition for get_extent tracepoint

Doing an if statement to test some condition to know if we should
trigger a tracepoint is pointless when tracing is disabled. This just
adds overhead and wastes a branch prediction. This is why the
TRACE_EVENT_CONDITION() was created. It places the check inside the jump
label so that the branch does not happen unless tracing is enabled.

That is, instead of doing:

if (em)
trace_btrfs_get_extent(root, em);

Which is basically this:

if (em)
if (static_key(trace_btrfs_get_extent)) {

Using a TRACE_EVENT_CONDITION() we can just do:

trace_btrfs_get_extent(root, em);

And the condition trace event will do:

if (static_key(trace_btrfs_get_extent)) {
if (em) {
...

The static key is a non conditional jump (or nop) that is faster than
having to check if em is NULL or not.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/inode.c
include/trace/events/btrfs.h