diff options
author | Aurelien Jarno <aurelien@aurel32.net> | 2019-01-02 18:21:18 +0100 |
---|---|---|
committer | Aurelien Jarno <aurelien@aurel32.net> | 2019-01-02 18:21:18 +0100 |
commit | fe20bb1d6084bbf38fef587b0fb33eb6257cc1ed (patch) | |
tree | 185786e7ead6ed990b21943769053f59aa4fdc71 /ChangeLog | |
parent | 2d9837c1fbf4658f199eae02681f08f040dfe3a8 (diff) | |
download | glibc-fe20bb1d6084bbf38fef587b0fb33eb6257cc1ed.tar.gz glibc-fe20bb1d6084bbf38fef587b0fb33eb6257cc1ed.tar.xz glibc-fe20bb1d6084bbf38fef587b0fb33eb6257cc1ed.zip |
ARM: fix kernel assisted atomics with GCC 8 (bug 24034)
The pre-ARMv7 CPUs are missing atomic compare and exchange and/or barrier instructions. Therefore those are implemented using kernel assistance, calling a kernel function at a specific address, and passing the arguments in the r0 to r4 registers. This is done by specifying registers for local variables. The a_ptr variable is placed in the r2 register and declared with __typeof (mem). According to the GCC documentation on local register variables, if mem is a constant pointer, the compiler may substitute the variable with its initializer in asm statements, which may cause the corresponding operand to appear in a different register. This happens in __libc_start_main with the pointer to the thread counter for static binaries (but not the shared ones): # ifdef SHARED unsigned int *ptr = __libc_pthread_functions.ptr_nthreads; # ifdef PTR_DEMANGLE PTR_DEMANGLE (ptr); # endif # else extern unsigned int __nptl_nthreads __attribute ((weak)); unsigned int *const ptr = &__nptl_nthreads; # endif This causes static binaries using threads to crash when the GNU libc is built with GCC 8 and most notably tst-cancel21-static. To fix that, use the same trick than for the volatile qualifier, defining a_ptr as a union. Changelog: [BZ #24034] * sysdeps/unix/sysv/linux/arm/atomic-machine.h (__arm_assisted_compare_and_exchange_val_32_acq): Use uint32_t rather than __typeof (...) for the a_ptr variable.
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index 745042e4fd..077e735387 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2019-01-02 Aurelien Jarno <aurelien@aurel32.net> + + [BZ #24034] + * sysdeps/unix/sysv/linux/arm/atomic-machine.h + (__arm_assisted_compare_and_exchange_val_32_acq): Use uint32_t rather + than __typeof (...) for the a_ptr variable. + 2019-01-02 Gabriel F. T. Gomes <gabriel@inconstante.eti.br> * debug/sprintf_chk.c (___sprintf_chk): Use PRINTF_CHK. |