[PATCH 2/9] Add the low level infrastructure for pthreads lock elision with TSX

Dominik Vogt vogt@linux.vnet.ibm.com
Mon May 6 12:44:00 GMT 2013


In __lll_lock_elision() there is this comment:

> +             /* Lock was busy. Fall back to normal locking.
> +                Could also _xend here but xabort with 0xff code
> +                is more visible in the profiler. */
> +             _xabort (_ABORT_LOCK_BUSY);

The code is certainly correct here, but I guess the comment is
not.  I think you could _not_ use _xend() here because the
transactions might be nested, and _xend() would end only the
innermost transaction level and not abort the outermost
transaction.  At least if I understand Intel transactions
correctly.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany



More information about the Libc-alpha mailing list