diff options
| author |   Ben Dooks <ben-linux@fluff.org> | 2022-07-21 20:55:09 +0100 |
|---|---|---|
| committer |   akpm <akpm@linux-foundation.org> | 2022-07-29 18:12:36 -0700 |
| commit | 787dbea11a5d6843999ff71a3fb9aa1ed6d5d889 (patch) | |
| tree | 8d66478e7d8d053c29a077300188fcd298f2d647 /arch/powerpc | |
| parent | ocfs2: fix a typo in a comment (diff) | |
| download | linux-dev-787dbea11a5d6843999ff71a3fb9aa1ed6d5d889.tar.xz linux-dev-787dbea11a5d6843999ff71a3fb9aa1ed6d5d889.zip | |
profile: setup_profiling_timer() is moslty not implemented
The setup_profiling_timer() is mostly un-implemented by many
architectures. In many places it isn't guarded by CONFIG_PROFILE which is
needed for it to be used. Make it a weak symbol in kernel/profile.c and
remove the 'return -EINVAL' implementations from the kenrel.
There are a couple of architectures which do return 0 from the
setup_profiling_timer() function but they don't seem to do anything else
with it. To keep the /proc compatibility for now, leave these for a
future update or removal.
On ARM, this fixes the following sparse warning:
arch/arm/kernel/smp.c:793:5: warning: symbol 'setup_profiling_timer' was not declared. Should it be static?
Link: https://lkml.kernel.org/r/20220721195509.418205-1-ben-linux@fluff.org
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'arch/powerpc')
| -rw-r--r-- | arch/powerpc/kernel/smp.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c index bcefab484ea6..c037c26540dd 100644 --- a/arch/powerpc/kernel/smp.c +++ b/arch/powerpc/kernel/smp.c @@ -1674,13 +1674,6 @@ void start_secondary(void *unused) BUG(); } -#ifdef CONFIG_PROFILING -int setup_profiling_timer(unsigned int multiplier) -{ - return 0; -} -#endif - static void __init fixup_topology(void) { int i; |