Re: [PATCH v3 5/8] riscv: mark some code and data as file-static

From: Paul Walmsley
Date: Wed Oct 23 2019 - 18:32:58 EST


On Wed, 23 Oct 2019, Greentime Hu wrote:

> Paul Walmsley <paul.walmsley@xxxxxxxxxx> æ 2019å10æ19æ éå äå3:58åéï
> >
> > Several functions and arrays which are only used in the files in which
> > they are declared are missing "static" qualifiers. Warnings for these
> > symbols are reported by sparse:
> >
> > arch/riscv/kernel/stacktrace.c:22:14: warning: symbol 'walk_stackframe' was not declared. Should it be static?

[ ... ]

> I think walk_stackframe() could not be static because it will be used
> in perf_callchain.c.

Thanks Greentime - will update the patch.


- Paul