On 09/06/2017 19:36, Igor Ignatyev wrote:
Alan, Chris,

it seems I had some problems uploading the webrev, I have reuploaded it to the 
same place -- 
http://cr.openjdk.java.net/~iignatyev//8181761/webrev.02/index.html

-- Igor

This mostly looks okay but there are still a few tests being penalized with a long list of classes in their @build tag.

Take LotsOfUpdatesTest as example. It's a trivial test to launch a child VM with `ulimit -n` set to a small value. Its use of ProcessTools to exec the child is penalized by adding this to its test description:

  31  * @build jdk.test.lib.Utils
  32  *        jdk.test.lib.Asserts
  33  *        jdk.test.lib.JDKToolFinder
  34  *        jdk.test.lib.JDKToolLauncher
  35  *        jdk.test.lib.Platform
  36  *        jdk.test.lib.process.*

Can we get a bug submitted to cull most of this? A lot of our tests needs to launch child VMs and a simple ProcessTools with minimal dependences would be preferred.

-Alan



Reply via email to