diff options
| author | Paul Murphy <murphyp@linux.vnet.ibm.com> | 2015-10-28 17:34:31 -0500 |
|---|---|---|
| committer | Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> | 2015-11-19 16:15:42 -0200 |
| commit | 86b4939846caf2bb072bba6057e1dc3ad187c2c2 (patch) | |
| tree | 0f22e932dca29d8ecaec7e4cab9befa08053c0d2 /sysdeps/unix/sysv/linux/powerpc/elision-trylock.c | |
| parent | 79adcb58f6ca687c61939bbe695ffc8c54558d5f (diff) | |
| download | glibc-86b4939846caf2bb072bba6057e1dc3ad187c2c2.tar.gz glibc-86b4939846caf2bb072bba6057e1dc3ad187c2c2.tar.xz glibc-86b4939846caf2bb072bba6057e1dc3ad187c2c2.zip | |
powerpc: More elision improvements
__lll_trylock_elision sets the adapt_count variable too aggressively, and incorrectly on persistent aborts. Taking a cue from s390, adapt_count is only updated if the lock is locked, or a persistent failure occurs. In addition, the abort codes have been renumbered and refactored for clarity. As it stands, glibc only cares if the abort is persistent or not. All aborts are now persistent, excepting a busy lock. This includes changing _ABORT_NESTED_TRYLOCK into a persistent abort. * sysdeps/unix/sysv/linux/powerpc/elision-trylock.c (__lll_trylock_elision): Fix setting of adapt_count. * sysdeps/unix/sysv/linux/powerpc/htm.h (_ABORT_PERSISTENT): Define to clarify persistent aborts. (_ABORT_NESTED_TRYLOCK): Renumber, and make persistent. (_ABORT_SYSCALL): Renumber, and clarify definition. (_ABORT_LOCK_BUSY): Renumber, make non-persistent.
Diffstat (limited to 'sysdeps/unix/sysv/linux/powerpc/elision-trylock.c')
| -rw-r--r-- | sysdeps/unix/sysv/linux/powerpc/elision-trylock.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/sysdeps/unix/sysv/linux/powerpc/elision-trylock.c b/sysdeps/unix/sysv/linux/powerpc/elision-trylock.c index 440939cda1..6f61eba48e 100644 --- a/sysdeps/unix/sysv/linux/powerpc/elision-trylock.c +++ b/sysdeps/unix/sysv/linux/powerpc/elision-trylock.c @@ -44,8 +44,12 @@ __lll_trylock_elision (int *futex, short *adapt_count) if (*futex == 0) return 0; - /* Lock was busy. Fall back to normal locking. */ - __builtin_tabort (_ABORT_LOCK_BUSY); + /* Lock was busy. This is never a nested transaction. + End it, and set the adapt count. */ + __builtin_tend (0); + + if (aconf.skip_lock_busy > 0) + *adapt_count = aconf.skip_lock_busy; } else { @@ -57,9 +61,6 @@ __lll_trylock_elision (int *futex, short *adapt_count) if (aconf.skip_trylock_internal_abort > 0) *adapt_count = aconf.skip_trylock_internal_abort; } - - if (aconf.skip_lock_busy > 0) - *adapt_count = aconf.skip_lock_busy; } use_lock: |