| Message ID | 20200531110015.598607-1-christophe.jaillet@wanadoo.fr (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 5311ebfb612f08ec2a712a86d0af7ee2b423a9fc |
| Headers | show |
| Series | arm64: debug: mark a function as __init to save some memory | expand |
Hi, On Sun, May 31, 2020 at 4:00 AM Christophe JAILLET <christophe.jaillet@wanadoo.fr> wrote: > > 'debug_monitors_init()' is only called via 'postcore_initcall'. > It can be marked as __init to save a few bytes of memory. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> > --- > arch/arm64/kernel/debug-monitors.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Seems OK to me. Reviewed-by: Douglas Anderson <dianders@chromium.org>
On Sun, 31 May 2020 13:00:15 +0200, Christophe JAILLET wrote: > 'debug_monitors_init()' is only called via 'postcore_initcall'. > It can be marked as __init to save a few bytes of memory. Applied to arm64 (for-next/core), thanks! [1/1] arm64: debug: mark a function as __init to save some memory https://git.kernel.org/arm64/c/5311ebfb612f Cheers,
diff --git a/arch/arm64/kernel/debug-monitors.c b/arch/arm64/kernel/debug-monitors.c index 15e80c876d46..5df49366e9ab 100644 --- a/arch/arm64/kernel/debug-monitors.c +++ b/arch/arm64/kernel/debug-monitors.c @@ -130,7 +130,7 @@ static int clear_os_lock(unsigned int cpu) return 0; } -static int debug_monitors_init(void) +static int __init debug_monitors_init(void) { return cpuhp_setup_state(CPUHP_AP_ARM64_DEBUG_MONITORS_STARTING, "arm64/debug_monitors:starting",
'debug_monitors_init()' is only called via 'postcore_initcall'. It can be marked as __init to save a few bytes of memory. Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- arch/arm64/kernel/debug-monitors.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)