diff options
author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2014-05-01 12:44:50 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2014-05-14 11:37:28 -0400 |
commit | 68f40969f0173c02ddc22a40df865c81c29070e4 (patch) | |
tree | 79a6312ecb0ed728adb66f731bc88f1d55632904 /kernel | |
parent | 19eab4a472cfe4a3ae51cff1711d795e3f9bb564 (diff) | |
download | op-kernel-dev-68f40969f0173c02ddc22a40df865c81c29070e4.zip op-kernel-dev-68f40969f0173c02ddc22a40df865c81c29070e4.tar.gz |
ftrace: Always inline ftrace_hash_empty() helper function
The ftrace_hash_empty() function is a simple test:
return !hash || !hash->count;
But gcc seems to want to make it a call. As this is in an extreme
hot path of the function tracer, there's no reason it needs to be
a call. I only wrote it to be a helper function anyway, otherwise
it would have been inlined manually.
Force gcc to inline it, as it could have also been a macro.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/ftrace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 61f39f8..98fa931 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c @@ -1105,7 +1105,7 @@ struct ftrace_page { static struct ftrace_page *ftrace_pages_start; static struct ftrace_page *ftrace_pages; -static bool ftrace_hash_empty(struct ftrace_hash *hash) +static bool __always_inline ftrace_hash_empty(struct ftrace_hash *hash) { return !hash || !hash->count; } |