RFR: 8255011: [TESTBUG] UnexpectedDeoptimizationAllTest.java timed out [v3]

[Nils Eliasson]

> Hi, 
> 
> This patch updates the code cache stress tests. I haven't been able to reproduce or retrieve a core file. 
> 
> What I can see is that the tests provokes compile storms, and that the single C1 thread (on a 4CPU system) sometimes has trouble keeping up. A factor may also be that the tests run time scale with the timeout time - so that the time allotted as margin before the timeout is only 20% of the total runtime. Combining this with Xcomp, and that the test may run concurrently with other stress tests, it is reasonable that a timeout may occur.
> 
> I suggest to cap the tests to 60 seconds of testing. I've experimented with meassuring how much work is done and use that as a metric - but the different tests that use the CodeCacheStressRunner has completely different profiles. 
> 
> In UnexpectedDeoptimizationAllTest.java I have adjusted the sleep time to 100 millis between the invalidations of the entire code cache.
> 
> In UnexpectedDeoptimizationTest.java  I have added a sleep of 10 miilis between deoptimizing parts of the stack. The idea is to give the stack time to growth a bit before the next deoptimization. Otherwise the test might end up running mostly in the interpreter.
> 
> Please review,
> Nils Eliasson

Nils Eliasson has updated the pull request incrementally with one additional commit since the last revision:

  add comment

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1030/files
  - new: https://git.openjdk.java.net/jdk/pull/1030/files/5050a2ef..5d72bf94

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1030&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1030&range=01-02

  Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1030.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1030/head:pull/1030

PR: https://git.openjdk.java.net/jdk/pull/1030