mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-14 01:00:07 +00:00
ea41469b7a
On ARM, where profil_counter is not static, it is brought in by references to various standard functions, as noted in <https://sourceware.org/ml/libc-alpha/2014-11/msg00890.html>, although it is not a standard function itself. I don't know if this also causes test failures on SPARC, although I see no reason for it not to do so. This patch fixes this namespace issue. profil_counter is renamed to __profil_counter and made a weak alias on ARM and SPARC. Because of the uses in profil.c / sprofil.c it seems simplest to make the rename globally, including on the other architectures for which profil_counter was static and so the change is of no substance. The variant names profil_counter_* used in sprofil.c are also renamed to start with __ so that undesired function names do not get exported in static libc. As I noted in bug 17726, profil_counter should probably be a compat symbol on ARM and SPARC, so it wouldn't exist at all in static libc even as a weak alias. Since defining a compat symbol still requires an internal name as a target of an alias, this patch still seems reasonable as an intermediate step towards that goal: it wouldn't be possible for the function simply to be static profil_counter on ARM and SPARC with profil_counter also being the exported compat symbol name, so profil.c / sprofil.c would still need to be prepared to call the function under another name (here, __profil_counter). Tested for x86_64 (testsuite, and that stripped installed shared libraries are unchanged by the patch) and ARM (ABI and linknamespace tests - this patch reduces the number of linknamespace failures I see on ARM from 227 to 5, the residue being math.h failures for fe* functions and for j0l/j1n/jnl/y0l/y1l/ynl aliases). 2014-12-17 Joseph Myers <joseph@codesourcery.com> [BZ #17725] * sysdeps/generic/profil-counter.h (profil_counter): Rename to __profil_counter. * sysdeps/unix/sysv/linux/hppa/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/i386/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/ia64/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/s390/s390-32/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/s390/s390-64/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/sh/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/tile/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/x86_64/profil-counter.h (profil_counter): Likewise. * sysdeps/unix/sysv/linux/arm/profil-counter.h (profil_counter): Likewise. [!__profil_counter] (profil_counter): Define as weak alias of __profil_counter. * sysdeps/unix/sysv/linux/sparc/sparc32/profil-counter.h (profil_counter): Rename to __profil_counter. [!__profil_counter] (profil_counter): Define as weak alias of __profil_counter. * sysdeps/unix/sysv/linux/sparc/sparc64/profil-counter.h (profil_counter): Rename to __profil_counter. [!__profil_counter] (profil_counter): Define as weak alias of __profil_counter. * sysdeps/posix/profil.c: Update comment referring to profil_counter. (__profil): Use __profil_counter instead of profil_counter. * sysdeps/posix/sprofil.c (profil_counter): Rename to __profil_counter. Use __profil_counter_ushort and __profil_counter_uint in definitions. (__sprofil): Use __profil_counter_uint and __profil_counter_ushort instead of profil_counter_uint and profil_counter_ushort. |
||
|---|---|---|
| .. | ||
| bits | ||
| net | ||
| netinet | ||
| nfs | ||
| sys | ||
| _G_config.h | ||
| _itoa.h | ||
| a.out.h | ||
| abort-instr.h | ||
| aio_misc.h | ||
| allocalim.h | ||
| asm-syntax.h | ||
| confstr.h | ||
| device-nrs.h | ||
| dirstream.h | ||
| dl-cache.h | ||
| dl-dtprocnum.h | ||
| dl-fptr.h | ||
| dl-hash.h | ||
| dl-irel.h | ||
| dl-librecon.h | ||
| dl-lookupcfg.h | ||
| dl-machine.h | ||
| dl-osinfo.h | ||
| dl-procinfo.c | ||
| dl-procinfo.h | ||
| dl-sysdep.h | ||
| dl-tls.h | ||
| dwarf2.h | ||
| elide.h | ||
| eloop-threshold.h | ||
| entry.h | ||
| errqueue.h | ||
| exit-thread.h | ||
| fd_to_filename.h | ||
| fips-private.h | ||
| fork.h | ||
| fpu_control.h | ||
| frame.h | ||
| framestate.c | ||
| gcc-compat.h | ||
| gccframe.h | ||
| get-rounding-mode.h | ||
| gmp-mparam.h | ||
| hp-timing-common.h | ||
| hp-timing.h | ||
| ifreq.h | ||
| ifunc-sel.h | ||
| intr-msg.h | ||
| inttypes.h | ||
| ldconfig.h | ||
| ldsodefs.h | ||
| libc-mmap.h | ||
| libcidn.abilist | ||
| libm-test-ulps | ||
| libnss_compat.abilist | ||
| libnss_db.abilist | ||
| libnss_dns.abilist | ||
| libnss_files.abilist | ||
| libnss_hesiod.abilist | ||
| libnss_nis.abilist | ||
| libnss_nisplus.abilist | ||
| local-setxid.h | ||
| localplt.data | ||
| machine-gmon.h | ||
| machine-lock.h | ||
| machine-sp.h | ||
| Makefile | ||
| malloc-machine.h | ||
| malloc-sysdep.h | ||
| math_ldbl_opt.h | ||
| math_ldbl.h | ||
| math_private.h | ||
| math-tests.h | ||
| memcopy.h | ||
| memusage.h | ||
| not-cancel.h | ||
| nscd-types.h | ||
| pagecopy.h | ||
| paths.h | ||
| profil-counter.h | ||
| pty-private.h | ||
| register-dump.h | ||
| rtld-lowlevel.h | ||
| safe-fatal.h | ||
| sigcontextinfo.h | ||
| siglist.h | ||
| sigset-cvt-mask.h | ||
| stackguard-macros.h | ||
| stackinfo.h | ||
| stdint.h | ||
| symbol-hacks.h | ||
| sysdep-cancel.h | ||
| sysdep.h | ||
| testrtsig.h | ||
| thread_state.h | ||
| tininess.h | ||
| tls-macros.h | ||
| tls.h | ||
| tst-audit.h | ||
| tst-stack-align.h | ||
| unsecvars.h | ||
| unwind-dw2-fde-glibc.c | ||
| unwind-dw2-fde.c | ||
| unwind-dw2-fde.h | ||
| unwind-dw2.c | ||
| unwind-pe.c | ||
| unwind-pe.h | ||
| unwind.h | ||
| utmp-equal.h | ||