Review request: 7196801: NPG: Fix java/lang/management/MemoryMXBean/LowMemoryTest2

[Stefan Karlsson]

On 10/11/13 7:44 PM, Coleen Phillimore wrote:
>
> Stefan,
>
> I added a check about the count exceeding 1 000 000 because it causes 
> the test to create an invalid class file.   With the shell script 
> changes, it won't happen but I would be happy if that check were there 
> so no one ever has to debug this thinking it has a corrupted constant 
> pool again.

OK. I though that was old debugging left-overs.

New webrev:
http://cr.openjdk.java.net/~stefank/7196801/webrev.01/

thanks,
StefanK

>
> Thanks,
> Coleen
>
> On 10/11/2013 1:33 PM, Stefan Karlsson wrote:
>> Please, review this test fix.
>>
>> http://cr.openjdk.java.net/~stefank/7196801/webrev.00/
>>
>> Most of the fix was implemented by Coleen, I've only done some small 
>> tweaks and updated some comments.
>>
>> The fix depends on these changes:
>>   JDK-8025996 Track metaspace usage when metaspace is expanded
>>   JDK-8024547 MaxMetaspaceSize should limit the committed memory used 
>> by the metaspaces
>>
>> Both of these changesets have been pushed to hotspot-gc and are 
>> currently undergoing testing. LowMemoryTest2 can only be removed from 
>> the test/ProblemList.txt when both JDK-8025996 and JDK-8024547 have 
>> propagated through all necessary repositories.
>>
>> thanks,
>> StefanK
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20131011/1444bbf9/attachment.html