Skip to content
Permalink
Browse files

8230402: Allocation of compile task fails with assert: "Leaking compi…

…lation tasks?"

Remove assert that is only hit with hand written edge case tests.

Reviewed-by: kvn, thartmann
  • Loading branch information
Christian Hagedorn
Christian Hagedorn committed May 4, 2020
1 parent 57fbf93 commit bbcb3b638b0488d4705cae73add15455ca3de977
@@ -33,9 +33,6 @@
#include "runtime/handles.inline.hpp"

CompileTask* CompileTask::_task_free_list = NULL;
#ifdef ASSERT
int CompileTask::_num_allocated_tasks = 0;
#endif

/**
* Allocate a CompileTask, from the free list if possible.
@@ -50,8 +47,6 @@ CompileTask* CompileTask::allocate() {
task->set_next(NULL);
} else {
task = new CompileTask();
DEBUG_ONLY(_num_allocated_tasks++;)
assert (WhiteBoxAPI || JVMCI_ONLY(UseJVMCICompiler ||) _num_allocated_tasks < 10000, "Leaking compilation tasks?");
task->set_next(NULL);
task->set_is_free(true);
}
@@ -75,10 +75,6 @@ class CompileTask : public CHeapObj<mtCompiler> {

private:
static CompileTask* _task_free_list;
#ifdef ASSERT
static int _num_allocated_tasks;
#endif

Monitor* _lock;
uint _compile_id;
Method* _method;
@@ -23,12 +23,12 @@

/*
* @test TestOverloadCompileQueues
* @bug 8163511
* @bug 8163511 8230402
* @summary Test overloading the C1 and C2 compile queues with tasks.
* @requires !vm.graal.enabled
* @run main/othervm -XX:-TieredCompilation -XX:CompileThreshold=2 -XX:CICompilerCount=1
* @run main/othervm/timeout=300 -XX:-TieredCompilation -XX:CompileThreshold=2 -XX:CICompilerCount=1
* compiler.classUnloading.methodUnloading.TestOverloadCompileQueues
* @run main/othervm -XX:TieredCompileTaskTimeout=1000 -XX:CompileThresholdScaling=0.001 -XX:CICompilerCount=2
* @run main/othervm/timeout=300 -XX:TieredCompileTaskTimeout=1000 -XX:CompileThresholdScaling=0.001 -XX:CICompilerCount=2
* compiler.classUnloading.methodUnloading.TestOverloadCompileQueues
*/

@@ -37,9 +37,13 @@
import java.lang.reflect.Method;
import java.net.URL;
import java.net.URLClassLoader;
import java.util.Arrays;

public class TestOverloadCompileQueues {
public static final int ITERS = 500; // Increase for longer stress testing
public static final int ITERS_A = 1000; // Increase for longer stress testing

public static int iArr[] = new int[100];

// Some methods to fill up the compile queue
public static void test0() { }
@@ -63,7 +67,65 @@ public static void test17() { }
public static void test18() { }
public static void test19() { }

// More methods that do some more complex things. Therefore, the compiler needs to spend some more time compiling them.
// With 50 methods, a queue size of 10000 is also reached in the second run with TieredCompilation enabled.
public static void testA0() { Arrays.sort(iArr); }
public static void testA1() { Arrays.sort(iArr); }
public static void testA2() { Arrays.sort(iArr); }
public static void testA3() { Arrays.sort(iArr); }
public static void testA4() { Arrays.sort(iArr); }
public static void testA5() { Arrays.sort(iArr); }
public static void testA6() { Arrays.sort(iArr); }
public static void testA7() { Arrays.sort(iArr); }
public static void testA8() { Arrays.sort(iArr); }
public static void testA9() { Arrays.sort(iArr); }
public static void testA10() { Arrays.sort(iArr); }
public static void testA11() { Arrays.sort(iArr); }
public static void testA12() { Arrays.sort(iArr); }
public static void testA13() { Arrays.sort(iArr); }
public static void testA14() { Arrays.sort(iArr); }
public static void testA15() { Arrays.sort(iArr); }
public static void testA16() { Arrays.sort(iArr); }
public static void testA17() { Arrays.sort(iArr); }
public static void testA18() { Arrays.sort(iArr); }
public static void testA19() { Arrays.sort(iArr); }
public static void testA20() { Arrays.sort(iArr); }
public static void testA21() { Arrays.sort(iArr); }
public static void testA22() { Arrays.sort(iArr); }
public static void testA23() { Arrays.sort(iArr); }
public static void testA24() { Arrays.sort(iArr); }
public static void testA25() { Arrays.sort(iArr); }
public static void testA26() { Arrays.sort(iArr); }
public static void testA27() { Arrays.sort(iArr); }
public static void testA28() { Arrays.sort(iArr); }
public static void testA29() { Arrays.sort(iArr); }
public static void testA30() { Arrays.sort(iArr); }
public static void testA31() { Arrays.sort(iArr); }
public static void testA32() { Arrays.sort(iArr); }
public static void testA33() { Arrays.sort(iArr); }
public static void testA34() { Arrays.sort(iArr); }
public static void testA35() { Arrays.sort(iArr); }
public static void testA36() { Arrays.sort(iArr); }
public static void testA37() { Arrays.sort(iArr); }
public static void testA38() { Arrays.sort(iArr); }
public static void testA39() { Arrays.sort(iArr); }
public static void testA40() { Arrays.sort(iArr); }
public static void testA41() { Arrays.sort(iArr); }
public static void testA42() { Arrays.sort(iArr); }
public static void testA43() { Arrays.sort(iArr); }
public static void testA44() { Arrays.sort(iArr); }
public static void testA45() { Arrays.sort(iArr); }
public static void testA46() { Arrays.sort(iArr); }
public static void testA47() { Arrays.sort(iArr); }
public static void testA48() { Arrays.sort(iArr); }
public static void testA49() { Arrays.sort(iArr); }

public static void main(String[] args) throws Throwable {
run();
runA();
}

public static void run() throws Throwable {
Class<?> thisClass = TestOverloadCompileQueues.class;
ClassLoader defaultLoader = thisClass.getClassLoader();
URL classesDir = thisClass.getProtectionDomain().getCodeSource().getLocation();
@@ -84,4 +146,23 @@ public static void main(String[] args) throws Throwable {
System.gc();
}
}

public static void runA() throws Throwable {
Class<?> thisClass = TestOverloadCompileQueues.class;
ClassLoader defaultLoader = thisClass.getClassLoader();
URL classesDir = thisClass.getProtectionDomain().getCodeSource().getLocation();

for (int i = 0; i < ITERS_A; ++i) {
// Load test class with own class loader
URLClassLoader myLoader = URLClassLoader.newInstance(new URL[] {classesDir}, defaultLoader.getParent());
Class<?> testClass = Class.forName(thisClass.getCanonicalName(), true, myLoader);

// Execute all test methods to trigger compilation and fill up compile queue
for (int j = 0; j < 50; ++j) {
Method method = testClass.getDeclaredMethod("testA" + j);
method.invoke(null);
method.invoke(null);
}
}
}
}

0 comments on commit bbcb3b6

Please sign in to comment.