RFR: 8264393: JDK-8258284 introduced dangling TLH race [v3]

Sat Apr 3 19:05:34 UTC 2021

On Sat, 3 Apr 2021 14:52:29 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

>> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   dholmes CR - fix nit in assert mesg.
>
> Not sure how I missed this race earlier. I never originally intended for hazard pointers to be set when exiting nested ThreadsListHandles. Anyway - the problem is understood and the fix looks good.

@fisk - Thanks for the review. The fault is mine from when I
worked on JDK-8258284. I even added the new test that
verified the improper behavior. In any case, it's fixed now and
comments are added that should prevent any of us that play
with this code from making the same mistake again.

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

PR: https://git.openjdk.java.net/jdk/pull/3272