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) |
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 61f39f8b62e1..98fa931b6864 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -1105,7 +1105,7 @@ struct ftrace_page { | |||
1105 | static struct ftrace_page *ftrace_pages_start; | 1105 | static struct ftrace_page *ftrace_pages_start; |
1106 | static struct ftrace_page *ftrace_pages; | 1106 | static struct ftrace_page *ftrace_pages; |
1107 | 1107 | ||
1108 | static bool ftrace_hash_empty(struct ftrace_hash *hash) | 1108 | static bool __always_inline ftrace_hash_empty(struct ftrace_hash *hash) |
1109 | { | 1109 | { |
1110 | return !hash || !hash->count; | 1110 | return !hash || !hash->count; |
1111 | } | 1111 | } |