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

[Aleksey Shipilev]

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.DefaultArtifactManager.resolve(DefaultArtifactManager.java:37)
	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"


-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20170425/887d69cc/signature.asc>