diff options
| author | Ben Dooks <ben-linux@fluff.org> | 2022-07-21 22:55:09 +0300 |
|---|---|---|
| committer | akpm <akpm@linux-foundation.org> | 2022-07-30 04:12:36 +0300 |
| commit | 787dbea11a5d6843999ff71a3fb9aa1ed6d5d889 (patch) | |
| tree | 8d66478e7d8d053c29a077300188fcd298f2d647 /arch/openrisc | |
| parent | 45ee6d1e935d879d86aebd1fd15afb3bc015c4a0 (diff) | |
| download | linux-787dbea11a5d6843999ff71a3fb9aa1ed6d5d889.tar.xz | |
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/openrisc')
| -rw-r--r-- | arch/openrisc/kernel/smp.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/arch/openrisc/kernel/smp.c b/arch/openrisc/kernel/smp.c index 27041db2c8b0..e1419095a6f0 100644 --- a/arch/openrisc/kernel/smp.c +++ b/arch/openrisc/kernel/smp.c @@ -197,12 +197,6 @@ void smp_send_stop(void) smp_call_function(stop_this_cpu, NULL, 0); } -/* not supported, yet */ -int setup_profiling_timer(unsigned int multiplier) -{ - return -EINVAL; -} - void __init set_smp_cross_call(void (*fn)(const struct cpumask *, unsigned int)) { smp_cross_call = fn; |