annotate test/compiler/whitebox/ClearMethodStateTest.java @ 14649:f6301b007a16

6498581: ThreadInterruptTest3 produces wrong output on Windows Summary: There is race condition between os::interrupt and os::is_interrupted on Windows. In JVM_Sleep(Thread.sleep), check if thread gets interrupted, it may see interrupted but not really interrupted so cause spurious waking up (early return from sleep). Fix by checking if interrupt event really gets set thus prevent false return. For intrinsic of _isInterrupted, on Windows, go fastpath only on bit not set. Reviewed-by: acorn, kvn Contributed-by: david.holmes@oracle.com, yumin.qi@oracle.com
author minqi
date Wed, 26 Feb 2014 15:20:41 -0800
parents f9a4b59ae350
children 4ca6dc0799b6
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
14203
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
24 import java.util.function.Function;
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
25
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
26 /*
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
27 * @test ClearMethodStateTest
12073
f99558245e5c 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 12026
diff changeset
28 * @bug 8006683 8007288 8022832
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
29 * @library /testlibrary /testlibrary/whitebox
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
30 * @build ClearMethodStateTest
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
31 * @run main ClassFileInstaller sun.hotspot.WhiteBox
14203
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
32 * @run main/othervm -Xbootclasspath/a:. -Xmixed -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI -XX:CompileCommand=compileonly,SimpleTestCase$Helper::* ClearMethodStateTest
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
33 * @summary testing of WB::clearMethodState()
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
34 * @author igor.ignatyev@oracle.com
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
35 */
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
36 public class ClearMethodStateTest extends CompilerWhiteBoxTest {
10113
4b2eebe03f93 8011971: WB API doesn't accept j.l.reflect.Constructor
iignatyev
parents: 9080
diff changeset
37
9080
b84fd7d73702 8007288: Additional WB API for compiler's testing
iignatyev
parents:
diff changeset
38 public static void main(String[] args) throws Exception {
14203
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
39 CompilerWhiteBoxTest.main(ClearMethodStateTest::new, args);
10113
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
14203
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
42 private ClearMethodStateTest(TestCase testCase) {
10113
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
14203
f9a4b59ae350 8028587: New tests development for intrisics for basic operators - add, neg, inc, dec, sub, mul
iignatyev
parents: 12073
diff changeset
66 if (testCase.isOsr()) {
12073
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 }