Re: [PATCH v2] LoongArch: vDSO: Emit GNU_EH_FRAME correctly

[Xi Ruoyao]

On Fri, 2026-02-27 at 18:04 +0800, Huacai Chen wrote:
> On Fri, Feb 27, 2026 at 5:57 PM Xi Ruoyao <xry111@xxxxxxxxxxx> wrote:
> > 
> > On Fri, 2026-02-27 at 16:58 +0800, Huacai Chen wrote:
> > 
> > /* snip */
> > 
> > > > diff --git a/arch/loongarch/include/asm/sigframe.h b/arch/loongarch/include/asm/sigframe.h
> > > > new file mode 100644
> > > > index 000000000000..59db3de6db85
> > > > --- /dev/null
> > > > +++ b/arch/loongarch/include/asm/sigframe.h
> > > > @@ -0,0 +1,12 @@
> > > > +/* SPDX-License-Identifier: GPL-2.0+ */
> > > > +/*
> > > > + * Separated from arch/loongarch/kernel/signal.c.
> > > I think this is also unnecessary.
> > 
> > Will remove this comment.
> > 
> > /* snip */
> > 
> > > > +        * This is also the reason we don't use SYM_FUNC_START.
> > > > +        */
> > > > +       nop
> > > > +SYM_START(__vdso_rt_sigreturn, SYM_L_GLOBAL, SYM_A_ALIGN)
> > > Is it possible to define SYM_SIGFUNC_START/SYM_SIGFUNC_END in
> > > linkage.h, and then use them here?
> > 
> > How about SYM_FUNC_{START,END}_NOCFI instead?
> Of course I don't know how to name them, I just want to define a
> wrapper to contain all .cfi_xxxx in this patch.

Will do it.  If they contain all the .cfi_ directives your naming is
better.

-- 
Xi Ruoyao <xry111@xxxxxxxxxxx>