[Bug nptl/19174] New: PowerPC: TLE enable pthread mutex performs poorly.

[Bug nptl/19174] New: PowerPC: TLE enable pthread mutex performs poorly.

[murphyp at linux dot vnet.ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

            Bug ID: 19174
           Summary: PowerPC: TLE enable pthread mutex performs poorly.
           Product: glibc
           Version: 2.21
            Status: NEW
          Severity: minor
          Priority: P2
         Component: nptl
          Assignee: unassigned at sourceware dot org
          Reporter: murphyp at linux dot vnet.ibm.com
                CC: drepper.fsp at gmail dot com
  Target Milestone: ---

When locking a critical section on multiple threads which contains no data
dependencies, any non-persistent abort will cause a fallback to locking causing
unnecessary serialization.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug nptl/19174] PowerPC: TLE enabled pthread mutex performs poorly.

[murphyp at linux dot vnet.ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

Paul E. Murphy <murphyp at linux dot vnet.ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|PowerPC: TLE enable pthread |PowerPC: TLE enabled
                   |mutex performs poorly.      |pthread mutex performs
                   |                            |poorly.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug nptl/19174] PowerPC: TLE enabled pthread mutex performs poorly.

[murphyp at linux dot vnet.ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

Paul E. Murphy <murphyp at linux dot vnet.ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at sourceware dot org   |murphyp at linux dot vnet.ibm.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug nptl/19174] PowerPC: TLE enabled pthread mutex performs poorly.

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  72f1463df85a522bfd1568e47bd81371522ee358 (commit)
       via  76c5ae00284d98f1994434c570bdcde9dbc6b49e (commit)
      from  8b126a4a2ca78a5d98cc7afa219d93c8a31ed944 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=72f1463df85a522bfd1568e47bd81371522ee358

commit 72f1463df85a522bfd1568e47bd81371522ee358
Author: Paul Murphy <murphyp@linux.vnet.ibm.com>
Date:   Thu Aug 27 09:48:04 2015 -0500

    powerpc: Fix usage of elision transient failure adapt param

    The skip_lock_out_of_tbegin_retries adaptive parameter was
    not being used correctly, nor as described.  This prevents
    a fallback for all users of the lock if a transient abort
    occurs within the accepted number of retries.

        [BZ #19174]
        * sysdeps/powerpc/nptl/elide.h (__elide_lock): Fix usage of
        .skip_lock_out_of_tbegin_retries.
        * sysdeps/unix/sysv/linux/powerpc/elision-lock.c
        (__lll_lock_elision): Likewise, and respect a value of
        try_tbegin <= 0.

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=76c5ae00284d98f1994434c570bdcde9dbc6b49e

commit 76c5ae00284d98f1994434c570bdcde9dbc6b49e
Author: Carlos Eduardo Seo <cseo@linux.vnet.ibm.com>
Date:   Fri Oct 9 16:01:35 2015 -0300

    Add AT_PLATFORM to _dl_aux_init ()

    Added AT_PLATFORM to _dl_aux_init () function to keep it in sync with
    _dl_sysdep_start ().

        * elf/dl-support.c (_dl_aux_init): Added AT_PLATFORM to the case
        statement.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                                      |   14 ++++++++++++++
 NEWS                                           |    2 +-
 elf/dl-support.c                               |    3 +++
 sysdeps/powerpc/nptl/elide.h                   |    6 +++---
 sysdeps/unix/sysv/linux/powerpc/elision-lock.c |   15 ++++++---------
 5 files changed, 27 insertions(+), 13 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug nptl/19174] PowerPC: TLE enabled pthread mutex performs poorly.

[joseph at codesourcery dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

--- Comment #2 from joseph at codesourcery dot com <joseph at codesourcery dot com> ---
Since you added the bug number to NEWS, I presume this commit was intended 
as a complete fix for the bug, in which case you should close it as FIXED 
with the appropriate milestone set.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug nptl/19174] PowerPC: TLE enabled pthread mutex performs poorly.

[murphyp at linux dot vnet.ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=19174

Paul E. Murphy <murphyp at linux dot vnet.ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.23

--- Comment #3 from Paul E. Murphy <murphyp at linux dot vnet.ibm.com> ---
This is now fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.