I agree with Erik.
It is better to just exclude this test from run with JFR enabled.
Adding /othervm should probably solve the problem.
Thanks,
Serguei
On 10/7/13 9:01 AM, Erik Gahlin wrote:
I don't think the test should run with JFR enabled. I would ask SQE
not to run unit tests with JFR and only add /othervm.
Erik
Jaroslav Bachorik skrev 10/7/13 3:59 PM:
The test captures the number of loaded classes right at the start and
then checks the diffs when it's finished. However, it seems that
there might by some async class loading still going on, initiated by
JFR.
The patch simply adds a loop to wait for the number of loaded classes
to settle before continuing. This should prevent the test failing
with JFR intermittently.
Issue: https://bugs.openjdk.java.net/browse/JDK-7144200
Webrev: http://cr.openjdk.java.net/~jbachorik/7144200/webrev.00/
Cheers,
-JB-
