RFR: 8156500: deadlock provoked by new stress test com/sun/jdi/OomDebugTest.java

[Kim Barrett]

> On Aug 24, 2016, at 9:26 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> Hi Kim,
> 
> Sorry this fell off the radar …

That’s okay.  There was a long gap while I was working on other things.

> On 24/08/2016 7:09 AM, Kim Barrett wrote:
>> Mandy asked for some additional comments to make life easier for
>> future readers.  I've also stopped pretending the description for
>> waitForReferenceProcessing is javadoc, since this function is private
>> and only accessible via SharedSecrets or reflection that avoids access
>> control.
>> 
>> Changes only in the jdk tree:
>> 
>> New webrevs:
>> full: http://cr.openjdk.java.net/~kbarrett/8156500/jdk.05/
>> incr: http://cr.openjdk.java.net/~kbarrett/8156500/jdk.05.inc/
> 
> Just to be clear, in waitForReferenceProcessing() you are okay with returning true on spurious wakeup?

Yes.

>> Unchanged hotspot webrevs:
>> full: http://cr.openjdk.java.net/~kbarrett/8156500/hotspot.04/
>> incr: http://cr.openjdk.java.net/~kbarrett/8156500/hotspot.04.inc/
> 
> No further comments from me.
> 
> Thanks,
> David