RFR(XL/M) : 8178788: wrap JCStress test suite as jtreg tests

Aleksey Shipilev shade at redhat.com
Tue Apr 25 10:47:05 UTC 2017

On 04/19/2017 12:12 AM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8178788/webrev.00/index.html
>> 69903 lines changed: 69903 ins; 0 del; 0 mod;
> (69524 lines are generated)
> Hi all,
> could you please review this patch which adds a jtreg test wrapper for
> jcstress test suite and jtreg tests which run jsctress tests thru this
> wrapper?
> webrev: http://cr.openjdk.java.net/~iignatyev//8178788/webrev.00/index.html 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8178788 testing:

TL;DR: This patch introduces more problems than it solves. Just run the jcstress
tests-all JAR against the tested runtime.

Wrapping jcstress tests with jtreg defies the purpose of jcstress harness --
that is, running lots of tests as fast as it possibly could without affecting
testing quality. For example, by cleverly reusing VMs between the tests, using
Whitebox to deoptimize without restarting the VMs, etc. It really wastes CPU
time to run each test in isolation.

Also, it does not "automatically" work, which defies "easy to run" goal:

Caused by: java.io.FileNotFoundException: Couldn't automatically resolve
dependency for jcstress-tests-all , revision 0.3
Please specify the location using jdk.test.lib.artifacts.jcstress-tests-all
	at jdk.test.lib.artifacts.ArtifactResolver.resolve(ArtifactResolver.java:54)
	at applications.jcstress.JcstressRunner.pathToArtifact(JcstressRunner.java:53)
	... 8 more

Okay, brilliant! How do I configure this, if I run "make test"?

CONF=linux-x86_64-normal-server-release LOG=info make test TEST="hotspot_all"


More information about the hotspot-dev mailing list