<div dir="ltr"><div dir="ltr">On Fri, Jan 27, 2023 at 3:19 PM Fangrui Song <<a href="mailto:maskray@google.com">maskray@google.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Dec 29, 2022 at 11:22 AM Palmer Dabbelt <<a href="mailto:palmer@dabbelt.com" target="_blank">palmer@dabbelt.com</a>> wrote:<br>
><br>
> On Wed, 21 Dec 2022 15:51:47 PST (-0800), <a href="mailto:maskray@google.com" target="_blank">maskray@google.com</a> wrote:<br>
> > The actual intention is that no dynamic relocation exists. However, some<br>
> > GNU ld ports produce unneeded R_*_NONE. (If a port fails to determine<br>
> > the exact .rel[a].dyn size, the trailing zeros become R_*_NONE<br>
> > relocations. E.g. ld's powerpc port recently fixed<br>
> > <a href="https://sourceware.org/bugzilla/show_bug.cgi?id=29540" rel="noreferrer" target="_blank">https://sourceware.org/bugzilla/show_bug.cgi?id=29540</a>) R_*_NONE are<br>
> > generally no-op in the dynamic loaders. So just ignore them.<br>
> ><br>
> > With the change, we can remove ARCH_REL_TYPE_ABS. ARCH_REL_TYPE_ABS is a<br>
> > bit misnomer as ports may check RELAVETIVE/GLOB_DAT/JUMP_SLOT which are<br>
> > not called "absolute relocations". (The patch is motivated by the arm64<br>
> > port missing R_AARCH64_RELATIVE.)<br>
> ><br>
> > Signed-off-by: Fangrui Song <<a href="mailto:maskray@google.com" target="_blank">maskray@google.com</a>><br>
> > Reviewed-by: Christophe Leroy <<a href="mailto:christophe.leroy@csgroup.eu" target="_blank">christophe.leroy@csgroup.eu</a>><br>
> > ---<br>
> > Change from v1:<br>
> > * rebase after 8ac3b5cd3e0521d92f9755e90d140382fc292510 (lib/vdso: use "grep -E" instead of "egrep")<br>
> > * change the commit message to mention an example GNU ld bug; no longer say the patch fixes a deprecated egrep use<br>
> > ---<br>
> >   arch/arm/vdso/Makefile            |  3 ---<br>
> >   arch/arm64/kernel/vdso/Makefile   |  3 ---<br>
> >   arch/arm64/kernel/vdso32/Makefile |  3 ---<br>
> >   arch/csky/kernel/vdso/Makefile    |  3 ---<br>
> >   arch/loongarch/vdso/Makefile      |  3 ---<br>
> >   arch/mips/vdso/Makefile           |  3 ---<br>
> >   arch/powerpc/kernel/vdso/Makefile |  1 -<br>
> >   arch/riscv/kernel/vdso/Makefile   |  3 ---<br>
> >   arch/s390/kernel/vdso32/Makefile  |  2 --<br>
> >   arch/s390/kernel/vdso64/Makefile  |  2 --<br>
> >   arch/x86/entry/vdso/Makefile      |  4 ----<br>
> >   lib/vdso/Makefile                 | 13 ++++---------<br>
> >   12 files changed, 4 insertions(+), 39 deletions(-)<br>
><br>
> [snip]<br>
><br>
> > diff --git a/arch/riscv/kernel/vdso/Makefile b/arch/riscv/kernel/vdso/Makefile<br>
> > index 06e6b27f3bcc..d85c37e11b21 100644<br>
> > --- a/arch/riscv/kernel/vdso/Makefile<br>
> > +++ b/arch/riscv/kernel/vdso/Makefile<br>
> > @@ -1,9 +1,6 @@<br>
> >   # SPDX-License-Identifier: GPL-2.0-only<br>
> >   # Copied from arch/tile/kernel/vdso/Makefile<br>
> ><br>
> > -# Absolute relocation type $(ARCH_REL_TYPE_ABS) needs to be defined before<br>
> > -# the inclusion of generic Makefile.<br>
> > -ARCH_REL_TYPE_ABS := R_RISCV_32|R_RISCV_64|R_RISCV_JUMP_SLOT<br>
> >   include $(srctree)/lib/vdso/Makefile<br>
> >   # Symbols present in the vdso<br>
> >   vdso-syms  = rt_sigreturn<br>
><br>
> Acked-by: Palmer Dabbelt <<a href="mailto:palmer@rivosinc.com" target="_blank">palmer@rivosinc.com</a>> # RISC-V<br>
><br>
> Thanks!<br>
<br>
Looks like this patch hasn't been picked yet...<br>
</blockquote></div><br clear="all"><div>Ping:)</div><div><br></div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr">宋方睿</div></div></div>