mirror of
https://sourceware.org/git/glibc.git
synced 2024-12-04 02:40:06 +00:00
e921c89e01
Some SPE opcodes clashes with some recent PowerISA opcodes and until recently gas did not complain about it. However binutils recently changed it and now VLE configured gas does not support to assembler some instruction that might class with VLE (HTM for instance). It also does not help that glibc build hardware lock elision support as default (regardless of assembler support). Although runtime will not actually enables TLE on SPE hardware (since kernel will not advertise it), I see little advantage on adding HTM support on SPE built glibc. SPE uses an incompatible ABI which does not allow share the same build with default powerpc and HTM code slows down SPE without any benefict. This patch fixes it by only building HTM when SPE configuration is not used. Checked with a powerpc-linux-gnuspe build. I also did some sniff tests on a e500 hardware without any issue. [BZ #22926] * sysdeps/powerpc/powerpc32/sysdep.h (ABORT_TRANSACTION_IMPL): Define empty for __SPE__. * sysdeps/powerpc/sysdep.h (ABORT_TRANSACTION): Likewise. * sysdeps/unix/sysv/linux/powerpc/elision-lock.c (__lll_lock_elision): Do not build hardware transactional code for __SPE__. * sysdeps/unix/sysv/linux/powerpc/elision-trylock.c (__lll_trylock_elision): Likewise. * sysdeps/unix/sysv/linux/powerpc/elision-unlock.c (__lll_unlock_elision): Likewise.
48 lines
1.6 KiB
C
48 lines
1.6 KiB
C
/* elision-unlock.c: Commit an elided pthread lock.
|
|
Copyright (C) 2015-2018 Free Software Foundation, Inc.
|
|
This file is part of the GNU C Library.
|
|
|
|
The GNU C Library is free software; you can redistribute it and/or
|
|
modify it under the terms of the GNU Lesser General Public
|
|
License as published by the Free Software Foundation; either
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
The GNU C Library is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
Lesser General Public License for more details.
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
License along with the GNU C Library; if not, see
|
|
<http://www.gnu.org/licenses/>. */
|
|
|
|
#include "pthreadP.h"
|
|
#include "lowlevellock.h"
|
|
#include "htm.h"
|
|
|
|
int
|
|
__lll_unlock_elision (int *lock, short *adapt_count, int pshared)
|
|
{
|
|
#ifndef __SPE__
|
|
/* When the lock was free we're in a transaction. */
|
|
if (*lock == 0)
|
|
__libc_tend (0);
|
|
else
|
|
{
|
|
/* Update adapt_count in the critical section to prevent a
|
|
write-after-destroy error as mentioned in BZ 20822. The
|
|
following update of adapt_count has to be contained within
|
|
the critical region of the fall-back lock in order to not violate
|
|
the mutex destruction requirements. */
|
|
short __tmp = atomic_load_relaxed (adapt_count);
|
|
if (__tmp > 0)
|
|
atomic_store_relaxed (adapt_count, __tmp - 1);
|
|
|
|
lll_unlock ((*lock), pshared);
|
|
}
|
|
#else
|
|
lll_unlock ((*lock), pshared);
|
|
#endif
|
|
return 0;
|
|
}
|