This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug dynamic-link/21265] _dl_runtime_resolve isn't compatible with Intel C++ __regcall calling convention

--- Comment #23 from cvs-commit at gcc dot <cvs-commit at gcc dot> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, release/2.26/master has been updated
       via  f82a6fc223cbd890b9de9007cfce63e6cae8f799 (commit)
      from  b2c78ae69eb5845c94db94e87a2addd695f978c0 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------;h=f82a6fc223cbd890b9de9007cfce63e6cae8f799

commit f82a6fc223cbd890b9de9007cfce63e6cae8f799
Author: H.J. Lu <>
Date:   Sun Oct 22 07:40:39 2017 -0700

    x86-64: Use fxsave/xsave/xsavec in _dl_runtime_resolve [BZ #21265]

    In _dl_runtime_resolve, use fxsave/xsave/xsavec to preserve all vector,
    mask and bound registers.  It simplifies _dl_runtime_resolve and supports
    different calling conventions. code size is reduced by more than
    1 KB.  However, use fxsave/xsave/xsavec takes a little bit more cycles
    than saving and restoring vector and bound registers individually.

    Latency for _dl_runtime_resolve to lookup the function, foo, from one
    shared library plus

                                 Before    After     Change

    Westmere (SSE)/fxsave         345      866       151%
    IvyBridge (AVX)/xsave         420      643       53%
    Haswell (AVX)/xsave           713      1252      75%
    Skylake (AVX+MPX)/xsavec      559      719       28%
    Skylake (AVX512+MPX)/xsavec   145      272       87%
    Ryzen (AVX)/xsavec            280      553       97%

    This is the worst case where portion of time spent for saving and
    restoring registers is bigger than majority of cases.  With smaller
    _dl_runtime_resolve code size, overall performance impact is negligible.

    On IvyBridge, differences in build and test time of binutils with lazy
    binding GCC and binutils are noises.  On Westmere, differences in
    bootstrap and "makc check" time of GCC 7 with lazy binding GCC and
    binutils are also noises.

        [BZ #21265]
        * sysdeps/x86/cpu-features-offsets.sym (XSAVE_STATE_SIZE_OFFSET):
        * sysdeps/x86/cpu-features.c: Include <libc-pointer-arith.h>.
        (get_common_indeces): Set xsave_state_size, xsave_state_full_size
        and bit_arch_XSAVEC_Usable if needed.
        (init_cpu_features): Remove bit_arch_Use_dl_runtime_resolve_slow
        and bit_arch_Use_dl_runtime_resolve_opt.
        * sysdeps/x86/cpu-features.h (bit_arch_Use_dl_runtime_resolve_opt):
        (bit_arch_Use_dl_runtime_resolve_slow): Likewise.
        (bit_arch_Prefer_No_AVX512): Updated.
        (bit_arch_MathVec_Prefer_No_AVX512): Likewise.
        (bit_arch_XSAVEC_Usable): New.
        (STATE_SAVE_OFFSET): Likewise.
        (STATE_SAVE_MASK): Likewise.
        [__ASSEMBLER__]: Include <cpu-features-offsets.h>.
        (cpu_features): Add xsave_state_size and xsave_state_full_size.
        (index_arch_Use_dl_runtime_resolve_opt): Removed.
        (index_arch_Use_dl_runtime_resolve_slow): Likewise.
        (index_arch_XSAVEC_Usable): New.
        * sysdeps/x86/cpu-tunables.c (TUNABLE_CALLBACK (set_hwcaps)):
        Support XSAVEC_Usable.  Remove Use_dl_runtime_resolve_slow.
        * sysdeps/x86_64/Makefile (tst-x86_64-1-ENV): New if tunables
        is enabled.
        * sysdeps/x86_64/dl-machine.h (elf_machine_runtime_setup):
        Replace _dl_runtime_resolve_sse, _dl_runtime_resolve_avx,
        _dl_runtime_resolve_avx_slow, _dl_runtime_resolve_avx_opt,
        _dl_runtime_resolve_avx512 and _dl_runtime_resolve_avx512_opt
        with _dl_runtime_resolve_fxsave, _dl_runtime_resolve_xsave and
        * sysdeps/x86_64/dl-trampoline.S (DL_RUNTIME_UNALIGNED_VEC_SIZE):
        instead of VEC_SIZE.
        (REGISTER_SAVE_BND0): Removed.
        (REGISTER_SAVE_BND1): Likewise.
        (REGISTER_SAVE_BND3): Likewise.
        (REGISTER_SAVE_RAX): Always defined to 0.
        (VMOV): Removed.
        (_dl_runtime_resolve_avx): Likewise.
        (_dl_runtime_resolve_avx_slow): Likewise.
        (_dl_runtime_resolve_avx_opt): Likewise.
        (_dl_runtime_resolve_avx512): Likewise.
        (_dl_runtime_resolve_avx512_opt): Likewise.
        (_dl_runtime_resolve_sse): Likewise.
        (_dl_runtime_resolve_sse_vex): Likewise.
        (USE_FXSAVE): New.
        (_dl_runtime_resolve_fxsave): Likewise.
        (USE_XSAVE): Likewise.
        (_dl_runtime_resolve_xsave): Likewise.
        (USE_XSAVEC): Likewise.
        (_dl_runtime_resolve_xsavec): Likewise.
        * sysdeps/x86_64/dl-trampoline.h (_dl_runtime_resolve_avx512):
        (_dl_runtime_resolve_avx512_opt): Likewise.
        (_dl_runtime_resolve_avx): Likewise.
        (_dl_runtime_resolve_avx_opt): Likewise.
        (_dl_runtime_resolve_sse): Likewise.
        (_dl_runtime_resolve_sse_vex): Likewise.
        (_dl_runtime_resolve_fxsave): New.
        (_dl_runtime_resolve_xsave): Likewise.
        (_dl_runtime_resolve_xsavec): Likewise.

    (cherry picked from commit b52b0d793dcb226ecb0ecca1e672ca265973233c)


Summary of changes:
 ChangeLog                            |   66 +++++++++
 NEWS                                 |    1 +
 sysdeps/x86/cpu-features-offsets.sym |    1 +
 sysdeps/x86/cpu-features.c           |   88 +++++++++--
 sysdeps/x86/cpu-features.h           |   31 +++-
 sysdeps/x86/cpu-tunables.c           |   17 ++-
 sysdeps/x86_64/Makefile              |    4 +
 sysdeps/x86_64/dl-machine.h          |   38 ++----
 sysdeps/x86_64/dl-trampoline.S       |   87 ++++--------
 sysdeps/x86_64/dl-trampoline.h       |  267 ++++++++++------------------------
 10 files changed, 294 insertions(+), 306 deletions(-)

You are receiving this mail because:
You are on the CC list for the bug.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]