This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] elf: Add tests with a local IFUNC resolver [BZ #23937]
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Florian Weimer <fweimer at redhat dot com>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Wed, 27 Feb 2019 13:23:52 -0800
- Subject: Re: [PATCH] elf: Add tests with a local IFUNC resolver [BZ #23937]
- References: <87mumqei4h.fsf@oldenburg2.str.redhat.com>
On Wed, Feb 20, 2019 at 4:53 AM Florian Weimer <fweimer@redhat.com> wrote:
>
> The existing tests all use global symbols (but with different
> visibility). Local symbols could be treated differently by the
> compiler and linker (as was the case on POWER ELFv2, causing
> bug 23937), and we did not have test coverage for this.
>
> Tested on x86-64 and POWER ELFv2 little-endian, with and without
> --disable-multi-arch. On POWER, the test cases elf/ifuncmain9,
> elf/ifuncmain9pic, elf/ifuncmain9pie reproduce bug 23937 with older
> binutils.
>
> 2019-02-20 Florian Weimer <fweimer@redhat.com>
>
> [BZ #23937]
> elf: Add test with a local IFUNC resolver.
> * elf/ifuncmain9.c: New file.
> * elf/ifuncmain9pic.c: Likewise.
> * elf/ifuncmain9picstatic.c: Likewise.
> * elf/ifuncmain9pie.c: Likewise.
> * elf/ifuncmain9static.c: Likewise.
> * elf/Makefile [multi-arch] (tests-ifuncstatic): Add
> ifuncmain9static, ifuncmain9picstatic.
> * elf/Makefile [multi-arch && build-shared] (tests-internal):
> Add ifuncmain9, ifuncmain9pic.
> * elf/Makefile [multi-arch && build-shared && have-fpie]
> (ifunc-pie-tests): Add ifuncmain9pie.
> (CFLAGS-ifuncmain9pic.c): Add $(pic-ccflag).
> (CFLAGS-ifuncmain9picstatic.c): Likewise.
> (CFLAGS-ifuncmain9pie.c): Add $(pie-ccflag).
>
LGTM.
Thanks.
--
H.J.