RFR (M) 8005013: Add NMT tests

[Christian Törnqvist]

It might fail if people haven't updated their jtreg, all the machines in the test lab should have been updated though. The updated jtreg is from November 29th so it's not that recent :)

 

Thanks,

Christian

 

From: Coleen Phillimore 
Sent: den 24 januari 2013 15:12
To: Christian Törnqvist
Cc: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR (M) 8005013: Add NMT tests

 

On 1/24/2013 7:21 AM, Christian Törnqvist wrote:

Hi Coleen,

 

> Does having  @library /testlibrary tell jtreg to compile the java code in test/library?  Does this work today or do you have to wait for a version of jtreg to support it?

 

Yes, it'll add /testlibrary to the classpath both when running and compiling the test. The @library tag has been in jtreg for a long time, however it required a relative path until a recent change in jtreg that added support for '/' to add an absolute path from the test root. 

 

Thanks,

Christian

 

From: Coleen Phillimore 
Sent: den 23 januari 2013 18:49
To: HYPERLINK "mailto:hotspot-runtime-dev at openjdk.java.net"hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR (M) 8005013: Add NMT tests

 


Christian,

These tests look great.  Using ProcessBuilder to run commands and checking their output is so much better than the shell scripts we've used to do this.

Does having  @library /testlibrary tell jtreg to compile the java code in test/library?  Does this work today or do you have to wait for a version of jtreg to support it?


Okay, so my question is: if you check it in today, will it fail anywhere because this recent change isn't available somewhere?
Since these are in the Hotspot repository and we don't run jtreg tests in JPRT, then all you need to do is publish the right version of jtreg so people individually don't fail these tests, is that correct?

Thanks,
Coleen





Thanks,
Coleen

On 01/10/2013 06:17 AM, Christian Törnqvist wrote:

Hi everyone,

 

These are tests written for the Native Memory Tracking feature along with a few helper classes, some of the tests depend on the WB API changes made in 8005012 to function properly. Webrev can be found at HYPERLINK "http://cr.openjdk.java.net/%7Eehelin/8005013/webrev.00/"http://cr.openjdk.java.net/~ehelin/8005013/webrev.00/

 

Thanks,

Christian

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20130124/e6a05a9a/attachment-0001.html