RFR: 8168396: Unexpected OOME in GcCauseTest02 and GcTest02

[Dmitry Fazunenko]

Michail,

just in case you're waiting for another approval, new version still 
looks good to me.

Thanks
Dima

On 16.11.2016 21:56, Michail Chernov wrote:
> Hi Thomas,
>
> Thank you for reviewing this.
> I added necessary @requires to both GcCauseTes02.java and 
> GcTest02.java since System.gc() was added to each of them.
> Checked locally - tests were being excluded as expected with 
> -XX:+ExplicitGCInvokesConcurrent
>
> Incremental: http://cr.openjdk.java.net/~mchernov/8168396/webrev.02_01/
> Full: http://cr.openjdk.java.net/~mchernov/8168396/webrev.02/
>
> Thanks,
> Michail
>
> On 16/11/2016 17:07, Thomas Schatzl wrote:
>> Hi Michail,
>>
>> On Mon, 2016-10-31 at 15:44 +0300, Michail Chernov wrote:
>>> Thank you very much for reviewing this!
>>>
>>> Could someone of reviewers please take a look at this change. This
>>> issue is integration blocker.
>>    looks good but for some tests inexplicably not have an @requires
>> ExplicitGCInvokesConcurrent != true.
>>
>> I think at least GcCauseTest02 should have. It does a system.gc at the
>> end, so the jstat output will not reflect the true value of heap
>> dimensions.
>>
>> Thanks,
>>    Thomas
>>
>>
>