RFR: 8210237: gc/stress/TestStressIHOPMultiThread.java fails with 'Unexpected exit from test [exit code: 1]' in CDS mode

[Jiangli Zhou]

Hi Gerard,

Thanks for the review! The assumption (can allocate up to the specified 
amount under low memory conditions) made by the test for all different 
configurations may not always be true and OutOfMemoryError could happen 
with some scenarios. The fix is to handle possible OutOfMemoryError in 
the test. The test is intended for IHOP testing with logging output. 
Adding the OOM handling does not seem to change the original intention. 
Sorry, I should have included more details in the RFR request.

Thanks,

Jiangli


On 9/14/18 11:55 AM, Gerard Ziemski wrote:
> hi Jiangli,
>
> So the fix here is to provide more diagnostic info, so that it can be later used to determine the real issue?
>
> Looks good if that’s the intend.
>
>
> cheers
>
>> On Sep 14, 2018, at 12:12 PM, Jiangli Zhou <jiangli.zhou at oracle.com> wrote:
>>
>> Please help review the fix for test TestStressIHOPMultiThread.java. This test has the similar issue as the one fixed earlier this week for JDK-8210193. Please see the bug report for detailed analysis.
>>
>>    webrev: http://cr.openjdk.java.net/~jiangli/8210237/webrev.00/
>>
>>    bug: https://bugs.openjdk.java.net/browse/JDK-8210237
>>
>> Tested with tier1 - tier3. Also tested with tier6 and tier7 with the default CDS archive patch.
>>
>> Thanks,
>>
>> Jiangli
>>