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

[Stefan Karlsson]

On 10/11/13 8:04 PM, Coleen Phillimore wrote:
>
> Looks good.

Thanks, Coleen.

StefanK

> Thanks!
> Coleen
>
> On 10/11/2013 2:02 PM, Stefan Karlsson wrote:
>> 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/bdeb5e5d/attachment.html