RFR (XXS): 7006810: G1: Introduce peace-of-mind checking in the Suspendible Thread Set

Joseph Provino joseph.provino at oracle.com
Wed Apr 8 23:34:03 UTC 2015


latest webrev is here:

http://cr.openjdk.java.net/~jprovino/7006810/webrev.02

It's not the final version because I have asserts commented out in 
join() and leave()
that need to be figured out.  And also Per's comments about whether or not
the asserts are correct in synchronize() / desynchronize().

joe

On 4/8/2015 7:18 PM, Joseph Provino wrote:
>
> On 4/6/2015 1:42 PM, Eric Caspole wrote:
>> Hi Joe,
>> Could all the new code be in a NOT_PRODUCT() or #ifdef ASSERT block 
>> so there doesn't need to be any new methods in the product build and 
>> it is more obvious to the reader that it is a debugging helper method?
>
> I changed to use #ifdef ASSERT and DEBUG_ONLY.
>
> Thanks.
>
> joe
>
>> Thanks,
>> Eric
>>
>> On 4/6/2015 1:25 PM, Joseph Provino wrote:
>>> CR: https://bugs.openjdk.java.net/browse/JDK-7006810
>>>
>>> webrev: http://cr.openjdk.java.net/~jprovino/7006810/webrev.00
>>>
>>> test:  jprt
>>
>



More information about the hotspot-gc-dev mailing list