annotate test/compiler/whitebox/ClearMethodStateTest.java @ 12986:1b422ef5288a

8025728: Missing volatile specifier for field G1AllocRegion::_alloc_region Summary: The field G1AllocRegion::_alloc_region needs to be declared volatile as it is used with that intention. Otherwise the compiler may generate the code that reloads the value which might have changed in the meantime, leading to spurious crashes. Reviewed-by: iveresov, simonis, tschatzl Contributed-by: Axel Siebenborn <axel.siebenborn@sap.com>
author tschatzl
date Wed, 23 Oct 2013 10:23:06 +0200
parents f99558245e5c
children f9a4b59ae350
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
1 /*
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
2 * Copyright (c) 2013, Oracle and/or its affiliates. All rights reserved.
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
4 *
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
7 * published by the Free Software Foundation.
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
8 *
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
13 * accompanied this code).
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
14 *
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
18 *
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
21 * questions.
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
22 */
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
23
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
24 /*
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
25 * @test ClearMethodStateTest
12073
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
26 * @bug 8006683 8007288 8022832
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
27 * @library /testlibrary /testlibrary/whitebox
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
28 * @build ClearMethodStateTest
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
29 * @run main ClassFileInstaller sun.hotspot.WhiteBox
12026
11237ee74aae 8019915: whitebox testClearMethodStateTest fails with tiered on sparc
iignatyev
parents: 10113
diff changeset
30 * @run main/othervm -Xbootclasspath/a:. -Xmixed -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI -XX:CompileCommand=compileonly,TestCase$Helper::* ClearMethodStateTest
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
31 * @summary testing of WB::clearMethodState()
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
32 * @author igor.ignatyev@oracle.com
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
33 */
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
34 public class ClearMethodStateTest extends CompilerWhiteBoxTest {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
35
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
36 public static void main(String[] args) throws Exception {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
37 for (TestCase test : TestCase.values()) {
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
38 new ClearMethodStateTest(test).runTest();
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
39 }
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
40 }
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
41
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
42 public ClearMethodStateTest(TestCase testCase) {
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
43 super(testCase);
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
44 // to prevent inlining of #method
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
45 WHITE_BOX.testSetDontInlineMethod(method, true);
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
46 }
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
47
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
48
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
49 /**
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
50 * Tests {@code WB::clearMethodState()} by calling it before/after
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
51 * compilation. For non-tiered, checks that counters will be rested after
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
52 * clearing of method state.
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
53 *
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
54 * @throws Exception if one of the checks fails.
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
55 */
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
56 @Override
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
57 protected void test() throws Exception {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
58 checkNotCompiled();
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
59 compile();
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
60 WHITE_BOX.clearMethodState(method);
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
61 checkCompiled();
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
62 WHITE_BOX.clearMethodState(method);
12073
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
63 deoptimize();
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
64 checkNotCompiled();
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
65
12073
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
66 if (testCase.isOsr) {
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
67 // part test isn't applicable for OSR test case
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
68 return;
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
69 }
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
70 if (!TIERED_COMPILATION) {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
71 WHITE_BOX.clearMethodState(method);
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
72 compile(COMPILE_THRESHOLD);
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
73 checkCompiled();
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
74
12073
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
75 deoptimize();
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
76 checkNotCompiled();
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
77 WHITE_BOX.clearMethodState(method);
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
78
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
79 // invoke method one less time than needed to compile
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
80 if (COMPILE_THRESHOLD > 1) {
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
81 compile(COMPILE_THRESHOLD - 1);
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
82 checkNotCompiled();
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
83 } else {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
84 System.err.println("Warning: 'CompileThreshold' <= 1");
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
85 }
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
86
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
87 compile(1);
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
88 checkCompiled();
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
89 } else {
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
90 System.err.println(
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
91 "Warning: part of test is not applicable in Tiered");
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
92 }
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
93 }
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
94 }