summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2014-05-14 00:41:20 +0000
committerWill Newton <will.newton@linaro.org>2014-06-06 11:45:35 +0100
commit73f1cf2c59ccde948b94d27bb9f2c734f563e265 (patch)
treec1a847576dcaae10792e1d025cff1bd69d148319
parentf987b7bf3769e1e2191017a5efd0a19e1c8bace4 (diff)
Clean up ARM old-ABI symbol versioning relics.
This patch cleans up some symbol versioning code in the ARM port that exists only as relics of the old-ABI port, which was removed some time ago. The minimum symbol version in the ARM port is GLIBC_2.4 (the version where the EABI port was introduced). Thus, any SHLIB_COMPAT conditionals where the later version is 2.4 or later are obsolete and can be removed. In addition, there is no need to set symbol versions before 2.4 explicitly if the symbols would have a version of 2.4 by default anyway. This includes most of the entries in sysdeps/unix/sysv/linux/arm/Versions: those for GLIBC_2.0 are for libgcc unwind functions that aren't actually in ARM EABI glibc at all, while those for GLIBC_2.2 and GLIBC_2.3.3 are for functions which for the old-ABI port may have had versions different from the architecture-independent default, but where for EABI the default suffices (both the default and the version in that file map to 2.4, so the entries in that file do nothing). The GLIBC_2.1 entries are needed (architecture-specific functions), but it seems less confusing for those to say GLIBC_2.4, as the actual version those symbols in fact have. Various cases in the <fenv.h> functions where a function is defined as __fe* with an fe* versioned alias are cleaned up just to define fe* directly, as done e.g. on AArch64. If in future we actually need an __fe* name for use from C90 functions in libm as discussed recently, of course we can add one on all architectures and make the fe* name into a weak alias for that particular function, but for now the __fe* names aren't needed. In the case of posix_fadvise64, the __posix_fadvise64_l64 name and posix_fadvise64 alias are kept as __posix_fadvise64_l64 is used in posix_fadvise. (For that to be a namespace-clean use, posix_fadvise64 needs to be a *weak* alias not a strong one as at present, but that's an independent preexisting bug.) (There remain references to GLIBC_2_2 in sysdeps/unix/sysv/linux/arm/{msgctl.c,semctl.c,shmctl.c}. As those files are used by alpha which has a genuine 2.2 version for those functions, I think those references need to stay as-is.) Tested that the disassembly of installed shared libraries is unchanged by this patch (though function names shown in disassembly change to no longer have @@GLIBC_2.4, now those functions get versioned only by the version map and not redundantly at assembler time) and that the ABI tests pass. * sysdeps/arm/fclrexcpt.c (__feclearexcept): Rename to feclearexcept. Remove symbol versioning code. * sysdeps/arm/fegetenv.c (__fegetenv): Rename to fegetenv. Remove symbol versioning code. * sysdeps/arm/fesetenv.c (__fesetenv): Rename to fesetenv. Remove symbol versioning code. * sysdeps/arm/feupdateenv.c (__feupdateenv): Rename to feupdateenv. Remove symbol versioning code. * sysdeps/arm/fgetexcptflg.c (__fegetexceptflag): Rename to fegetexceptflag. Remove symbol versioning code. * sysdeps/arm/fsetexcptflg.c (__fesetexceptflag): Rename to fesetexceptflag. Remove symbol versioning code. * sysdeps/unix/sysv/linux/arm/Versions (libc): Remove GLIBC_2.0, GLIBC_2.2 and GLIBC_2.3.3 entries. Change GLIBC_2.1 to GLIBC_2.4. * sysdeps/unix/sysv/linux/arm/posix_fadvise64.c (__posix_fadvise64_l32): Remove prototype. [SHLIB_COMPAT(libc, GLIBC_2_2, GLIBC_2_3_3)]: Remove conditional code.
-rw-r--r--libc/ports/ChangeLog.arm.linaro21
-rw-r--r--libc/ports/sysdeps/arm/fclrexcpt.c12
-rw-r--r--libc/ports/sysdeps/arm/fegetenv.c12
-rw-r--r--libc/ports/sysdeps/arm/fesetenv.c6
-rw-r--r--libc/ports/sysdeps/arm/feupdateenv.c12
-rw-r--r--libc/ports/sysdeps/arm/fgetexcptflg.c9
-rw-r--r--libc/ports/sysdeps/arm/fsetexcptflg.c10
-rw-r--r--libc/ports/sysdeps/unix/sysv/linux/arm/Versions32
-rw-r--r--libc/ports/sysdeps/unix/sysv/linux/arm/posix_fadvise64.c17
9 files changed, 32 insertions, 99 deletions
diff --git a/libc/ports/ChangeLog.arm.linaro b/libc/ports/ChangeLog.arm.linaro
index 2a2f50ba4..e2b966459 100644
--- a/libc/ports/ChangeLog.arm.linaro
+++ b/libc/ports/ChangeLog.arm.linaro
@@ -1,3 +1,24 @@
+2014-05-14 Joseph Myers <joseph@codesourcery.com>
+
+ * sysdeps/arm/fclrexcpt.c (__feclearexcept): Rename to
+ feclearexcept. Remove symbol versioning code.
+ * sysdeps/arm/fegetenv.c (__fegetenv): Rename to fegetenv. Remove
+ symbol versioning code.
+ * sysdeps/arm/fesetenv.c (__fesetenv): Rename to fesetenv. Remove
+ symbol versioning code.
+ * sysdeps/arm/feupdateenv.c (__feupdateenv): Rename to
+ feupdateenv. Remove symbol versioning code.
+ * sysdeps/arm/fgetexcptflg.c (__fegetexceptflag): Rename to
+ fegetexceptflag. Remove symbol versioning code.
+ * sysdeps/arm/fsetexcptflg.c (__fesetexceptflag): Rename to
+ fesetexceptflag. Remove symbol versioning code.
+ * sysdeps/unix/sysv/linux/arm/Versions (libc): Remove GLIBC_2.0,
+ GLIBC_2.2 and GLIBC_2.3.3 entries. Change GLIBC_2.1 to GLIBC_2.4.
+ * sysdeps/unix/sysv/linux/arm/posix_fadvise64.c
+ (__posix_fadvise64_l32): Remove prototype.
+ [SHLIB_COMPAT(libc, GLIBC_2_2, GLIBC_2_3_3)]: Remove conditional
+ code.
+
2014-04-28 Wilco <wdijkstr@arm.com>
* sysdeps/arm/fenv_private.h: New file.
diff --git a/libc/ports/sysdeps/arm/fclrexcpt.c b/libc/ports/sysdeps/arm/fclrexcpt.c
index 8b54114e3..72eaab98d 100644
--- a/libc/ports/sysdeps/arm/fclrexcpt.c
+++ b/libc/ports/sysdeps/arm/fclrexcpt.c
@@ -22,7 +22,7 @@
int
-__feclearexcept (int excepts)
+feclearexcept (int excepts)
{
if (ARM_HAVE_VFP)
{
@@ -47,12 +47,4 @@ __feclearexcept (int excepts)
/* Unsupported, so fail unless nothing needs to be done. */
return (excepts != 0);
}
-
-#include <shlib-compat.h>
-#if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2)
-strong_alias (__feclearexcept, __old_feclearexcept)
-compat_symbol (libm, __old_feclearexcept, feclearexcept, GLIBC_2_1);
-#endif
-
-libm_hidden_ver (__feclearexcept, feclearexcept)
-versioned_symbol (libm, __feclearexcept, feclearexcept, GLIBC_2_2);
+libm_hidden_def (feclearexcept)
diff --git a/libc/ports/sysdeps/arm/fegetenv.c b/libc/ports/sysdeps/arm/fegetenv.c
index 7003a0130..a45c1af3c 100644
--- a/libc/ports/sysdeps/arm/fegetenv.c
+++ b/libc/ports/sysdeps/arm/fegetenv.c
@@ -22,7 +22,7 @@
int
-__fegetenv (fenv_t *envp)
+fegetenv (fenv_t *envp)
{
if (ARM_HAVE_VFP)
{
@@ -37,12 +37,4 @@ __fegetenv (fenv_t *envp)
/* Unsupported, so fail. */
return 1;
}
-
-#include <shlib-compat.h>
-#if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2)
-strong_alias (__fegetenv, __old_fegetenv)
-compat_symbol (libm, __old_fegetenv, fegetenv, GLIBC_2_1);
-#endif
-
-libm_hidden_ver (__fegetenv, fegetenv)
-versioned_symbol (libm, __fegetenv, fegetenv, GLIBC_2_2);
+libm_hidden_def (fegetenv)
diff --git a/libc/ports/sysdeps/arm/fesetenv.c b/libc/ports/sysdeps/arm/fesetenv.c
index af4f25d47..dabb88bb6 100644
--- a/libc/ports/sysdeps/arm/fesetenv.c
+++ b/libc/ports/sysdeps/arm/fesetenv.c
@@ -22,7 +22,7 @@
int
-__fesetenv (const fenv_t *envp)
+fesetenv (const fenv_t *envp)
{
if (ARM_HAVE_VFP)
{
@@ -58,6 +58,4 @@ __fesetenv (const fenv_t *envp)
return 1;
}
-#include <shlib-compat.h>
-libm_hidden_ver (__fesetenv, fesetenv)
-versioned_symbol (libm, __fesetenv, fesetenv, GLIBC_2_2);
+libm_hidden_def (fesetenv)
diff --git a/libc/ports/sysdeps/arm/feupdateenv.c b/libc/ports/sysdeps/arm/feupdateenv.c
index 58ec5f66d..f5deb60ad 100644
--- a/libc/ports/sysdeps/arm/feupdateenv.c
+++ b/libc/ports/sysdeps/arm/feupdateenv.c
@@ -23,7 +23,7 @@
int
-__feupdateenv (const fenv_t *envp)
+feupdateenv (const fenv_t *envp)
{
if (ARM_HAVE_VFP)
{
@@ -45,12 +45,4 @@ __feupdateenv (const fenv_t *envp)
/* Unsupported, so fail. */
return 1;
}
-
-#include <shlib-compat.h>
-#if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2)
-strong_alias (__feupdateenv, __old_feupdateenv)
-compat_symbol (libm, __old_feupdateenv, feupdateenv, GLIBC_2_1);
-#endif
-
-libm_hidden_ver (__feupdateenv, feupdateenv)
-versioned_symbol (libm, __feupdateenv, feupdateenv, GLIBC_2_2);
+libm_hidden_def (feupdateenv)
diff --git a/libc/ports/sysdeps/arm/fgetexcptflg.c b/libc/ports/sysdeps/arm/fgetexcptflg.c
index 114597990..beb43be74 100644
--- a/libc/ports/sysdeps/arm/fgetexcptflg.c
+++ b/libc/ports/sysdeps/arm/fgetexcptflg.c
@@ -23,7 +23,7 @@
int
-__fegetexceptflag (fexcept_t *flagp, int excepts)
+fegetexceptflag (fexcept_t *flagp, int excepts)
{
if (ARM_HAVE_VFP)
{
@@ -41,10 +41,3 @@ __fegetexceptflag (fexcept_t *flagp, int excepts)
/* Unsupported, so fail. */
return 1;
}
-
-#include <shlib-compat.h>
-#if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2)
-strong_alias (__fegetexceptflag, __old_fegetexceptflag)
-compat_symbol (libm, __old_fegetexceptflag, fegetexceptflag, GLIBC_2_1);
-#endif
-versioned_symbol (libm, __fegetexceptflag, fegetexceptflag, GLIBC_2_2);
diff --git a/libc/ports/sysdeps/arm/fsetexcptflg.c b/libc/ports/sysdeps/arm/fsetexcptflg.c
index 0c88c0fa7..7e3d007e9 100644
--- a/libc/ports/sysdeps/arm/fsetexcptflg.c
+++ b/libc/ports/sysdeps/arm/fsetexcptflg.c
@@ -23,7 +23,7 @@
int
-__fesetexceptflag (const fexcept_t *flagp, int excepts)
+fesetexceptflag (const fexcept_t *flagp, int excepts)
{
if (ARM_HAVE_VFP)
{
@@ -46,11 +46,3 @@ __fesetexceptflag (const fexcept_t *flagp, int excepts)
/* Unsupported, so fail unless nothing needs to be done. */
return (excepts != 0);
}
-
-#include <shlib-compat.h>
-#if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2)
-strong_alias (__fesetexceptflag, __old_fesetexceptflag)
-compat_symbol (libm, __old_fesetexceptflag, fesetexceptflag, GLIBC_2_1);
-#endif
-
-versioned_symbol (libm, __fesetexceptflag, fesetexceptflag, GLIBC_2_2);
diff --git a/libc/ports/sysdeps/unix/sysv/linux/arm/Versions b/libc/ports/sysdeps/unix/sysv/linux/arm/Versions
index 1d9e96464..ade9e9164 100644
--- a/libc/ports/sysdeps/unix/sysv/linux/arm/Versions
+++ b/libc/ports/sysdeps/unix/sysv/linux/arm/Versions
@@ -1,39 +1,9 @@
libc {
- GLIBC_2.0 {
- # Exception handling support functions from libgcc
- __register_frame; __register_frame_table; __deregister_frame;
- __frame_state_for; __register_frame_info_table;
- }
- GLIBC_2.1 {
+ GLIBC_2.4 {
ioperm; iopl;
inb; inw; inl;
outb; outw; outl;
}
- GLIBC_2.2 {
- # functions used in other libraries
- __xstat64; __fxstat64; __lxstat64;
-
- # a*
- alphasort64;
-
- # g*
- glob64;
-
- # New rlimit interface
- getrlimit; setrlimit; getrlimit64;
-
- # r*
- readdir64; readdir64_r;
-
- # s*
- scandir64;
-
- # v*
- versionsort64;
- }
- GLIBC_2.3.3 {
- posix_fadvise64; posix_fallocate64;
- }
GLIBC_2.11 {
fallocate64;
}
diff --git a/libc/ports/sysdeps/unix/sysv/linux/arm/posix_fadvise64.c b/libc/ports/sysdeps/unix/sysv/linux/arm/posix_fadvise64.c
index 7c14eec8f..fc61c7904 100644
--- a/libc/ports/sysdeps/unix/sysv/linux/arm/posix_fadvise64.c
+++ b/libc/ports/sysdeps/unix/sysv/linux/arm/posix_fadvise64.c
@@ -20,7 +20,6 @@
#include <sysdep.h>
int __posix_fadvise64_l64 (int fd, off64_t offset, off64_t len, int advise);
-int __posix_fadvise64_l32 (int fd, off64_t offset, size_t len, int advise);
/* Advice the system about the expected behaviour of the application with
respect to the file associated with FD. */
@@ -36,20 +35,4 @@ __posix_fadvise64_l64 (int fd, off64_t offset, off64_t len, int advise)
return 0;
return INTERNAL_SYSCALL_ERRNO (ret, err);
}
-
-#include <shlib-compat.h>
-
-#if SHLIB_COMPAT(libc, GLIBC_2_2, GLIBC_2_3_3)
-
-int
-attribute_compat_text_section
-__posix_fadvise64_l32 (int fd, off64_t offset, size_t len, int advise)
-{
- return __posix_fadvise64_l64 (fd, offset, len, advise);
-}
-
-versioned_symbol (libc, __posix_fadvise64_l64, posix_fadvise64, GLIBC_2_3_3);
-compat_symbol (libc, __posix_fadvise64_l32, posix_fadvise64, GLIBC_2_2);
-#else
strong_alias (__posix_fadvise64_l64, posix_fadvise64);
-#endif