diff options
author | Mathieu Malaterre <malat@debian.org> | 2018-03-08 15:28:56 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2018-06-22 05:07:47 -0400 |
commit | 57d6a7938a8fc6cee8420b40ca244220b41721f5 (patch) | |
tree | 38615ee173a2f541b3aebaa12c77107c5e889082 | |
parent | 90718e32e1dcc2479acfa208ccfc6442850b594c (diff) |
perf/core: Move the inline keyword at the beginning of the function declaration
When building perf with W=1 the following warning triggers:
CC kernel/events/ring_buffer.o
kernel/events/ring_buffer.c:105:1: warning: ‘inline’ is not at beginning of declaration [-Wold-style-declaration]
static bool __always_inline
^~~~~~
...
Move the inline keyword to the beginning of the function declaration.
Signed-off-by: Mathieu Malaterre <malat@debian.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: trival@kernel.org
Link: http://lkml.kernel.org/r/20180308202856.9378-1-malat@debian.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/events/ring_buffer.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/kernel/events/ring_buffer.c b/kernel/events/ring_buffer.c index 1d8ca9ea9979..d11f60cbe8ca 100644 --- a/kernel/events/ring_buffer.c +++ b/kernel/events/ring_buffer.c | |||
@@ -103,7 +103,7 @@ out: | |||
103 | preempt_enable(); | 103 | preempt_enable(); |
104 | } | 104 | } |
105 | 105 | ||
106 | static bool __always_inline | 106 | static __always_inline bool |
107 | ring_buffer_has_space(unsigned long head, unsigned long tail, | 107 | ring_buffer_has_space(unsigned long head, unsigned long tail, |
108 | unsigned long data_size, unsigned int size, | 108 | unsigned long data_size, unsigned int size, |
109 | bool backward) | 109 | bool backward) |
@@ -114,7 +114,7 @@ ring_buffer_has_space(unsigned long head, unsigned long tail, | |||
114 | return CIRC_SPACE(tail, head, data_size) >= size; | 114 | return CIRC_SPACE(tail, head, data_size) >= size; |
115 | } | 115 | } |
116 | 116 | ||
117 | static int __always_inline | 117 | static __always_inline int |
118 | __perf_output_begin(struct perf_output_handle *handle, | 118 | __perf_output_begin(struct perf_output_handle *handle, |
119 | struct perf_event *event, unsigned int size, | 119 | struct perf_event *event, unsigned int size, |
120 | bool backward) | 120 | bool backward) |
@@ -414,7 +414,7 @@ err: | |||
414 | } | 414 | } |
415 | EXPORT_SYMBOL_GPL(perf_aux_output_begin); | 415 | EXPORT_SYMBOL_GPL(perf_aux_output_begin); |
416 | 416 | ||
417 | static bool __always_inline rb_need_aux_wakeup(struct ring_buffer *rb) | 417 | static __always_inline bool rb_need_aux_wakeup(struct ring_buffer *rb) |
418 | { | 418 | { |
419 | if (rb->aux_overwrite) | 419 | if (rb->aux_overwrite) |
420 | return false; | 420 | return false; |