On 2026-02-03 09:19, Jens Remus wrote: > From: Josh Poimboeuf <[email protected]> > > Use SYM_FUNC_{START,END} instead of all the boilerplate. No functional > change. > > Signed-off-by: Josh Poimboeuf <[email protected]> > Signed-off-by: Steven Rostedt (Google) <[email protected]> > Signed-off-by: Jens Remus <[email protected]> > --- > > 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) <[email protected]>
