Re: [PATCH v5] Add and use new glibc-internal futex API.

[Roland McGrath <roland@hack.frob.com>]

> This works so far:
> grep -r futex_ | grep '((unsigned int'

OK.

> I believe we can commit this early next week, so you can get your
> testing done.  I also want to give David some time to test on sparc too.

NaCl testing is all happy.

> > It wouldn't hurt to write up somewhere, probably on the wiki (though in
> > code comments in this patch would also be fine by me) the known/intended
> > further steps of transition and cleanup in this area.
> 
> I added a few notes here: https://sourceware.org/glibc/wiki/Concurrency

Good.

> I've removed the changes to nptl/cancellation.c from this patch because
> I was running into build failures on i686-linux, which happen when calls
> to __libc_fatal() or abort() exist in the code.  This seems to happen
> when IS_IN (libc) is true.  I don't know why that happens but it seems
> orthogonal to the futex API.  Therefore, to move forward with the futex
> API, I choose to simply drop the respective changes from the patch.
> 
> Roland, do you have any suggestion how this could be fixed?  (You argued
> for calling __libc_fatal there ;)  Or anyone else?

I'll look into it.


Thanks,
Roland