summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>2014-05-01 12:44:50 -0400
committerSteven Rostedt <rostedt@goodmis.org>2014-05-14 11:37:28 -0400
commit68f40969f0173c02ddc22a40df865c81c29070e4 (patch)
tree79a6312ecb0ed728adb66f731bc88f1d55632904 /kernel
parent19eab4a472cfe4a3ae51cff1711d795e3f9bb564 (diff)
downloadop-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.c2
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;
}
OpenPOWER on IntegriCloud