On 2026-02-03 09:19, Jens Remus wrote:
> From: Josh Poimboeuf <jpoimboe@kernel.org>
>
> Use SYM_FUNC_{START,END} instead of all the boilerplate. No functional
> change.
>
> Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
> Signed-off-by: Jens Remus <jremus@linux.ibm.com>
> ---
>
> Notes (jremus):
> Changes in v7:
> - Rebase on H. Peter Anvin's vDSO changes on tip:x86/entry.
>
> arch/x86/entry/vdso/vdso32/system_call.S | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/arch/x86/entry/vdso/vdso32/system_call.S b/arch/x86/entry/vdso/vdso32/system_call.S
> index 9157cf9c5749..a90f4f7de396 100644
> --- a/arch/x86/entry/vdso/vdso32/system_call.S
> +++ b/arch/x86/entry/vdso/vdso32/system_call.S
> @@ -9,11 +9,7 @@
> #include <asm/alternative.h>
>
> .text
> - .globl __kernel_vsyscall
> - .type __kernel_vsyscall,@function
> - ALIGN
> -__kernel_vsyscall:
> - CFI_STARTPROC
> +SYM_FUNC_START(__kernel_vsyscall)
>
> /*
> * If using int $0x80, there is no reason to muck about with the
> @@ -85,7 +81,5 @@ SYM_INNER_LABEL(int80_landing_pad, SYM_L_GLOBAL)
> CFI_RESTORE ecx
> CFI_ADJUST_CFA_OFFSET -4
> RET
> - CFI_ENDPROC
> -
> - .size __kernel_vsyscall,.-__kernel_vsyscall
> +SYM_FUNC_END(__kernel_vsyscall)
> .previous
Looks good to me.
Acked-by: H. Peter Anvin (Intel) <hpa@zytor.com>