[kees:kspp/cfi/x86 10/69] kernel/trace/fgraph.c:330:6: sparse: sparse: symbol 'ftrace_graph_return_stub' was not declared. Should it be static?

[kees:kspp/cfi/x86 10/69] kernel/trace/fgraph.c:330:6: sparse: sparse: symbol 'ftrace_graph_return_stub' was not declared. Should it be static?

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 984 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git kspp/cfi/x86
head:   3baeecfdeffc3ae699b0282859a8098b376b8f05
commit: 561ab42f3378fadd7d8a1656a631605785e7dc1e [10/69] ftrace: fix function type mismatches
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.1-29-g781bc5d-dirty
        git checkout 561ab42f3378fadd7d8a1656a631605785e7dc1e
        make ARCH=x86_64 allmodconfig
        make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>


sparse warnings: (new ones prefixed by >>)

>> kernel/trace/fgraph.c:330:6: sparse: sparse: symbol 'ftrace_graph_return_stub' was not declared. Should it be static?

Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

[RFC PATCH kees] ftrace: ftrace_graph_return_stub() can be static

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 604 bytes --]


Fixes: 561ab42f3378 ("ftrace: fix function type mismatches")
Signed-off-by: kbuild test robot <lkp@intel.com>
---
 fgraph.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/trace/fgraph.c b/kernel/trace/fgraph.c
index ecfd4a4a106a2..8c3f66732da35 100644
--- a/kernel/trace/fgraph.c
+++ b/kernel/trace/fgraph.c
@@ -327,7 +327,7 @@ void ftrace_graph_sleep_time_control(bool enable)
 	fgraph_sleep_time = enable;
 }
 
-void ftrace_graph_return_stub(struct ftrace_graph_ret *trace)
+static void ftrace_graph_return_stub(struct ftrace_graph_ret *trace)
 {
 }