RFR: JDK-8218128: vmTestbase/nsk/jvmti/ResourceExhausted/resexhausted003 and 004 use wrong path to test classes

Leonid Mesnik leonid.mesnik at oracle.com
Fri Mar 8 18:24:18 UTC 2019



> On Mar 8, 2019, at 4:44 AM, Gary Adams <gary.adams at oracle.com> wrote:
> 
> The tests have been excluded for a long time now.
> It appears that the tests are working at this time.
> I'll have more confidence in taking them off the ProblemList when
> the testing has been done on more platforms.
> 
Please add this information into bug, to make clear that this change was intentional.

> Please let me know what you'd like the description to say to bring it up to date.

I think that it is enough just to change PermGen to Metaspace.

Leonid
> 
> On 3/7/19, 4:06 PM, Leonid Mesnik wrote:
>> Hi
>> The problemlist
>> 
>> http://cr.openjdk.java.net/~gadams/8218128/webrev.00/test/hotspot/jtreg/ProblemList.txt.udiff.html
>> 
>> Tests are excluded because of
>> https://bugs.openjdk.java.net/browse/JDK-6606767
>> which is not fixed yet. Why do you want to remove them from
>> problemlist?
>> 
>> Also I think it is a good time to update test desrption in the
>> 
>> http://cr.openjdk.java.net/~gadams/8218128/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ResourceExhausted/resexhausted003/TestDescription.java.udiff.html
>> 
>> Currently it says about PermGen OOME.
>> 
>> Leonid
>> 
>> On Thu, 2019-03-07 at 13:57 -0500, Gary Adams wrote:
>>> This proposed fix will restore the ResourceExhausted tests.
>> Test 3 and 4 were on the ProblemList because of the potential
>> path issues in finding the correct classes. This change searches the
>> test.class.path for the appropriate vmTestbase classes rather than
>> using
>> incorrect settings on the command line.
>> 
>> Some clean up has been done to remove quarantine keyword
>> and @ignore directives. Should additional clean up be done to remove
>> bug numbers, etc.?
>> 
>> TEST.PROPERTIES were added so test 3 so it is consistent with the
>> other
>> tests
>> in the group.
>> 
>>    Issue: https://bugs.openjdk.java.net/browse/JDK-8218128
>>    Webrev:
>> http://cr.openjdk.java.net/~gadams/8218128/webrev.00/index.html
>> 
>> Local testing has been successful on a linux-x64-debug build.
>> Testing on mach5 for other platforms next.
>> 
> 



More information about the serviceability-dev mailing list