entry: Correct 'noinstr' attributes
authorIngo Molnar <mingo@kernel.org>
Sat, 25 Jul 2020 09:19:51 +0000 (11:19 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Sun, 26 Jul 2020 13:42:20 +0000 (15:42 +0200)
The noinstr attribute is to be specified before the return type in the
same way 'inline' is used.

Similar cases were recently fixed for x86 in commit 7f6fa101dfac ("x86:
Correct noinstr qualifiers"), but the generic entry code was based on the
the original version and did not carry the fix over.

Fixes: a5497bab5f72 ("entry: Provide generic interrupt entry/exit code")
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20200725091951.744848-3-mingo@kernel.org
kernel/entry/common.c

index 495f5c051b03de8fe4422695aeff1cc42f1d86fd..9852e0d62d95ccd572c4fa648ee6da1063afbedd 100644 (file)
@@ -256,7 +256,7 @@ noinstr void irqentry_exit_to_user_mode(struct pt_regs *regs)
        exit_to_user_mode();
 }
 
-irqentry_state_t noinstr irqentry_enter(struct pt_regs *regs)
+noinstr irqentry_state_t irqentry_enter(struct pt_regs *regs)
 {
        irqentry_state_t ret = {
                .exit_rcu = false,
@@ -333,7 +333,7 @@ void irqentry_exit_cond_resched(void)
        }
 }
 
-void noinstr irqentry_exit(struct pt_regs *regs, irqentry_state_t state)
+noinstr void irqentry_exit(struct pt_regs *regs, irqentry_state_t state)
 {
        lockdep_assert_irqs_disabled();