diff options
author | Joseph Myers <joseph@codesourcery.com> | 2014-12-17 18:10:37 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2014-12-17 18:10:37 +0000 |
commit | ea41469b7ab2c8e35b1c10054f7076a95f80003a (patch) | |
tree | dea4be59a0db46f351b806421cbdeef9c4ee3de6 /sysdeps/posix/sprofil.c | |
parent | 8ac5a76a99e51a91eac9cecbe941fc05ddf11e03 (diff) | |
download | glibc-ea41469b7ab2c8e35b1c10054f7076a95f80003a.zip glibc-ea41469b7ab2c8e35b1c10054f7076a95f80003a.tar.gz glibc-ea41469b7ab2c8e35b1c10054f7076a95f80003a.tar.bz2 |
Fix profil_counter namespace (bug 17725).
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.
Diffstat (limited to 'sysdeps/posix/sprofil.c')
-rw-r--r-- | sysdeps/posix/sprofil.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/sysdeps/posix/sprofil.c b/sysdeps/posix/sprofil.c index 0e19d16..e2b7a49 100644 --- a/sysdeps/posix/sprofil.c +++ b/sysdeps/posix/sprofil.c @@ -175,17 +175,17 @@ profil_count_uint (void *pcp) profil_count (pcp, 1); } -/* Get the machine-dependent definition of `profil_counter', the signal +/* Get the machine-dependent definition of `__profil_counter', the signal handler for SIGPROF. It calls `profil_count' (above) with the PC of the interrupted code. */ -#define profil_counter profil_counter_ushort +#define __profil_counter __profil_counter_ushort #define profil_count(pc) profil_count (pc, 0) #include <profil-counter.h> -#undef profil_counter +#undef __profil_counter #undef profil_count -#define profil_counter profil_counter_uint +#define __profil_counter __profil_counter_uint #define profil_count(pc) profil_count (pc, 1) #include <profil-counter.h> @@ -334,9 +334,9 @@ __sprofil (struct prof *profp, int profcnt, struct timeval *tvp, /* Install SIGPROF handler. */ if (flags & PROF_UINT) - act.sa_handler = (sighandler_t) &profil_counter_uint; + act.sa_handler = (sighandler_t) &__profil_counter_uint; else - act.sa_handler = (sighandler_t) &profil_counter_ushort; + act.sa_handler = (sighandler_t) &__profil_counter_ushort; act.sa_flags = SA_RESTART; __sigfillset (&act.sa_mask); if (__sigaction (SIGPROF, &act, &prof_info.saved_action) < 0) |