This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PR18457] Don't require rtld lock to compute DTV addr for static TLS


On Wed, 2015-06-03 at 17:31 -0300, Alexandre Oliva wrote:
> On Jun  3, 2015, Alexandre Oliva <aoliva@redhat.com> wrote:
> 
> > How's this?
> 
> Or rather this, that I got after updating the patch file :-)

Why did you remove the atomic accesses altogether?  I argued that we
need *more*, not less.  You can't just pretend those issues don't exist.
You made good progress towards a consistent fix with the reasoning on
synchronization you provided for the static case, and the information
you provided for the non-static case.

This doesn't add any of the documentation I want to see either.  Not OK.

> (the testcase is adjusted to use atomics)

Test case is OK.  Thanks.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]