RFR(S): 8152358 - code and comment cleanups found during the hunt for 8077392

[Daniel D. Daugherty]

Rachel,

Thanks for taking a look at the logging changes.

Dan

On 3/28/16 11:17 AM, Rachel Protacio wrote:
> Just to weigh in on the logging - the biasedLocking.cpp changes look 
> good to me! Thanks for fixing those spacing issues.
>
> Rachel
>
> On 3/22/2016 5:35 PM, Daniel D. Daugherty wrote:
>> Greetings,
>>
>> As a follow on to my fix for the following bugs:
>>
>>     JDK-8077392 Stream fork/join tasks occasionally fail to complete
>>     https://bugs.openjdk.java.net/browse/JDK-8077392
>>
>>     JDK-8131715 backout the fix for JDK-8079359 when JDK-8077392 is 
>> fixed
>>     https://bugs.openjdk.java.net/browse/JDK-8131715
>>
>> I have a fix for the following bug:
>>
>>     JDK-8152358 code and comment cleanups found during the hunt for 
>> 8077392
>>     https://bugs.openjdk.java.net/browse/JDK-8152358
>>
>> These cleanups are pretty straight forward so I'm not going to
>> include the usual long, detailed analysis... :-)
>>
>> Here is the webrev URL:
>>
>> http://cr.openjdk.java.net/~dcubed/8152358-webrev/0-jdk9-hs-rt/
>>
>> This webrev is a superset of the webrev for the JDK-8077392 and
>> JDK-8131715 fixes and is the code that I'm currently testing...
>>
>> Testing:
>>
>> - the original failing test is running in a parallel stress config
>>   on my Solaris X64 server; just under 23 hours and just under
>>   3000 iterations without a failure in either instance; I'm planning
>>   to let the stress run go for at least 72 hours.
>> - RT/SVC nightly equivalent (in progress)
>>
>> As always, comments, suggestions and/or questions are welcome.
>>
>> Dan
>