Re: [PATCH v2 0/2] riscv: add support for GNU indirect function

[Vincent Chen <vincent.chen@sifive.com>]

On Tue, Dec 15, 2020 at 5:16 PM Vincent Chen <vincent.chen@sifive.com> wrote:
>
> The riscv Binutils supported GNU indirect function in the commit
> 02dd9d25. To make the entire mechanism work, these patches adds the
> required ports to enable Glibc to address the IRELATIVE relocation.
>
> Because the dynamic linker resolves the IRELATIVE relocation before
> jumping into the executable program, the $gp register is not initialized
> when executing the IFUNC resolver function. It may cause some unexpected
> error to occur when the dynamic loader resolves the IREALTIVE relocation
> of the position-dependent executable program. To solve this issue, the
> second patch enables dynamic linker to look for the __global_pointer$
> symbol and use it to initialize $gp before resolving the IREALTIVE
> relocation of the PDE program.
>
>
> Changes since v1:
> 1. Move the riscv entry to the end of libc-abis file
> 2. Add the second input argument to IFUNC resolver to preserve the
>    extension flexibility
> 3. Enable ld.so to initilize $gp by __global_pointer$ symbol instead
>    of the DT_RISCV_GP dynamic entry to avoid to modify the riscv ELF
>    psABI specification.
>

Gentle ping on this series. Thanks and happy new year.

>
> Vincent Chen (2):
>   riscv: support GNU indirect function
>   riscv: Initialize $gp before resolving the IRELATIVE relocation
>
>  libc-abis                  |  1 +
>  sysdeps/riscv/dl-irel.h    | 56 ++++++++++++++++++++++++++++++++++++++
>  sysdeps/riscv/dl-machine.h | 44 +++++++++++++++++++++++++++++-
>  3 files changed, 100 insertions(+), 1 deletion(-)
>  create mode 100644 sysdeps/riscv/dl-irel.h
>
> --
> 2.29.2
>